Skip to content

HTTP transition for wsgi #2425

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
Merged
merged 53 commits into from
Apr 19, 2024
Merged

HTTP transition for wsgi #2425

merged 53 commits into from
Apr 19, 2024

Conversation

lzchen
Copy link
Contributor

@lzchen lzchen commented Apr 15, 2024 •
edited
Loading

Part of: #2453

Http migration plan for wsgi.

For changes NOT related to span attributes, we simply snap to the new semantic conventions like for span name.

Credits to @lmolkova for the original pr.

@lzchen lzchen requested a review from a team April 15, 2024 21:41
@github-actions github-actions bot requested a review from ocelotl April 15, 2024 21:41
ocelotl
ocelotl suggested changes Apr 16, 2024
View reviewed changes
@lzchen lzchen merged commit 2317adc into open-telemetry:main Apr 19, 2024
@lzchen lzchen deleted the wsgi branch April 19, 2024 17:59
@lzchen lzchen mentioned this pull request Apr 23, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xrmx xrmx xrmx left review comments

+2 more reviewers

@lmolkova lmolkova lmolkova left review comments

@ocelotl ocelotl ocelotl approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@ocelotl ocelotl

Labels
None yet
Projects
@xrmx's Python PR digest
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lzchen @xrmx @lmolkova @ocelotl