Downgrade error log messages which accompany a 404 request status from loglevel

Stefan Fritsch · Stefan Fritsch · commit 8104f18be847 · 2011-10-08T07:36:56.000Z
error to info. List thread at http://mail-archives.apache.org/mod_mbox/httpd-dev/201110.mbox/%3Calpine.DEB.2.00.1110041948280.664@eru.sfritsch.de%3E PR: 35768 git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1180328 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/CHANGES b/CHANGES
@@ -12,6 +12,10 @@ Changes with Apache 2.3.15
      PR 51714. [Stefan Fritsch, Jim Jagielski, Ruediger Pluem, Eric Covener,
      <lowprio20 gmail.com>]
 
+  *) core, mod_actions, mod_asis: Downgrade error log messages which accompany
+     a 404 request status from loglevel error to info. PR: 35768. [Stefan
+     Fritsch]
+
   *) core: Fix hook sorting with perl modules. PR: 45076. [Torsten Foertsch
      <torsten foertsch gmx net>]
 
diff --git a/modules/generators/mod_asis.c b/modules/generators/mod_asis.c
@@ -44,7 +44,7 @@ static int asis_handler(request_rec *r)
     }
 
     if (r->finfo.filetype == APR_NOFILE) {
-        ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+        ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,
                     "File does not exist: %s", r->filename);
         return HTTP_NOT_FOUND;
     }
diff --git a/modules/mappers/mod_actions.c b/modules/mappers/mod_actions.c
@@ -187,7 +187,7 @@ static int action_handler(request_rec *r)
 
     if (action && (t = apr_table_get(conf->action_types, action))) {
         if (*t++ == '0' && r->finfo.filetype == APR_NOFILE) {
-            ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+            ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,
                           "File does not exist: %s", r->filename);
             return HTTP_NOT_FOUND;
         }
diff --git a/server/core.c b/server/core.c
@@ -4216,7 +4216,7 @@ static int default_handler(request_rec *r)
 
     if (r->method_number == M_GET || r->method_number == M_POST) {
         if (r->finfo.filetype == APR_NOFILE) {
-            ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+            ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,
                           "File does not exist: %s", r->filename);
             return HTTP_NOT_FOUND;
         }
@@ -4225,7 +4225,7 @@ static int default_handler(request_rec *r)
          * raw I/O on a dir.
          */
         if (r->finfo.filetype == APR_DIR) {
-            ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+            ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,
                           "Attempt to serve directory: %s", r->filename);
             return HTTP_NOT_FOUND;
         }
@@ -4234,7 +4234,7 @@ static int default_handler(request_rec *r)
             r->path_info && *r->path_info)
         {
             /* default to reject */
-            ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
+            ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,
                           "File does not exist: %s",
                           apr_pstrcat(r->pool, r->filename, r->path_info, NULL));
             return HTTP_NOT_FOUND;

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ static int asis_handler(request_rec *r)`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`if (r->finfo.filetype == APR_NOFILE) {`
`47`		`- ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,`
	`47`	`+ ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,`
`48`	`48`	`"File does not exist: %s", r->filename);`
`49`	`49`	`return HTTP_NOT_FOUND;`
`50`	`50`	`}`
Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ static int action_handler(request_rec *r)`
`187`	`187`
`188`	`188`	`if (action && (t = apr_table_get(conf->action_types, action))) {`
`189`	`189`	`if (*t++ == '0' && r->finfo.filetype == APR_NOFILE) {`
`190`		`- ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,`
	`190`	`+ ap_log_rerror(APLOG_MARK, APLOG_INFO, 0, r,`
`191`	`191`	`"File does not exist: %s", r->filename);`
`192`	`192`	`return HTTP_NOT_FOUND;`
`193`	`193`	`}`