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

Add HTTP/3 #2507

Merged
merged 3 commits into from
Apr 27, 2022
Merged

Add HTTP/3 #2507

merged 3 commits into from
Apr 27, 2022

Conversation

ladd
Copy link
Contributor

@ladd ladd commented Apr 22, 2022

Changes

Minor change to include HTTP/3 as a listed protocol.

I considered changing the phrasing: HTTP/2instead of HTTP 2. Thoughts?

@ladd ladd requested review from a team April 22, 2022 21:25
@arminru
Copy link
Member

arminru commented Apr 26, 2022

I considered changing the phrasing: HTTP/2instead of HTTP 2. Thoughts?

I think either is fine but if you change it for 2 please do the same for 1.0 and 1.1 so we are consistent.

@ladd
Copy link
Contributor Author

ladd commented Apr 26, 2022

I considered changing the phrasing: HTTP/2instead of HTTP 2. Thoughts?

I think either is fine but if you change it for 2 please do the same for 1.0 and 1.1 so we are consistent.

Done.

@reyang reyang merged commit 7409f4f into open-telemetry:main Apr 27, 2022
@arminru arminru added area:semantic-conventions Related to semantic conventions semconv:HTTP labels Apr 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:semantic-conventions Related to semantic conventions semconv:HTTP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants