add Sec-Websocket-Protocol from Opts #248

wants to merge 1 commit into


None yet
2 participants

mihawk commented Aug 13, 2012

i need thix patch to allow sub protocol for websocket

mihawk commented Aug 25, 2012

salut, essen
que pense tu de se patch,
le but étant d'utiliser les sub protocole en websocket,
je ne sais pas si c'est la meilleur façon d’appréhender le problème,
j ai vu que tu avais laissé des commentaires à ce sujet dans le code source,
j ai pour but d’implémenter pour chicagoboss,


essen commented Aug 25, 2012

Hey, I'm undecided yet on that issue. There's bigger changes coming first so I'll get back to you after these get done and things get clearer. Thanks!


essen commented Sep 17, 2012

I'll get on that one soon now that the big changes are merged.


essen commented Jan 14, 2013

I have added parsing support of Sec-Websocket-Protocol in cowboy_req:parse_header. It can be used in websocket_init to find a proper protocol, which can then be set using set_resp_header. Procedure will be documented in the guide chapter to be pushed soon.

Thanks for your proposal!

essen closed this Jan 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment