Просмотр исходного кода

fix: add api auth failed code def & fix bad spelling

DDDHuang 3 лет назад
Родитель
Сommit
9037b807ae
2 измененных файлов с 11 добавлено и 2 удалено
  1. 8 0
      apps/emqx/include/http_api.hrl
  2. 3 2
      apps/emqx_dashboard/src/emqx_dashboard.erl

+ 8 - 0
apps/emqx/include/http_api.hrl

@@ -14,6 +14,12 @@
 %% limitations under the License.
 %%--------------------------------------------------------------------
 
+%% HTTP API Auth
+-define(WRONG_USERNAME_OR_PWD, 'WRONG_USERNAME_OR_PWD').
+-define(WRONG_USERNAME_OR_PWD_OR_API_KEY_OR_API_SECRET,
+    'WRONG_USERNAME_OR_PWD_OR_API_KEY_OR_API_SECRET'
+).
+
 %% Bad Request
 -define(BAD_REQUEST, 'BAD_REQUEST').
 -define(NOT_MATCH, 'NOT_MATCH').
@@ -50,6 +56,8 @@
 
 %% All codes
 -define(ERROR_CODES, [
+    {'WRONG_USERNAME_OR_PWD', <<"Wrong username or pwd">>},
+    {'WRONG_USERNAME_OR_PWD_OR_API_KEY_OR_API_SECRET', <<"Wrong username & pwd or key & secret">>},
     {'BAD_REQUEST', <<"Request parameters are not legal">>},
     {'NOT_MATCH', <<"Conditions are not matched">>},
     {'ALREADY_EXISTS', <<"Resource already existed">>},

+ 3 - 2
apps/emqx_dashboard/src/emqx_dashboard.erl

@@ -26,6 +26,7 @@
 -export([authorize/1]).
 
 -include_lib("emqx/include/logger.hrl").
+-include_lib("emqx/include/http_api.hrl").
 
 -define(BASE_PATH, "/api/v5").
 
@@ -166,11 +167,11 @@ authorize(Req) ->
                             ok;
                         {error, <<"not_allowed">>} ->
                             return_unauthorized(
-                                <<"WORNG_USERNAME_OR_PWD">>,
+                                ?WRONG_USERNAME_OR_PWD,
                                 <<"Check username/password">>);
                         {error, _} ->
                             return_unauthorized(
-                                <<"WORNG_USERNAME_OR_PWD_OR_API_KEY_OR_API_SECRET">>,
+                                ?WRONG_USERNAME_OR_PWD_OR_API_KEY_OR_API_SECRET,
                                 <<"Check username/password or api_key/api_secret">>)
                     end;
                 {error, _} ->