On Wed, 2004-02-11 at 15:15 -0500, Andres Salomon wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > Philippe M. Chiasson wrote: > [...] > | > | Can you try this and confirm that it works the way it should for your > | setup ? > | > > It looks like there's still an issue with the way filetest is being > used. As Damon Buckwalters pointed out to me, filetest::access requires > the filename; a filehandle won't work. I've attached his patch, which > looks correct. >
Problem with that approach is that it will add the cost of 2 stat calls for each and every single ModPerl::Registry scripts ;-( I think it might be worth considering a new ModPerl::RegistryCooker subclass just for ACL support. Since most people don't use ACLs, it seems unjustified to me to incur the overhead to everybody. People with need for ACL support could just be made to use ModPerl:: RegistryACL, for example ...
signature.asc
Description: This is a digitally signed message part