RFR: 8352623: MultiExchange should cancel exchange impl if responseFilters throws
Daniel Jeliński
djelinski at openjdk.org
Fri Mar 21 17:32:06 UTC 2025
On Fri, 21 Mar 2025 16:46:56 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
> When adding authClient.close() noAuthClient.close() to UnauthorizedTest::teardown, the test fails in jtreg timeout.
> Usage of the ReferenceTracker reveals that some streams have been left dangling, and this can be tracked to the MultiExchange not cancelling the stream when responseFilters throws.
LGTM. Should we handle non-checked exceptions as well?
-------------
Marked as reviewed by djelinski (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/24164#pullrequestreview-2706712511
More information about the net-dev
mailing list