[CONTROLLER-1147] websocket does not implement the ws:// protocol defined in the URI scheme in rfc6455 Created: 10/Feb/15 Updated: 25/Jul/23 Resolved: 05/May/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | adsal |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | RichardHill | Assignee: | Jan Hajnar |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 2699 |
| Description |
|
Tested Helium SU2 Steps I subscribed to "http://<host>:8181/opendaylight-inventory:nodes/datastore=CONFIGURATION/scope=BASE" In the wiki I couldn't initially find the URI to point the websocket client at because I expected a WS:// protocol as described in section 3 http://tools.ietf.org/html/rfc6455 This specification defines two URI schemes, using the ABNF syntax ws-URI = "ws:" "//" host [ ":" port ] path [ "?" query ] host = <host, defined in [RFC3986], Section 3.2.2> ************* To investigate the effect of using http:// I used the python websocket client |
| Comments |
| Comment by RichardHill [ 10/Feb/15 ] |
|
Further reading of the RFC (Section 4) indicates the failure to connect is expected because during an opening handshake "A client will need to |
| Comment by Jan Hajnar [ 02/Mar/15 ] |
|
Hi, I tested simple websocket client written in javascript and also chrome extension Simple Websocket Client and when you substitute "http" protocol in path for "ws" I was able to successfuly connect to server and receive notifications. here is simple patch for helium that returns location header parameter with "ws" protocol. You should be able to listen to notifications on that path via standard websocket clients. For Lithium further modification is probably needed. |
| Comment by Carol Sanders [ 05/May/15 ] |
|
This bug is part of the project to Move all ADSAL associated component bugs to ADSAL. |