HTTP2: Implement GoAway handling for the non-error case

croservices / cro-http

HTTP (including HTTPS and HTTP/2) support for the Cro library for building distributed systems in Raku.

https://cro.services/

Artistic License 2.0

49 stars 26 forks source link

HTTP2: Implement GoAway handling for the non-error case #180

Closed patrickbkr closed 1 year ago

patrickbkr commented 1 year ago

This approach creates a direct communication channel from the Cro::HTTP2::ResponseParser to Cro::HTTP::Client via the $go-away-supply. I'm feeling a bit uneasy about this coupling. Any better ideas of how to solve this are welcome.

In general this PR fixes Cro::Client to not try to re-use connections the remote end has already closed.

patrickbkr commented 1 year ago

As I didn't come up with any better idea in the last few months, I'd like to move forward with this PR. Thus I'm removing the draft status.