Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WebSocket should support jetty-io Connection.Listener #3380

Closed
joakime opened this issue Feb 21, 2019 · 2 comments
Closed

WebSocket should support jetty-io Connection.Listener #3380

joakime opened this issue Feb 21, 2019 · 2 comments
Assignees
@joakime

Comments

@joakime
Copy link
Contributor

joakime commented Feb 21, 2019

Connections created for WebSocket should support jetty-io Connection.Listener as well.

This means when the WebSocket Connection is created, regardless of client or server, the Connection.Listener should be added/attached before that connection is upgraded.
@joakime joakime self-assigned this Feb 21, 2019
joakime added a commit that referenced this issue Feb 21, 2019
@joakime
Issue #3380 - Adding configured Connection.Listeners to WebSocket Con…
0f4294e 
…nections

Signed-off-by: Joakim Erdfelt <joakim@erdfelt.com>
@joakime
Copy link
Contributor Author

joakime commented Feb 21, 2019

Opened PR #3381

joakime added a commit that referenced this issue Feb 21, 2019
@joakime
Merge pull request #3381 from eclipse/jetty-10-issue-3380-websocket-c…
3826397 
…onnection-listener

Issue #3380 - Adding configured Connection.Listeners to WebSocket Connections
@joakime
Copy link
Contributor Author

joakime commented Feb 21, 2019

Merged in commit 3826397

@joakime joakime closed this as completed Feb 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joakime joakime

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@joakime