* Ruediger Pluem wrote: > On 12/26/2008 10:41 PM, [email protected] wrote: > > URL: http://svn.apache.org/viewvc?rev=729538&view=rev > > Hm. Why this rather complex approach with the request_status hook? > Why not doing the subrequest here or even better just inserting the file > buckets into the brigade right here and be done?
The whole point of the sendfile stuff is to free the backend socket as early as possible and leave the rest to the apache. So I'm going out of the proxy loop and handle everything in the request status hook, which runs after the cleanup. Also I don't want to duplicate the file delivery logic from core (including sendfile, mmap etc), so putting file buckets myself into the stream sounds not like the way to go. > > > + url += sizeof(SCHEME); /* keep the slashes */ > > Why cutting off the scheme? Good question. I've just tried to find a reason, but I couldn't. Maybe there was one a year ago ;-) I'm gonna remove this line. Thanks, nd -- print "Just Another Perl Hacker"; # André Malo, <http://www.perlig.de/> #
