Browse Source

Merge pull request #12601 from lafirest/fix/eldap_log

 fix(ldap): fix that logs of eldap will never be logged
lafirest 2 years ago
parent
commit
02de92a53d

+ 1 - 1
apps/emqx_ldap/src/emqx_ldap.app.src

@@ -1,6 +1,6 @@
 {application, emqx_ldap, [
 {application, emqx_ldap, [
     {description, "EMQX LDAP Connector"},
     {description, "EMQX LDAP Connector"},
-    {vsn, "0.1.6"},
+    {vsn, "0.1.7"},
     {registered, []},
     {registered, []},
     {applications, [
     {applications, [
         kernel,
         kernel,

+ 4 - 3
apps/emqx_ldap/src/emqx_ldap.erl

@@ -312,11 +312,12 @@ do_ldap_query(
             {error, {unrecoverable_error, Reason}}
             {error, {unrecoverable_error, Reason}}
     end.
     end.
 
 
-log(Level, Format, Args) ->
+%% Note: the value of the `_Level` here always is 2
+log(_Level, Format, Args) ->
     ?SLOG(
     ?SLOG(
-        Level,
+        info,
         #{
         #{
-            msg => "ldap_log",
+            msg => "eldap_info",
             log => io_lib:format(Format, Args)
             log => io_lib:format(Format, Args)
         }
         }
     ).
     ).

+ 1 - 0
changes/ce/fix-12601.en.md

@@ -0,0 +1 @@
+Fixed that the logs of LDAP driver would never be logged, now all of them are logged with `info` level.