Croway opened a new pull request, #28505:
URL: https://github.com/apache/camel/pull/28505

   Jira: https://issues.apache.org/jira/browse/CAMEL-25917
   
   Backport of #28237 to `camel-4.22.x` (cherry-pick of 56aa5f4970a6).
   
   On a WebSocket path whose producer applies a security provider or allowed 
roles, a connection accepted while only the producer served the path, before 
its consumer was first started, did not run the custom `handlers` that the 
consumer configures. Once the consumer started, that connection was still 
served. The handshake result now records whether it ran in consumer mode, so 
such a connection is no longer served once a consumer with `handlers` is up, 
and the consumer closes it as soon as it starts. The window tests send on that 
connection on a best effort basis, since it may already be closed.
   
   - The only conflict was `camel-4x-upgrade-guide-4_24.adoc`, which does not 
exist on this branch, so it is left out. The upgrade guide entry for 4.22.3 is 
added on `main` in a separate doc-sync PR.
   - `CamelWebSocketHandler` and the tests applied cleanly.
   - The CAMEL-25994 backport #28337 builds on this change (its 
`HandshakeResult` already carries the `consumer` flag added here), so it should 
be merged after this one.
   
   Tests: `components/camel-undertow` `mvn verify` on this branch: BUILD 
SUCCESS, 225 tests, 0 failures, 1 skipped (including 
`SecurityProviderWebSocketConsumerHandlersWindowTest`, 
`UndertowWsOAuthProfileConsumerWindowTest` and 
`UndertowWsSecurityWithoutProviderTest`)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to