Skip to content

Conversation

@fl4via
Copy link
Member

@fl4via fl4via commented Mar 25, 2023

…tify clientCallback instead of IOException complaining of invalid connection state. At HttpClientTestCase.testSslServerIdentity, enforce that the exception received is ClosedChannelException

Jira: https://issues.redhat.com/browse/UNDERTOW-2249
Altenative to PR #1454
Which option to be chosen is being discussed at Zulip: https://wildfly.zulipchat.com/#narrow/stream/174183-undertow/topic/Should.20ClientConnection.20request.20get.20IOException.20when.20closed.3F

Ported to 2.4.x by #1821

@fl4via fl4via added duplicate Duplicates other pull request(s) enhancement Enhances existing behaviour or code under verification Currently being verified (running tests, reviewing) before posting a review to contributor labels Mar 25, 2023
@fl4via fl4via removed under verification Currently being verified (running tests, reviewing) before posting a review to contributor duplicate Duplicates other pull request(s) labels Apr 14, 2023
@fl4via fl4via added the waiting CI check Ready to be merged but waiting for CI check label Apr 14, 2023
…tify clientCallback instead of IOException complaining of invalid connection state. At HttpClientTestCase.testSslServerIdentity, enforce that the exception received is ClosedChannelException

Signed-off-by: Flavia Rainone <[email protected]>
@fl4via fl4via added next release This PR will be merged before next release or has already been merged (for payload double check) and removed waiting CI check Ready to be merged but waiting for CI check labels Oct 17, 2025
@fl4via fl4via merged commit c342da9 into undertow-io:main Oct 17, 2025
13 checks passed
@fl4via fl4via deleted the UNDERTOW-2249 branch October 17, 2025 00:53
@fl4via fl4via added the new feature / API change New feature to be introduced or a change to the API (non suitable to minor releases) label Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhances existing behaviour or code new feature / API change New feature to be introduced or a change to the API (non suitable to minor releases) next release This PR will be merged before next release or has already been merged (for payload double check)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant