Skip to content

Commit 44d9d00

Browse files
committed
Merge r733127, r733219 from trunk:
pass a server_rec to ap_log_error on two interim response messages use ap_log_rerror instead of r->server per niq's comments Submitted by: covener Reviewed/backported by: jim git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.2.x@733756 13f79535-47bb-0310-9956-ffa450edef68
1 parent 368e472 commit 44d9d00

2 files changed

Lines changed: 2 additions & 11 deletions

File tree

STATUS

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -114,15 +114,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
114114
trunk works
115115
+1 covener, niq, rpluem
116116

117-
* mod_proxy_http: Add server_rec to ap_log_error() calls for interim
118-
response messages
119-
trunk
120-
http://svn.apache.org/viewvc?rev=733127&view=rev
121-
http://svn.apache.org/viewvc?rev=733219&view=rev
122-
2.2.x:
123-
trunk works
124-
+1: covener, niq, rpluem
125-
126117
* util_script (CGI): return 504 (Gateway timeout) rather than 500
127118
when a script times out before returning status line/headers.
128119
PR 42190

modules/proxy/mod_proxy_http.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1622,7 +1622,7 @@ apr_status_t ap_proxy_http_process_response(apr_pool_t * p, request_rec *r,
16221622
*/
16231623
const char *policy = apr_table_get(r->subprocess_env,
16241624
"proxy-interim-response");
1625-
ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, NULL,
1625+
ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r,
16261626
"proxy: HTTP: received interim %d response",
16271627
r->status);
16281628
if (!policy || !strcasecmp(policy, "RFC")) {
@@ -1632,7 +1632,7 @@ apr_status_t ap_proxy_http_process_response(apr_pool_t * p, request_rec *r,
16321632
* policies and maybe also add option to bail out with 502
16331633
*/
16341634
else if (strcasecmp(policy, "Suppress")) {
1635-
ap_log_error(APLOG_MARK, APLOG_WARNING, 0, NULL,
1635+
ap_log_rerror(APLOG_MARK, APLOG_WARNING, 0, r,
16361636
"undefined proxy interim response policy");
16371637
}
16381638
}

0 commit comments

Comments
 (0)