George Hartzell writes:
 > [...]
 >   static-content.patch extends the entry-points table so that you can
 >     point specific entry-points at directories, which are then used as
 >     the docroot in a call to serve-static.

Some self-feedback.

Around line 83 of the patched Server.pm I checked to see if the
directory exists and is *scannable*.  In retrospect I think that I
want *readable* (-r).

I think that an existence test is redundant given the directory-ness
test.

The new serve_response method might be better called _serve_response,
the privacy implied by a leading underscore would excuse the lack of
pod documentation.

Thoughts?

g.




#####  CGI::Application community mailing list  ################
##                                                            ##
##  To unsubscribe, or change your message delivery options,  ##
##  visit:  http://www.erlbaum.net/mailman/listinfo/cgiapp    ##
##                                                            ##
##  Web archive:   http://www.erlbaum.net/pipermail/cgiapp/   ##
##  Wiki:          http://cgiapp.erlbaum.net/                 ##
##                                                            ##
################################################################

Reply via email to