DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=38070>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=38070 ------- Additional Comments From [EMAIL PROTECTED] 2006-02-21 10:49 ------- I have opened Red Hat's bugzilla case and ask RH to back port your patch into RH's. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=176663 Then RH's engineer wrote in comment #5. >Comment #5 From Joe Orton ([EMAIL PROTECTED]) on 2006-02-20 10:49 >EST [reply] > >The fix committed upstream prevents handling of conditional requests with a CGI >script which outputs an explicit (albeit redundant) "Status: 200" header. This >would count as a regression so we would not include that patch as-is in a RHEL >update. > >I've prepared a (simpler) alternative patch, which fixes the real issue and >will >make packages available for testing. > --- httpd-2.0.52/server/util_script.c.cgistatus +++ httpd-2.0.52/server/util_script.c @@ -462,6 +462,13 @@ if ((cgi_status == HTTP_OK) && (r->method_number == M_GET)) { cond_status = ap_meets_conditions(r); + + /* In case an explicit Status: header had set + * r->status_line, then unset it here, so that the + * actual handler return value will be honoured. */ + if (cond_status != OK) { + r->status_line = NULL; + } } apr_table_overlap(r->err_headers_out, merge, APR_OVERLAP_TABLES_MERGE); With Red Hat's fix, apache with sample cgi return 200,304,304,304. With Nick's fix, apache with sample cgi return 200,200,200,200. Both cases, no more white (empty) display. But,which one is better solution? -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
