ap_http_header_filter: avoid double encoding of output headers in traces.
When output headers are logged (TRACE4) in send_all_header_fields(), we don't
need to explicitely ap_escape_logitem() them since ap_log_rerror() takes care
of log files escaping already.
Otherwise, characters like '"' are unncesseralily encoded, and '\\' is doubly
encoded (including for controls).
@@ -898,8 +898,7 @@ static apr_status_t send_all_header_fields(header_struct *h,
t_elt = (const apr_table_entry_t *)(elts->elts);
do {
ap_log_rerror(APLOG_MARK, APLOG_TRACE4, 0, r, " %s: %s",
- ap_escape_logitem(r->pool, t_elt->key),
- ap_escape_logitem(r->pool, t_elt->val));
+ t_elt->key, t_elt->val);
t_elt++;
} while (t_elt < t_end);
}
