apple / swift-nio-http2

HTTP/2 support for SwiftNIO
https://swiftpackageindex.com/apple/swift-nio-http2/main/documentation/niohttp2
Apache License 2.0
465 stars 82 forks source link

Remote pushes with invalid headers leave a stream dangling #333

Open Lukasa opened 2 years ago

Lukasa commented 2 years ago

When we receive a push promise, we first create the promised stream and then receive the frame. This is fine, but if receiving the frame produces an error (e.g. because the headers are invalid) then we never delete the stream again. That's no good.