jetty.project icon indicating copy to clipboard operation
jetty.project copied to clipboard

Fixes #5685 - AsyncProxyServlet calls onProxyResponseSuccess() when internally it throws "Response header too large" exception.

Open sbordet opened this issue 1 year ago • 0 comments
trafficstars

  • Introduced "maxResponseHeadersSize" parameter to AbstractProxyServlet.
  • Introduced HttpGenerator.maxResponseHeadersSize and added checks to not exceed it.
  • Fixed HttpParser to generate a 400 in case HttpParser.maxHeaderBytes are exceeded for a response.
  • HTTP2Flusher now resets streams in case of failures.
  • Removed cases in HTTP2Session where a GOAWAY frame was generated with lastStreamId=0. GOAWAY.lastStreamId=0 means that not a single request was processed by the server, which was obviously incorrect.
  • Written tests for both ProxyHandler and ProxyServlet about max response headers size exceeded.

sbordet avatar Oct 07 '24 21:10 sbordet