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
Looks like misultin_ws:get(origin) only uses the "Origin" header, which is correct for newer versions of the websocket protocol.
Older versions still rely on "Sec-Websocket-Origin"
I'm doing origin lookups like this in my code, because not all clients are sending an Origin: header yet:
case WS:get(origin) of false -> Headers = WS:get(headers), proplists:get_value("Sec-Websocket-Origin", Headers, false); Origin -> Origin end
Would probably make sense for misultin_ws:get(origin) to take care of that.
The text was updated successfully, but these errors were encountered:
fixed 33ab595
Sorry, something went wrong.
No branches or pull requests
Looks like misultin_ws:get(origin) only uses the "Origin" header, which is correct for newer versions of the websocket protocol.
Older versions still rely on "Sec-Websocket-Origin"
I'm doing origin lookups like this in my code, because not all clients are sending an Origin: header yet:
case WS:get(origin) of
false ->
Headers = WS:get(headers),
proplists:get_value("Sec-Websocket-Origin", Headers, false);
Origin -> Origin
end
Would probably make sense for misultin_ws:get(origin) to take care of that.
The text was updated successfully, but these errors were encountered: