protojure / lib

A collection of libraries to support Protojure applications at runtime
https://protojure.readthedocs.io
Apache License 2.0
64 stars 14 forks source link

Implement proper connection-close subscription in Undertow #71

Closed ghaskins closed 4 years ago

ghaskins commented 4 years ago

The current logic has a problem in that it registers a per exchange subscription to a per-connection resource. This can result in effectively a resource leak if a client employs a 1:N connection:request model, which http2 encourages.

This patch addresses this by building an explicit per-exchange subscription substrate, thereby allowing resources to be freed based on the lifecycle of a request idependently from the connection lifecycle.

Signed-off-by: Greg Haskins greg@manetu.com