Somebody remind me why that ap_status_drops_conn call was added in the first place? It was a recent change, and seemed important at the time.
--Cliff On 20 Jun 2002 [EMAIL PROTECTED] wrote: > gregames 2002/06/20 12:34:29 > > Modified: modules/http http_request.c > Log: > prevent infinite recursion if an ErrorDocument gets an error > > Reported by: Colm MacCrthaigh <[EMAIL PROTECTED]> > > Revision Changes Path > 1.148 +1 -1 httpd-2.0/modules/http/http_request.c > > Index: http_request.c > =================================================================== > RCS file: /home/cvs/httpd-2.0/modules/http/http_request.c,v > retrieving revision 1.147 > retrieving revision 1.148 > diff -u -r1.147 -r1.148 > --- http_request.c 10 Jun 2002 22:02:57 -0000 1.147 > +++ http_request.c 20 Jun 2002 19:34:28 -0000 1.148 > @@ -117,7 +117,7 @@ > * error condition, we just report on the original error, and give up on > * any attempt to handle the other thing "intelligently"... > */ > - if (r->status != HTTP_OK && !ap_status_drops_connection(type)) { > + if (r->status != HTTP_OK) { > recursive_error = type; > > while (r->prev && (r->prev->status != HTTP_OK)) > > > >