On Monday 10 December 2012, Marion & Christophe JAILLET wrote: > Just above this, there is the following comment : > /* > * Date and Server are less interesting, use TRACE5 for > them while * using TRACE4 for the other headers. > */ > However, I don't see where the other headers are logged.
It's done in send_all_header_fields() which sends everything except "Date" and "Server". The comment is here because the trace4 for the ", headers:" string does not match the trace5 of the Server+Date header. > You can find a little above: > ap_log_rerror(APLOG_MARK, APLOG_TRACE3, 0, r, > "Response sent with status %d%s", > r->status, > APLOGrtrace4(r) ? ", headers:" : ""); > but, IMO, it does not log the headers, it only prints "headers: ". > Apparently, it has been that way, since the beginning in > http://svn.apache.org/viewvc?view=revision&sortby=date&revision=963057 > > Should something be added here to match the comment, or remove the > comment ? Neither. But maybe add a comment that points to send_all_header_fields()?
