Skip to content

Conversation

@conghuo
Copy link

@conghuo conghuo commented Jul 22, 2024

Generate a new Span during the rewrite phase and parse the TraceID to place it in the new header (print $http_trace_id to the access log)

conghuo added 2 commits July 22, 2024 18:34
Generate a new Span during the rewrite phase and parse the TraceID to place it in the new header (for easy access log printing)
Generate a new Span during the rewrite phase and parse the TraceID to place it in the new header (for easy access log printing)
@wu-sheng
Copy link
Member

You don't need a new span AFAIK.
We should not put trace id in header separately, this is not by protocol.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants