Mercurial > prosody-modules
annotate mod_log_sasl_mech/mod_log_sasl_mech.lua @ 5787:e79f9dec35c0
mod_c2s_conn_throttle: Reduce log level from error->info
Our general policy is that "error" should never be triggerable by remote
entities, and that it is always about something that requires admin
intervention. This satisfies neither condition.
The "warn" level can be used for unexpected events/behaviour triggered by
remote entities, and this could qualify. However I don't think failed auth
attempts are unexpected enough.
I selected "info" because it is what is also used for other notable session
lifecycle events.
author | Matthew Wild <mwild1@gmail.com> |
---|---|
date | Thu, 07 Dec 2023 15:46:50 +0000 |
parents | 5ff8022466ab |
children |
rev | line source |
---|---|
1292
2d061333d0c2
mod_log_sasl_mech: Logs authentication mechanism used
Kim Alvefur <zash@zash.se>
parents:
diff
changeset
|
1 |
2d061333d0c2
mod_log_sasl_mech: Logs authentication mechanism used
Kim Alvefur <zash@zash.se>
parents:
diff
changeset
|
2 module:hook("authentication-success", function (event) |
1393
4baaa5a66a5a
mod_log_sasl_mech: Log SASL mechanism attached to session
Kim Alvefur <zash@zash.se>
parents:
1292
diff
changeset
|
3 local session = event.session; |
5772
5ff8022466ab
mod_log_sasl_mech: Handle auth event from other than mod_saslauth
Kim Alvefur <zash@zash.se>
parents:
1393
diff
changeset
|
4 local sasl_handler = session and session.sasl_handler; |
5ff8022466ab
mod_log_sasl_mech: Handle auth event from other than mod_saslauth
Kim Alvefur <zash@zash.se>
parents:
1393
diff
changeset
|
5 local log = session and session.log or module._log |
5ff8022466ab
mod_log_sasl_mech: Handle auth event from other than mod_saslauth
Kim Alvefur <zash@zash.se>
parents:
1393
diff
changeset
|
6 log("info", "Authenticated with %s", sasl_handler and sasl_handler.selected or "legacy auth"); |
1292
2d061333d0c2
mod_log_sasl_mech: Logs authentication mechanism used
Kim Alvefur <zash@zash.se>
parents:
diff
changeset
|
7 end); |