Skip to content

Conversation

emattheis
Copy link
Contributor

This PR makes the negotiated subprotocol available to WebSocketClientConnection in addition to WebSocketConnection by moving the accessor to the common interface/implementation. This is crucial for clients that can support multiple protocols and need to know which protocol was selected by the server.

@emattheis emattheis force-pushed the wsnext-clientconnection-subprotocol branch 7 times, most recently from 7ec024d to 736e0e9 Compare June 24, 2025 15:14
@gsmet gsmet requested a review from mkouba June 24, 2025 15:55
@geoand
Copy link
Contributor

geoand commented Jun 25, 2025

@mkouba PTAL

Copy link
Contributor

@mkouba mkouba left a comment

Choose a reason for hiding this comment

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

Very nice!

@mkouba mkouba added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jun 25, 2025
@geoand geoand force-pushed the wsnext-clientconnection-subprotocol branch from 736e0e9 to 0c49734 Compare June 25, 2025 13:50

This comment has been minimized.

Copy link

quarkus-bot bot commented Jun 25, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 0c49734.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@geoand geoand merged commit b857e19 into quarkusio:main Jun 25, 2025
26 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.25 - main milestone Jun 25, 2025
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jun 25, 2025
@emattheis emattheis deleted the wsnext-clientconnection-subprotocol branch June 25, 2025 15:47
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