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.
Thanks, Seema. -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: dav.conf URL: <http://mail.opensolaris.org/pipermail/webstack-discuss/attachments/20090226/4dee8ecc/attachment.ksh>
