Hi Arturo

I just realized that we have introduced a memory leak in H2 protocol
handler with the stream priority support.

Why do we have a map that we populate but never ever make any use of?
This makes no sense. Moreover, we never ever clean it up. This is
really bad.

https://github.com/apache/httpcomponents-core/blob/master/httpcore5-h2/src/main/java/org/apache/hc/core5/http2/impl/nio/AbstractH2StreamMultiplexer.java#L144

Please either find a way to push PriorityValue into H2Streams whose
streams actually get cleaned up once they reach the end of their life
cycle or remove it entirely.

Once again ,we have introduced a feature with no good reason and no
clear use case in mind and now it causes us troubles.

Oleg

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to