search

jetty / jetty.project

Eclipse Jetty® - Web Container & Clients - supports HTTP/2, HTTP/1.1, HTTP/1.0, websocket, servlets, and more
https://eclipse.dev/jetty
Other
3.87k stars 1.91k forks source link

Fix `HTTP2Test.testLargeRequestHeaders` #12536

Closed lorban closed 1 week ago

lorban commented 1 week ago

Make sure ReadOnlyBufferException is not thrown from read-only methods of RetainableByteBuffer.Mutable.Wrapper, even when the buffer is retained.

This fixes such failures: https://jenkins.webtide.net/blue/organizations/jenkins/jetty.project/detail/jetty-12.1.x-ee9-ResponseTest/1/tests/

joakime commented 1 week ago

I'd like to see this fixed/merged soon, as it is breaking other 12.1.x PRs

https://jenkins.webtide.net/blue/organizations/jenkins/jetty.project/detail/PR-12531/5/tests