Skip to content

feat: add http.route attribute to open telemetry on requests#640

Merged
gavinbarron merged 20 commits into
mainfrom
feat/add-otel-http.route
Feb 6, 2026
Merged

feat: add http.route attribute to open telemetry on requests#640
gavinbarron merged 20 commits into
mainfrom
feat/add-otel-http.route

Conversation

@gavinbarron

@gavinbarron gavinbarron commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

Closes #577

@gavinbarron gavinbarron requested a review from a team as a code owner February 4, 2026 00:16
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
… IDisposable'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>

@baywet baywet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in addition to the comment left in place, the unit tests are a bit of garbage. Lots of repetition, fails to cover a sample with query string parameters.
I suggest they are re-written to use a Theory with InlineData instead.

Comment thread src/http/httpClient/HttpClientRequestAdapter.cs Outdated
@github-project-automation github-project-automation Bot moved this to In Progress 🚧 in Kiota Feb 4, 2026
@baywet baywet requested a review from Copilot February 4, 2026 13:18
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as outdated.

…ts.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
Comment thread tests/http/httpClient/HttpClientRequestAdapterObservabilityTests.cs Fixed
This was referenced May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

kiota should propagate client-side route information to Open Telemetry http.route tag

4 participants