-
Notifications
You must be signed in to change notification settings - Fork 74
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.
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
Fixes wrong headline level of tracestate Header Field Values #404
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Co-authored-by: Armin Ruech <armin.ruech@dynatrace.com>
@@ -235,7 +235,7 @@ In order to increase interoperability across multiple protocols and encourage su | |||
|
|||
Vendors MUST expect the header name in any case (upper, lower, mixed), and SHOULD send the header name in lowercase. | |||
|
|||
##### tracestate Header Field Values | |||
### Header Values |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this header will be harder to link to directly with anchor links
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be the same for Header Name
- it isn't unique either and the document processor is smart enough to create
- https://www.w3.org/TR/trace-context/#header-name
and - https://www.w3.org/TR/trace-context/#header-name-0
to distinguish between same headlines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please split PR into two: fixing indents and rename of a title? Perhaps better title can be found that is both - unique in a document and clearer.
See my reply to your comment. To keep this simple, I am reverting the change of the header names here and will create a new PR later. |
closes #403
Preview | Diff