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 url.scheme to http client metrics #357

Merged
merged 2 commits into from
Oct 2, 2023

Conversation

lmolkova
Copy link
Contributor

Fixes #356

Changes

Adds url.scheme to HTTP client metrics

Merge requirement checklist

@lmolkova lmolkova requested review from a team September 29, 2023 19:08
@lmolkova lmolkova force-pushed the url-scheme-http-client branch from cbdc9b8 to b26a6b0 Compare September 29, 2023 19:17
@joaopgrassi joaopgrassi merged commit fe1797e into open-telemetry:main Oct 2, 2023
8 checks passed
breedx-splk pushed a commit to breedx-splk/semantic-conventions that referenced this pull request Oct 3, 2023
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.

Should HTTP client metrics have url.scheme?
5 participants