Skip to content

Conversation

@adomaskizogian
Copy link
Contributor

@adomaskizogian adomaskizogian commented Jan 6, 2026

fixes #8258

get http request body from GetBody if it is set, otherwise fallback to Body field.
This allows reusing the request body bytes if the same http request struct is reused on subsequent requests.

@adomaskizogian adomaskizogian requested review from a team and dmathieu as code owners January 6, 2026 15:02
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.3%. Comparing base (7d8be10) to head (260a8b3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #8352   +/-   ##
=====================================
  Coverage   82.3%   82.3%           
=====================================
  Files        186     186           
  Lines      13765   13772    +7     
=====================================
+ Hits       11330   11337    +7     
  Misses      2030    2030           
  Partials     405     405           
Files with missing lines Coverage Δ
instrumentation/net/http/otelhttp/transport.go 95.6% <100.0%> (+0.2%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmathieu dmathieu merged commit 464789f into open-telemetry:main Jan 9, 2026
28 checks passed
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.

[Bug]: Unable to reuse http.Request with body when client is instrumented with otelhttp.NewTransport

3 participants