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.

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

Propagators API: RFC7230 has been obsoleted by RFC9110 and RFC9112 #3444

Closed
hiroyaonoe opened this issue Apr 26, 2023 · 1 comment · Fixed by #4153
Closed

Propagators API: RFC7230 has been obsoleted by RFC9110 and RFC9112 #3444

hiroyaonoe opened this issue Apr 26, 2023 · 1 comment · Fixed by #4153
Assignees
Labels
spec:context Related to the specification/context directory triage:accepted:ready

Comments

@hiroyaonoe
Copy link
Contributor

What are you trying to achieve?

RFC7230, which specifies HTTP header fields, has been obsoleted by RFC9110 and RFC9112 in June 2022.

In order to increase compatibility, the key/value pairs MUST only consist of US-ASCII characters
that make up valid HTTP header fields as per [RFC 7230](https://tools.ietf.org/html/rfc7230#section-3.2).

Shouldn't we update the Propagators API documentation to reference RFC9110 Section 5 (or Section 6.3), which specifies the latest HTTP Header Fields specification?
RFC9110 is a compilation of HTTP specifications since HTTP/2 and HTTP/3 are newly introduced.
Therefore, there are no changes to the HTTP header fields specification, and this issue does not alter the Propagators API specification.

What did you expect to see?

 In order to increase compatibility, the key/value pairs MUST only consist of US-ASCII characters 
 that make up valid HTTP header fields as per [RFC9110](https://tools.ietf.org/html/rfc9110/#name-fields).

Additional context.
Propagators API#TextMap Propagator

@hiroyaonoe hiroyaonoe added the spec:context Related to the specification/context directory label Apr 26, 2023
@lmolkova
Copy link
Contributor

@hiroyaonoe thank you for reporting this! would you be willing to send to PR to update the RFC link? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:context Related to the specification/context directory triage:accepted:ready
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

5 participants