Skip to content

WFLY-21293 Session.requestDone(...) invoked outside of ControlPoint boundary#19532

Merged
rhusar merged 2 commits intowildfly:mainfrom
pferraro:WFLY-21293
Jan 29, 2026
Merged

WFLY-21293 Session.requestDone(...) invoked outside of ControlPoint boundary#19532
rhusar merged 2 commits intowildfly:mainfrom
pferraro:WFLY-21293

Conversation

@pferraro
Copy link
Contributor

@pferraro pferraro commented Jan 16, 2026

Modification of #19491, using an exchange attachment override timing of Session.requestDone(...) invocations.


Warning

This section is automatically managed by the WildFly Bot. Manual modifications will be overwritten.

Additional WildFly Issue Links Found:

More information about the wildfly-bot[bot]

@wildfly-bot wildfly-bot bot requested review from fl4via, rhusar and ropalka January 16, 2026 21:33
@pferraro
Copy link
Contributor Author

pferraro commented Jan 16, 2026

Per @scottmarlow, this resolves the TCK regression seen following #19491.

[javatest.batch] Finished Test: PASSED........com/sun/ts/tests/websocket/platform/jakarta/websocket/server/handshakerequest/authenticatedlogoff/WSCClient.java#connectionHasBeenClosedWithStatus1008AfterInvalidationTest
[javatest.batch] ********************************************************************************
[javatest.batch] Number of tests completed: 1 (1 passed, 0 failed, 0 with errors)

@pferraro pferraro marked this pull request as draft January 17, 2026 09:16
@pferraro pferraro force-pushed the WFLY-21293 branch 3 times, most recently from 8eb9f0b to b252d87 Compare January 20, 2026 16:43
@pferraro pferraro marked this pull request as ready for review January 20, 2026 16:46
@rhusar
Copy link
Member

rhusar commented Jan 23, 2026

Just merged the undertow upgrade, lets retest

@rhusar
Copy link
Member

rhusar commented Jan 23, 2026

/retest

Copy link
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great to me, awesome stuff @pferraro

This should stabilize the remaining clustering tests flakiness.

Left a tiny note.

@rhusar rhusar merged commit e541812 into wildfly:main Jan 29, 2026
14 checks passed
@rhusar
Copy link
Member

rhusar commented Jan 29, 2026

Thanks @pferraro and @ropalka !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants