Avoid unnecessary `RST_STREAM` frames in HTTP/2 implementation

elixir-mint / mint

Functional HTTP client for Elixir with support for HTTP/1 and HTTP/2 🌱

Apache License 2.0

1.37k stars 112 forks source link

Avoid unnecessary `RST_STREAM` frames in HTTP/2 implementation #433

Closed v0idpwn closed 6 months ago

v0idpwn commented 7 months ago

When a stream is locally closed and a EndStream flag is received, Mint is sending RST_STREAM with NO_ERROR. This is unnecessary as a stream can be deemed closed by both sides at this point, and this frame is just ignored.

whatyouhide commented 7 months ago

Good catch @v0idpwn. Wanna make a PR to fix this? 🙃

v0idpwn commented 7 months ago

Yeah, will try to send one soon!