Fixes #12313 - Jetty 12 ee9/ee10 doesn't invoke callbacks when h2 client sends RST_STREAM.

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.86k stars 1.91k forks source link

Closed sbordet closed 3 weeks ago

sbordet commented 1 month ago

Fixed invocation of AsyncListener.onError(), now called even if the response is already committed, in both EE9 and EE10.
Reworked EE9 HttpChannel state machine in case of failures to be like EE10's. In particular, calling abort now is a state change, rather than a failure of the Handler callback. In this way, the handle() loop continues, enters case TERMINATED, and the callback is completed in onCompleted().
Fixed EE9 handling of idle timeout in HttpChannel.onRequest(), that was missing.

wendigo commented 3 weeks ago

Is there a chance this will land in 12.0.15?

lorban commented 3 weeks ago

@wendigo it's very likely that this PR gets merged before 12.0.15 gets released.