feat(otlphttp): configure response body size limit - #8847
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8847 +/- ##
=====================================
Coverage 88.5% 88.5%
=====================================
Files 333 333
Lines 21101 21129 +28
=====================================
+ Hits 18687 18718 +31
+ Misses 2414 2411 -3
🚀 New features to boost your workflow:
|
|
Heads-up while refreshing this branch: current The two overlap, so before I rebase I'd like to align on the intended shape:
Happy to rebase under either decision — the option plumbing (oconf + trace/metric/log exporters) is already in place on this branch. |
Compose WithMaxResponseBodySize with the package-level 4 MiB default, keep every response-body read on http.MaxBytesReader including MaxInt64, and name the full exporter modules in the changelog. Signed-off-by: King Star <mcxin.y@gmail.com>
|
Merged current
Verified with |
Signed-off-by: King Star <mcxin.y@gmail.com>
Summary
Fixes #8819.
WithMaxResponseBodySizeto the OTLP/HTTP trace, metric, and log exporters.Verification
go test ./...passes in the trace, metric, and log OTLP/HTTP exporter modules.make precommitcompleted generation, formatting/lint, README validation, and module consistency checks.bridge/opentracinggRPC test, which failed locally with an EOF while reading the server preface; no changed package failed.