You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bugs@httpd.apache.org by bu...@apache.org on 2009/09/09 23:43:34 UTC

DO NOT REPLY [Bug 47812] New: error_log hook often does not have access to the request_rec

https://issues.apache.org/bugzilla/show_bug.cgi?id=47812

           Summary: error_log hook often does not have access to the
                    request_rec
           Product: Apache httpd-2
           Version: 2.3-HEAD
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Core
        AssignedTo: bugs@httpd.apache.org
        ReportedBy: phil.pickett@springsource.com


--- Comment #0 from Philip Pickett <ph...@springsource.com> 2009-09-09 14:43:33 PDT ---
There are places in the code that don't have access to a request_rec, however
there seem to be a lot of places where developers go out of their way to _not_
pass a request_rec to ap_log_rerror but instead use ap_log_error (according to
a very quick grep of the source code), there are >250 counts of things like
this happening:

from mod_proxy_util.c, line 2102:
-----------------------------------------
ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, r->server,
"proxy: connecting %s to %s:%d", *url, uri->hostname,
uri->port);
-----------------------------------------
... where a server is passed to ap_log_error() *from a valid request_rec* for
no apparent reason. It seems to me the vast majority of error log messages that
would be useful to be able to tally with the source request come from
ap_log_error calls like this.

According to the documentation for ap_log_error() in http_log.h:

"If a request_rec is available, use that with ap_log_rerror() in preference to
calling this function. Otherwise, if a conn_rec is available, use that with
ap_log_cerror() in preference to calling this function"

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 47812] error_log hook often does not have access to the request_rec

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47812

Stefan Fritsch <sf...@sfritsch.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |FixedInTrunk

--- Comment #2 from Stefan Fritsch <sf...@sfritsch.de> 2011-06-15 22:58:28 UTC ---
We have fixed a lot of these in trunk. If you find more, submit them to
bugzilla.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 47812] error_log hook often does not have access to the request_rec

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47812



--- Comment #1 from Nick Kew <ni...@webthing.com> 2009-09-09 16:38:17 PDT ---
This isn't really a bug.  But if you'd like to submit a patch, it might get
applied as a minor cleanup.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


DO NOT REPLY [Bug 47812] error_log hook often does not have access to the request_rec

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=47812

Stefan Fritsch <sf...@sfritsch.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Stefan Fritsch <sf...@sfritsch.de> 2012-02-26 16:59:01 UTC ---
2.4.1 is released, closing
If you find any more of these in 2.4.x, please open a new report.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org