search

jrief / django-websocket-redis

Websockets for Django applications using Redis as message queue
http://django-websocket-redis.awesto.com/
MIT License
896 stars 222 forks source link

Add support for Sec-WebSocket-Protocol header #149

Closed ErwinJunge closed 8 years ago

ErwinJunge commented 8 years ago

This branch adds support for the Sec-WebSocket-Protocol header. Without this change, the websocket will fail to connect if the Sec-WebSocket-Protocol header is uses by the client.

jrief commented 8 years ago

wouldn't it make sense to add this to one of the testing clients?

ErwinJunge commented 8 years ago

Good point, I've added a test.

jrief commented 8 years ago

@ErwinJunge Since I never had the need for the HTTP_SEC_WEBSOCKET_PROTOCOL, what is your comment about #151. Shall I merge it?