* Akins, Brian wrote:

> On 1/25/08 3:51 PM, "André Malo" <[EMAIL PROTECTED]> wrote:
> > I don't recommend doing that as it contains a race condition (the file
> > might be changed in the meantime).
>
> That race is in the default_handler as well, isn't it?

Yes. IIRC there's already a bug report for it.

> (That's one of the reasons I like my approach, we just rely on the
> features/bugs of the core. Fix/enhance that and everyone benefits. )

Huh? I don't see why we should dup the bug.

> > By backend I meant here the script providing
> > the x-sendfile header. That's the only place knowing the behaviour of
> > the file exactly.
>
> In theory, if  backend gives us an x-sendfile, it shouldn't touch it.
> Especially if it's a x-sendfile-temp (lighttpd calls this something
> strange).

Yes, if it's temp you're right. In the other case there's no guarantee given 
to the webserver. X-Sendfile should just take the file and apply the rest 
of the headers sent by the backend. Everything doing else lessens its 
usefulness.

nd
-- 
sub the($){+shift} sub answer (){ord q
        [* It is always 42! *]       }
           print the answer
# André Malo # http://www.perlig.de/ #

Reply via email to