Skip to content

Commit

Permalink
Revert "Convert check_vhost_access/3 to new API"
Browse files Browse the repository at this point in the history
This reverts commit abe9ee7.
  • Loading branch information
lukebakken committed Oct 24, 2019
1 parent df6067a commit 448cc61
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions src/rabbit_auth_backend_ip_range.erl
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ user_login_authorization(_Username) ->
user_login_authorization(_Username, _AuthProps) ->
{ok, none}.

check_vhost_access(#auth_user{tags = Tags}, _VHostPath, AuthzData) ->
Address = extract_address(AuthzData),
check_vhost_access(#auth_user{tags = Tags}, _VHostPath, Sock) ->
Address = extract_address(Sock),

% filter out applicable masks
case lists:filtermap(
Expand Down Expand Up @@ -85,11 +85,10 @@ extract_address(undefined) ->
extract_address(unknown) ->
unknown;
% for native direct connections the address is set to unknown
extract_address(#authz_socket_info{peername={unknown, _Port}}) ->
undefined;
extract_address(#authz_socket_info{peername={Address, _Port}}) ->
Address;
extract_address(#{peeraddr := Address}) ->
extract_address(#authz_socket_info{peername={unknown, _Port}}) -> undefined;
extract_address(#authz_socket_info{peername={Address, _Port}}) -> Address;
extract_address(Sock) ->
{ok, {Address, _Port}} = rabbit_net:peername(Sock),
Address.

format_masks(Masks) ->
Expand Down

0 comments on commit 448cc61

Please sign in to comment.