Seema Alevoor wrote: > > > Jeff Trawick wrote: >> Seema Alevoor wrote: >>> Hi, >>> >>> Webrev is at http://cr.opensolaris.org/~seema/6782598/ >>> >>> Please review. >>> >>> Changes: >>> - Fixed runtime and system configuration paths within dav.conf >>> - Changed configuration within info.conf to allow >>> access to /server-status and /server-info from localhost. >>> >> dav.conf: >> by any chance to you have a copy of dav.conf with all the variables >> substituted? >> > Attached. >> info.conf: >> >> It is installed to the samples-conf.d directory, so no worries here >> that we enable the status pages for loopback, since the admin has to >> copy to conf.d. >> >> There is a performance consideration however with the DNS lookup >> (allow from .example.com). >> >> Before: >> admin has to copy to conf.d but they don't have a working >> configuration until they fix the "allow from" to allow access to >> valid clients (a lot of admins did s/.example.com/127.0.0.1/) >> >> Now: >> admin just has to copy to conf.d and it starts working for loopback, >> but Apache first does a reverse DNS lookup to see if the client's >> hostname matches .example.com >> >> I suggest commenting out the "Allow from .example.com" to avoid the >> working-in-degraded-mode scenario. >> > Agreed. I have updated the webrev.
+1
