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.

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

Websocket connection problems #113

Open
mkurkov opened this issue Aug 17, 2016 · 1 comment
Open

Websocket connection problems #113

mkurkov opened this issue Aug 17, 2016 · 1 comment

Comments

@mkurkov
Copy link

mkurkov commented Aug 17, 2016

Hi, I've run into several issues, trying to use escalus websocket transport.

  1. Some of underlying libs (wsock) uses re:replace as template parser to create HTTP message, but if wspath has parameters with '&'s re:replace changes this to {{resource}}, so websocket connection fails.
  2. Another issue with wsecli/wsock is that it's not possible to provide sec-websocket-protocol: xmpp header. Not sure if it makes sense to add wsheaders param to escalus or just hardcode somewhere in escalus_ws. My server required this, so again connection failed.
  3. I saw that new version of escalus has a nice feature of user-defined transport modules, so I decided to implement one instead of patching wsock/wsecli. But some websocket specific logic works only if transport=ws, so it doesn't work with my own module name as transport. This one can be easily fixed by separating transport and module parameters.

Just thought this info could be helpful for improving escalus.

@michalwski
Copy link
Contributor

Hi @mkurkov,

Thanks for the info, this is very helpful. We already thought about replacing wsock with some other, better lib. Your input makes the change more important now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants