From 6891234390c2f1135bab7eb9778293738e73c9a5 Mon Sep 17 00:00:00 2001 From: "Zaiming (Stone) Shi" Date: Fri, 29 Sep 2023 09:48:15 +0200 Subject: [PATCH] chore: return simplified error reason for less logging --- .../emqx_dashboard_sso/src/emqx_dashboard_sso_ldap.erl | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/apps/emqx_dashboard_sso/src/emqx_dashboard_sso_ldap.erl b/apps/emqx_dashboard_sso/src/emqx_dashboard_sso_ldap.erl index de789df0b..499e24c5b 100644 --- a/apps/emqx_dashboard_sso/src/emqx_dashboard_sso_ldap.erl +++ b/apps/emqx_dashboard_sso/src/emqx_dashboard_sso_ldap.erl @@ -135,11 +135,13 @@ login( ensure_user_exists(Username); {ok, #{result := 'invalidCredentials'} = Reason} -> {error, Reason}; - {error, _} = Error -> - Error + {error, _Reason} -> + %% All error reasons are logged in resource buffer worker + {error, ldap_bind_query_failed} end; - {error, _} = Error -> - Error + {error, _Reason} -> + %% All error reasons are logged in resource buffer worker + {error, ldap_query_failed} end. ensure_user_exists(Username) ->