On Thu, Jan 8, 2015 at 5:29 PM, Ewald Dieterich <ew...@mailbox.org> wrote: > > Looks good. I tested both your patch and my modified one and now error > response and access log entry are both OK.
Thanks Ewald for testing. Eric, all proxy modules concerned in trunk could be fixed with "(status == AP_FILTER_ERROR) ? DONE : ap_map_http_request_error(status, HTTP_BAD_REQUEST)", but not 2.4.x where "(status == AP_FILTER_ERROR) ? DONE : HTTP_BAD_REQUEST)" is a simple fix. Should we also (propose to )backport to 2.4.x the whole ap_map_http_request_error() stuff (r1482522 + r1529991 + r1482918, quite huge...) so that we don't lose specific errors returned by ap_http_filter (ie. turned to 400), or is the simple fix OK?
