chore: update emqx_authn.appup.src

apps/emqx_authn/src/emqx_authn.appup.src

@@ -1,5 +1,11 @@
 %% -*- mode: erlang -*-
 %% Unless you know what you are doing, DO NOT edit manually!!
 {VSN,
-  [{"0.1.0",[{load_module,emqx_authn_utils,brutal_purge,soft_purge,[]}]}],
+  [{"0.1.0",[
-  [{"0.1.0",[{load_module,emqx_authn_utils,brutal_purge,soft_purge,[]}]}]}.
+     {load_module,emqx_authn_utils,brutal_purge,soft_purge,[]},
+     {load_module,emqx_authn_redis,brutal_purge,soft_purge,[]}]
+   }],
+  [{"0.1.0",[
+     {load_module,emqx_authn_utils,brutal_purge,soft_purge,[]},
+     {load_module,emqx_authn_redis,brutal_purge,soft_purge,[]}]
+   }]}.

apps/emqx_authn/src/emqx_authn_utils.erl

@@ -80,7 +80,7 @@ check_password_from_selected_map(Algorithm, Selected, Password) ->
     ),
     case Hash of
         undefined ->
-            {error, bad_username_or_password};
+            {error, not_authorized};
         _ ->
             Salt = maps:get(<<"salt">>, Selected, <<>>),
             case

apps/emqx_authn/src/simple_authn/emqx_authn_redis.erl

@@ -138,8 +138,7 @@ authenticate(
         {ok, []} ->
             ignore;
         {ok, Values} ->
-            case merge(Fields, Values) of
+            Selected = merge(Fields, Values),
-                #{<<"password_hash">> := _} = Selected ->
             case
                 emqx_authn_utils:check_password_from_selected_map(
                     Algorithm, Selected, Password
@@ -150,16 +149,6 @@ authenticate(
                 {error, Reason} ->
                     {error, Reason}
             end;
-                _ ->
-                    ?SLOG(error, #{
-                        msg => "cannot_find_password_hash_field",
-                        cmd => Command,
-                        keys => NKey,
-                        fields => Fields,
-                        resource => ResourceId
-                    }),
-                    ignore
-            end;
         {error, Reason} ->
             ?SLOG(error, #{
                 msg => "redis_query_failed",