Chapril/xmpp.chapril.org-ejabberd
25
1
Fork 0
mirror of https://github.com/processone/ejabberd.git synced 2024-11-22 16:20:52 +01:00
Code Releases Activity

Fix RFC6454 violation on websocket connection when validating Origin header

Browse Source
This commit is contained in:
Paweł Dorofiejczyk 2019-04-24 16:59:54 +02:00
parent 8b501f5fe6
commit cc5829bc33
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/ejabberd_websocket.erl
View File

@ -63,15 +63,15 @@
-define(HEADER, [?CT_XML, ?AC_ALLOW_ORIGIN, ?AC_ALLOW_HEADERS]).
check(_Path, Headers) ->
RequiredHeaders = [{'Upgrade', <<"websocket">>},
{'Connection', ignore}, {'Host', ignore},
{<<"Sec-Websocket-Key">>, ignore},
{<<"Sec-Websocket-Version">>, <<"13">>},
{<<"Origin">>, get_origin()}],
HeadersValidators = [{'Upgrade', <<"websocket">>, true},
{'Connection', ignore, true}, {'Host', ignore, true},
{<<"Sec-Websocket-Key">>, ignore, true},
{<<"Sec-Websocket-Version">>, <<"13">>, true},
{<<"Origin">>, get_origin(), false}],
F = fun ({Tag, Val}) ->
F = fun ({Tag, Val, Required}) ->
case lists:keyfind(Tag, 1, Headers) of
false -> true; % header not found, keep in list
false -> Required; % header not found, keep in list if required
{_, HVal} ->
case Val of
ignore -> false; % ignore value -> ok, remove from list
@ -82,9 +82,9 @@ check(_Path, Headers) ->
end
end
end,
case lists:filter(F, RequiredHeaders) of
case lists:filter(F, HeadersValidators) of
[] -> true;
_MissingHeaders -> false
_InvalidHeaders -> false
end.
socket_handoff(LocalPath, #request{method = 'GET', ip = IP, q = Q, path = Path,