Skip to content

fix: bound HTTPServer request resources#2333

Open
zeitlinger wants to merge 1 commit into
mainfrom
agent/bound-httpserver-resources
Open

fix: bound HTTPServer request resources#2333
zeitlinger wants to merge 1 commit into
mainfrom
agent/bound-httpserver-resources

Conversation

@zeitlinger

Copy link
Copy Markdown
Member

Summary

  • use a fixed-size default executor with a bounded queue and non-blocking rejection
  • close rejected authenticated request bodies instead of draining an unbounded body
  • close rejected exchanges after sending HTTP 403

This is the focused replacement for the #2284 portion of #2297.

Fixes #2284

Ongoing discussion

None currently. Earlier review feedback about preserving default concurrency and closing rejected exchanges is incorporated here.

Validation

  • mise run lint:fix
  • mise run build
  • ./mvnw test -pl prometheus-metrics-exporter-httpserver -Dcoverage.skip=true -Dcheckstyle.skip=true

Signed-off-by: Gregor Zeitlinger <gregor.zeitlinger@grafana.com>
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ API changes detected — maintainer review required

This PR modifies the published API diff for the following module(s):

  • prometheus-metrics-exporter-httpserver

Please review the changes in docs/apidiffs/current_vs_latest/ carefully before approving.

@zeitlinger
zeitlinger marked this pull request as ready for review July 23, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

# Security Report: Thread Pool Exhaustion via Slow Clients (Slowloris-style DoS) in prometheus/client_java HTTPServer

1 participant