Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use trace_id in logs #43

Merged
merged 2 commits into from
Dec 1, 2021
Merged

Use trace_id in logs #43

merged 2 commits into from
Dec 1, 2021

Conversation

emschwartz
Copy link
Contributor

Also use tracing spans to propagate structured fields

Resolves FP-945

@emschwartz emschwartz requested a review from a team as a code owner November 30, 2021 17:39
@linear
Copy link

linear bot commented Nov 30, 2021

FP-945 Use trace ID in Proxy

hatchan
hatchan approved these changes Dec 1, 2021
View reviewed changes
@emschwartz
Use trace_id in logs
7d07306 
Also use tracing spans to propagate structured fields
@emschwartz
Fix typo
1d6e1b8
@emschwartz emschwartz merged commit 3fe48f6 into main Dec 1, 2021
@emschwartz emschwartz deleted the trace_id branch December 1, 2021 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hatchan hatchan hatchan approved these changes

@Zagitta Zagitta Awaiting requested review from Zagitta

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@emschwartz @hatchan