We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
origin
Note: for support questions, please use one of these channels: stackoverflow or slack
We are socket.io-client in the Node env. The target server has origin check. If we use pure ws implementation, the origin can be sent to the server.
The origin is not passed to the ws library in the engine.io-client.
Should we add origin field to the options? There are some other options missing in engine.io-client.
https://github.com/websockets/ws/blob/master/doc/ws.md#new-websocketaddress-protocols-options
The code change should be before this line
engine.io-client/lib/transports/websocket.js
Line 112 in 1519765
The text was updated successfully, but these errors were encountered:
feat: add missing ws options
d134fee
Reference: https://github.com/websockets/ws/blob/master/doc/ws.md#new-websocketaddress-protocols-options Related: - #574 - #615
Closed by d134fee (included in engine.io-client@4.1.0)
engine.io-client@4.1.0
Sorry, something went wrong.
No branches or pull requests
Note: for support questions, please use one of these channels: stackoverflow or slack
You want to:
Current behavior
We are socket.io-client in the Node env. The target server has origin check. If we use pure ws implementation, the
origin
can be sent to the server.The
origin
is not passed to the ws library in the engine.io-client.Steps to reproduce (if the current behavior is a bug)
Expected behavior
Setup
Other information (e.g. stacktraces, related issues, suggestions how to fix)
Should we add origin field to the options? There are some other options missing in engine.io-client.
https://github.com/websockets/ws/blob/master/doc/ws.md#new-websocketaddress-protocols-options
The code change should be before this line
engine.io-client/lib/transports/websocket.js
Line 112 in 1519765
The text was updated successfully, but these errors were encountered: