summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Pazdziora <jpazdziora@redhat.com>2016-11-15 13:41:48 +0100
committerJan Pazdziora <jpazdziora@redhat.com>2016-11-15 13:41:58 +0100
commit405ff8c85de71735ef5fe7893b6efd1dd19f5508 (patch)
tree77df8a5f066fb5b40d20f5bf17b0b7e98c4234ea
parent5fc7b898355293e3f413b2ab4da4fee2b942fe3c (diff)
downloadmod_authnz_pam-405ff8c85de71735ef5fe7893b6efd1dd19f5508.tar.gz
mod_authnz_pam-405ff8c85de71735ef5fe7893b6efd1dd19f5508.tar.xz
mod_authnz_pam-405ff8c85de71735ef5fe7893b6efd1dd19f5508.zip
Prefer ap_log_rerror for messages related to a particular request.
Suggested by: Phil Endecott
-rw-r--r--mod_authnz_pam.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/mod_authnz_pam.c b/mod_authnz_pam.c
index 87d3150..24f56e0 100644
--- a/mod_authnz_pam.c
+++ b/mod_authnz_pam.c
@@ -170,7 +170,7 @@ static authn_status pam_authenticate_with_login_password(request_rec * r, const
if (ret == PAM_NEW_AUTHTOK_REQD) {
authnz_pam_config_rec * conf = ap_get_module_config(r->per_dir_config, &authnz_pam_module);
if (conf && conf->expired_redirect_url) {
- ap_log_error(APLOG_MARK, APLOG_ERR, 0, r->server,
+ ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
"mod_authnz_pam: PAM_NEW_AUTHTOK_REQD: redirect to [%s]",
conf->expired_redirect_url);
apr_table_addn(r->headers_out, "Location", format_location(r, conf->expired_redirect_url, login));
@@ -181,14 +181,14 @@ static authn_status pam_authenticate_with_login_password(request_rec * r, const
}
if (ret != PAM_SUCCESS) {
const char * strerr = pam_strerror(pamh, ret);
- ap_log_error(APLOG_MARK, APLOG_WARNING, 0, r->server, "mod_authnz_pam: %s %s: %s", stage, param, strerr);
+ ap_log_rerror(APLOG_MARK, APLOG_WARNING, 0, r, "mod_authnz_pam: %s %s: %s", stage, param, strerr);
apr_table_setn(r->subprocess_env, _EXTERNAL_AUTH_ERROR_ENV_NAME, apr_pstrdup(r->pool, strerr));
pam_end(pamh, ret);
return AUTH_DENIED;
}
apr_table_setn(r->subprocess_env, _REMOTE_USER_ENV_NAME, login);
r->user = apr_pstrdup(r->pool, login);
- ap_log_error(APLOG_MARK, APLOG_NOTICE, 0, r->server, "mod_authnz_pam: PAM authentication passed for user %s", login);
+ ap_log_rerror(APLOG_MARK, APLOG_NOTICE, 0, r, "mod_authnz_pam: PAM authentication passed for user %s", login);
pam_end(pamh, ret);
return AUTH_GRANTED;
}