Merge pull request #193 from patrickbkr/http2-multi-stream-window-upd…

…ates Fix HTTP2 connections with multiple streams stalling
croservices · Aug 9, 2024 · 39022ce · 39022ce
2 parents d42bd97 + d98aca1
commit 39022ce
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/lib/Cro/HTTP2/FrameParser.pm6 b/lib/Cro/HTTP2/FrameParser.pm6
@@ -64,7 +64,16 @@ class Cro::HTTP2::FrameParser does Cro::Transform does Cro::ConnectionState[Cro:
                                                  stream-identifier => $sid,
                                                  conn => $packet.connection);
                             if $result ~~ Cro::HTTP2::Frame::Data {
-                                unless $result.end-stream {
+                                if $result.end-stream {
+                                    start {
+                                        my $bytes = $result.data.bytes;
+                                        $connection-state.window-size.emit:
+                                            Cro::HTTP2::Frame::WindowUpdate.new:
+                                                stream-identifier => 0,
+                                                flags => 0, increment => $bytes;
+                                    }
+                                }
+                                else {
                                     start {
                                         my $bytes = $result.data.bytes;
                                         $connection-state.window-size.emit: