spring-framework

Spring Framework

Tree: 02f0f92a72

Brian Clozel 764336f0f2 Fix Jetty DataBufferFactory memory leak Prior to this commit, gh-32097 added native support for Jetty for both client and server integrations. The `JettyDataBufferFactory` was promoted as a first class citizen, extracted from a private class in the client support. To accomodate with server-side requirements, an extra `buffer.retain()` call was performed. While this is useful for server-side support, this introduced a bug in the data buffer factory, as wrapping an existing chunk means that this chunk is already retained. This commit fixes the buffer factory implementation and moved existing tests from mocks to actual pooled buffer implementations from Jetty. The extra `buffer.retain()` is now done from the server support, right before wrapping the buffer. Fixes gh-35319	4 months ago
..
src	Fix Jetty DataBufferFactory memory leak	4 months ago
spring-web.gradle	Simplify dependency management for JUnit artifacts	6 months ago

Brian Clozel 764336f0f2 Fix Jetty DataBufferFactory memory leak

Prior to this commit, gh-32097 added native support for Jetty for both
client and server integrations. The `JettyDataBufferFactory` was
promoted as a first class citizen, extracted from a private class in the
client support. To accomodate with server-side requirements, an extra
`buffer.retain()` call was performed.
While this is useful for server-side support, this introduced a bug in
the data buffer factory, as wrapping an existing chunk means that this
chunk is already retained.

This commit fixes the buffer factory implementation and moved existing
tests from mocks to actual pooled buffer implementations from Jetty.
The extra `buffer.retain()` is now done from the server support, right
before wrapping the buffer.

Fixes gh-35319

4 months ago

src

Fix Jetty DataBufferFactory memory leak

4 months ago

spring-web.gradle

Simplify dependency management for JUnit artifacts

6 months ago