Skip to content

Moved max body bytes to BodyReaderHandler#241

Merged
nt0xa merged 1 commit into
mainfrom
fix/http-body-reader-limit
Jun 16, 2026
Merged

Moved max body bytes to BodyReaderHandler#241
nt0xa merged 1 commit into
mainfrom
fix/http-body-reader-limit

Conversation

@nt0xa

@nt0xa nt0xa commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Removed MaxBytesHandler, body limit now is in BodyReaderHandler

Removed MaxBytesHandler, body limit now is in BodyReaderHandler
@nt0xa nt0xa added maintenance Deps update, tool changes, configuration changes patch Patch changes labels Jun 16, 2026
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.54%. Comparing base (a367f81) to head (89d116d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
internal/cmd/server/http.go 0.00% 5 Missing ⚠️
pkg/httpx/handlers.go 77.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
+ Coverage   42.06%   42.54%   +0.47%     
==========================================
  Files         210      213       +3     
  Lines        8052     8067      +15     
==========================================
+ Hits         3387     3432      +45     
+ Misses       4384     4359      -25     
+ Partials      281      276       -5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nt0xa nt0xa merged commit aaea3bf into main Jun 16, 2026
4 checks passed
@nt0xa nt0xa deleted the fix/http-body-reader-limit branch June 16, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Deps update, tool changes, configuration changes patch Patch changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant