On Fri, 21 Mar 2025 17:29:48 GMT, Daniel Jeliński wrote:
> LGTM. Should we handle non-checked exceptions as well?
hmm... I believe we should. Will update.
-
PR Comment: https://git.openjdk.org/jdk/pull/24164#issuecomment-2744180924
On Fri, 21 Mar 2025 19:55:47 GMT, Daniel Fuchs 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 M
> 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 responseFilte
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.
On Fri, 21 Mar 2025 16:46:56 GMT, Daniel Fuchs 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 Multi