Bug 44165 - mod_proxy_http apr_status_t vs. http status cleanup
Summary: mod_proxy_http apr_status_t vs. http status cleanup
Status: RESOLVED FIXED
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_proxy (show other bugs)
Version: 2.5-HEAD
Hardware: Other other
: P4 minor (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2008-01-03 14:04 UTC by Eric Covener
Modified: 2008-02-29 18:42 UTC (History)
0 users


Attachments
return http status codes from static reqbody functions (10.99 KB, patch)
2008-01-03 14:05 UTC, Eric Covener 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Covener 2008-01-03 14:04:53 UTC 
Along the path of passing the post body to the origin server, 
mod_proxy_http returns apr_status_t values instead of HTTP status codes / OK.

Pushing the change into the various static functions (instead of cleaning up at
the last minute) lets more care be given to the HTTP status code ultimately
returned.

Current trunk/2.2.x has code to fixup the bad return code from handler, but it
does emit a message (and loses sometimes loses what could have been a
descriptive HTTP status code)
Comment 1 Eric Covener 2008-01-03 14:05:52 UTC 
Created attachment 21340 [details]
return http status codes from static reqbody functions