Naming: catch_no_user_vhost => catch_no_such_user_or_vhost
This commit is contained in:
parent
915af29bbb
commit
e238ffdabe
|
|
@ -49,7 +49,7 @@
|
||||||
-export([direct_request/6]).
|
-export([direct_request/6]).
|
||||||
-export([qs_val/2]).
|
-export([qs_val/2]).
|
||||||
-export([get_path_prefix/0]).
|
-export([get_path_prefix/0]).
|
||||||
-export([catch_no_user_vhost/2]).
|
-export([catch_no_such_user_or_vhost/2]).
|
||||||
|
|
||||||
-import(rabbit_misc, [pget/2]).
|
-import(rabbit_misc, [pget/2]).
|
||||||
|
|
||||||
|
|
@ -1090,8 +1090,8 @@ qs_val(Name, ReqData) ->
|
||||||
Qs = cowboy_req:parse_qs(ReqData),
|
Qs = cowboy_req:parse_qs(ReqData),
|
||||||
proplists:get_value(Name, Qs, undefined).
|
proplists:get_value(Name, Qs, undefined).
|
||||||
|
|
||||||
-spec catch_no_user_vhost(fun(() -> Result), Replacement) -> Result | Replacement.
|
-spec catch_no_such_user_or_vhost(fun(() -> Result), Replacement) -> Result | Replacement.
|
||||||
catch_no_user_vhost(Fun, Replacement) ->
|
catch_no_such_user_or_vhost(Fun, Replacement) ->
|
||||||
try
|
try
|
||||||
Fun()
|
Fun()
|
||||||
catch throw:{error, {E, _}} when E =:= no_such_user; E =:= no_such_vhost ->
|
catch throw:{error, {E, _}} when E =:= no_such_user; E =:= no_such_vhost ->
|
||||||
|
|
|
||||||
|
|
@ -88,7 +88,7 @@ perms(ReqData) ->
|
||||||
not_found ->
|
not_found ->
|
||||||
not_found;
|
not_found;
|
||||||
VHost ->
|
VHost ->
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms =
|
Perms =
|
||||||
rabbit_auth_backend_internal:list_user_vhost_permissions(
|
rabbit_auth_backend_internal:list_user_vhost_permissions(
|
||||||
|
|
|
||||||
|
|
@ -42,7 +42,7 @@ resource_exists(ReqData, Context) ->
|
||||||
|
|
||||||
to_json(ReqData, Context) ->
|
to_json(ReqData, Context) ->
|
||||||
User = rabbit_mgmt_util:id(user, ReqData),
|
User = rabbit_mgmt_util:id(user, ReqData),
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms = rabbit_auth_backend_internal:list_user_permissions(User),
|
Perms = rabbit_auth_backend_internal:list_user_permissions(User),
|
||||||
rabbit_mgmt_util:reply_list([[{user, User} | Rest] || Rest <- Perms],
|
rabbit_mgmt_util:reply_list([[{user, User} | Rest] || Rest <- Perms],
|
||||||
|
|
|
||||||
|
|
@ -39,7 +39,7 @@ resource_exists(ReqData, Context) ->
|
||||||
|
|
||||||
to_json(ReqData, Context) ->
|
to_json(ReqData, Context) ->
|
||||||
VHost = rabbit_mgmt_util:id(vhost, ReqData),
|
VHost = rabbit_mgmt_util:id(vhost, ReqData),
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms = rabbit_auth_backend_internal:list_vhost_permissions(VHost),
|
Perms = rabbit_auth_backend_internal:list_vhost_permissions(VHost),
|
||||||
rabbit_mgmt_util:reply_list([[{vhost, VHost} | Rest] || Rest <- Perms],
|
rabbit_mgmt_util:reply_list([[{vhost, VHost} | Rest] || Rest <- Perms],
|
||||||
|
|
|
||||||
|
|
@ -94,7 +94,7 @@ topic_perms(ReqData) ->
|
||||||
not_found ->
|
not_found ->
|
||||||
not_found;
|
not_found;
|
||||||
VHost ->
|
VHost ->
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms =
|
Perms =
|
||||||
rabbit_auth_backend_internal:list_user_vhost_topic_permissions(
|
rabbit_auth_backend_internal:list_user_vhost_topic_permissions(
|
||||||
|
|
|
||||||
|
|
@ -42,7 +42,7 @@ resource_exists(ReqData, Context) ->
|
||||||
|
|
||||||
to_json(ReqData, Context) ->
|
to_json(ReqData, Context) ->
|
||||||
User = rabbit_mgmt_util:id(user, ReqData),
|
User = rabbit_mgmt_util:id(user, ReqData),
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms = rabbit_auth_backend_internal:list_user_topic_permissions(User),
|
Perms = rabbit_auth_backend_internal:list_user_topic_permissions(User),
|
||||||
rabbit_mgmt_util:reply_list([[{user, User} | Rest] || Rest <- Perms],
|
rabbit_mgmt_util:reply_list([[{user, User} | Rest] || Rest <- Perms],
|
||||||
|
|
|
||||||
|
|
@ -39,7 +39,7 @@ resource_exists(ReqData, Context) ->
|
||||||
|
|
||||||
to_json(ReqData, Context) ->
|
to_json(ReqData, Context) ->
|
||||||
VHost = rabbit_mgmt_util:id(vhost, ReqData),
|
VHost = rabbit_mgmt_util:id(vhost, ReqData),
|
||||||
rabbit_mgmt_util:catch_no_user_vhost(
|
rabbit_mgmt_util:catch_no_such_user_or_vhost(
|
||||||
fun() ->
|
fun() ->
|
||||||
Perms = rabbit_auth_backend_internal:list_vhost_topic_permissions(VHost),
|
Perms = rabbit_auth_backend_internal:list_vhost_topic_permissions(VHost),
|
||||||
rabbit_mgmt_util:reply_list([[{vhost, VHost} | Rest] || Rest <- Perms],
|
rabbit_mgmt_util:reply_list([[{vhost, VHost} | Rest] || Rest <- Perms],
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue