Merge pull request #8329 from savonarola/fix-log-errors

fix: remove unnecessary error messages generated by JWT auth

CHANGES-4.4.md

@@ -7,6 +7,7 @@
 
 ### Bug fixes
 - Clean trace zip files when file has been downloaded.
+- Remove unnecessary error messages generated by JWT auth.
 
 ## v4.4.4
 

apps/emqx_auth_jwt/src/emqx_auth_jwt.app.src

@@ -1,6 +1,6 @@
 {application, emqx_auth_jwt,
  [{description, "EMQ X Authentication with JWT"},
-  {vsn, "4.4.2"}, % strict semver, bump manually!
+  {vsn, "4.4.3"}, % strict semver, bump manually!
   {modules, []},
   {registered, [emqx_auth_jwt_sup]},
   {applications, [kernel,stdlib,jose]},

apps/emqx_auth_jwt/src/emqx_auth_jwt.appup.src

@@ -1,16 +1,9 @@
 %% -*- mode: erlang -*-
 %% Unless you know what you are doing, DO NOT edit manually!!
 {VSN,
- [
-    {<<"4\\.4\\.[0-1]">>, [
-      {restart_application,emqx_auth_jwt}
-    ]},
-    {<<".*">>, []}
- ],
- [
-    {<<"4\\.4\\.[0-1]">>, [
-      {restart_application,emqx_auth_jwt}
-    ]},
-    {<<".*">>, []}
- ]
-}.
+  [{"4.4.2",[{load_module,emqx_auth_jwt,brutal_purge,soft_purge,[]}]},
+   {<<"4\\.4\\.[0-1]">>,[{restart_application,emqx_auth_jwt}]},
+   {<<".*">>,[]}],
+  [{"4.4.2",[{load_module,emqx_auth_jwt,brutal_purge,soft_purge,[]}]},
+   {<<"4\\.4\\.[0-1]">>,[{restart_application,emqx_auth_jwt}]},
+   {<<".*">>,[]}]}.

apps/emqx_auth_jwt/src/emqx_auth_jwt.erl

@@ -72,7 +72,13 @@ check_acl(ClientInfo = #{jwt_claims := Claims},
         _ ->
             ?DEBUG("no_acl_jwt_claim", []),
             ignore
-    end.
+    end;
+check_acl(_ClientInfo,
+          _PubSub,
+          _Topic,
+          _NoMatchAction,
+          _AclEnv) ->
+    ignore.
 
 is_expired(Exp) when is_binary(Exp)  ->
     ExpInt  = binary_to_integer(Exp),