Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325128 - WebSocket handshake fails when sending message during onConnect
Summary: WebSocket handshake fails when sending message during onConnect
Status: RESOLVED FIXED
Alias: None
Product: Jetty
Classification: RT
Component: server (show other bugs)
Version: unspecified   Edit
Hardware: All Linux
: P3 normal (vote)
Target Milestone: 7.1.x   Edit
Assignee: Greg Wilkins CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-13 12:11 EDT by Neil Dunn CLA
Modified: 2010-09-14 05:14 EDT (History)
1 user (show)

See Also:

Attachments
Example Websocket Javascript client (1.15 KB, text/html)
2010-09-13 12:13 EDT, Neil Dunn CLA 		no flags Details
Example Server exhibiting bug. (1.76 KB, text/x-java)
2010-09-13 12:13 EDT, Neil Dunn CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Neil Dunn CLA 2010-09-13 12:11:17 EDT 
Build Identifier: jetty-distribution-7.1.6.v20100715

Using Jetty 7.1.6 and the stable branch of Google Chrome (6.0.472.55).

Sending a message in WebSocket#onConnect via Outbound#sendMessage causes the Websocket's handshake to fail.

An example snippet of code which exhibits the bug is attached.

Perhaps this is not a bug, but the API does not document this. It also worked in previous versions of Jetty + Chrome. (before the hixie hixie shake).

If it is not a bug, there should be a hook to know when one can actually send messages to the WebSocket.

Reproducible: Always

Steps to Reproduce:
1. Setup a Jetty server with the sample code
2. Run the main method
3. Connect to the server in Google Chrome
4. Click "Test"
5. The WebSocket receives an onclose event before an onopen event, representing a handshake failure.
Comment 1 Neil Dunn CLA 2010-09-13 12:13:08 EDT 
Created attachment 178755 [details]
Example Websocket Javascript client
Comment 2 Neil Dunn CLA 2010-09-13 12:13:59 EDT 
Created attachment 178756 [details]
Example Server exhibiting bug.
Comment 3 Greg Wilkins CLA 2010-09-14 05:14:20 EDT 
fixed in r2272 and a few more test harnesses added for -75, -76, -00 and -01

cheers