From 38a9f0ebdd581bb53dcda8c4e71dffa45b8d898b Mon Sep 17 00:00:00 2001 From: Glenn Strauss Date: Wed, 16 Jun 2021 17:25:57 -0400 Subject: [PATCH] [mod_access] remove excess trace trace is still issued if access is denied and r->log_request_handling is set --- src/mod_access.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) diff --git a/src/mod_access.c b/src/mod_access.c index 327215f6..5b722bfa 100644 --- a/src/mod_access.c +++ b/src/mod_access.c @@ -118,9 +118,7 @@ static int mod_access_check (const array * const allow, const array * const deny } /** - * URI handler - * - * we will get called twice: + * handler is called twice: * - after the clean up of the URL and * - after the pathinfo checks are done * @@ -128,17 +126,9 @@ static int mod_access_check (const array * const allow, const array * const deny */ URIHANDLER_FUNC(mod_access_uri_handler) { plugin_data *p = p_d; - mod_access_patch_config(r, p); - - if (NULL == p->conf.access_allow && NULL == p->conf.access_deny) { + if (NULL == p->conf.access_allow && NULL == p->conf.access_deny) return HANDLER_GO_ON; /* access allowed; nothing to match */ - } - - if (r->conf.log_request_handling) { - log_error(r->conf.errh, __FILE__, __LINE__, - "-- mod_access_uri_handler called"); - } return mod_access_check(p->conf.access_allow, p->conf.access_deny, &r->uri.path, r->conf.force_lowercase_filenames)