Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Received-SPF: error (athena.apache.org: encountered temporary error during SPF
 processing of domain of jim@jagunet.com)
Content-Type: text/plain; charset=iso-8859-1
Mime-Version: 1.0 (Mac OS X Mail 6.5 \(1508\))
Subject: Re: Revisiting the pre_htaccess hook
From: Jim Jagielski <jim@jaguNET.com>
In-Reply-To: 
 <CAKUrXK70C478qy=gVDd-A-6MsCfdhM_FZeS4KY_jbmwL-j+sNA@mail.gmail.com>
Date: Wed, 26 Jun 2013 09:01:58 -0400
Cc: Stefan Fritsch <sf@sfritsch.de>,
 "Steinar H. Gunderson" <sgunderson@bigfoot.com>
Content-Transfer-Encoding: 7bit
Message-Id: <437B5AE7-D562-4CCB-B3D8-D586AC545E43@jaguNET.com>
References: <201306091157.55040.sf@sfritsch.de>
 <CAKUrXK70C478qy=gVDd-A-6MsCfdhM_FZeS4KY_jbmwL-j+sNA@mail.gmail.com>
To: dev@httpd.apache.org

I like the idea as well... The only issue I see is wondering
if/when we'll have the implementation.

On Jun 25, 2013, at 9:12 AM, Jeff Trawick <trawick@gmail.com> wrote:

> On Sun, Jun 9, 2013 at 5:57 AM, Stefan Fritsch <sf@sfritsch.de> wrote:
> Hi,
> 
> first of all, sorry that it took me so long to review at this.
> 
> The current pre_access hook is executed before opening the htaccess
> and then can abort the request with a HTTP error code.
> 
> Wouldn't a hook for opening the htaccess file make more sense because
> it would have more possible use cases? Then modules could use this
> hook to find htaccess files somewhere else, generate/extend/filter
> them on the fly, etc. For example:
> 
> apr_status_t  ap_run_open_htaccess(request_rec *r, ap_configfile_t *f,
> const char **full_name, const char *dirname, const char *accessname);
> 
> The hook would then build full_name (used later for error logging)
> from dirname and accessname and open it with ap_pcfg_openfile().
> 
> Downside would be that the hook cannot provide an arbitrary HTTP
> status code. Only no error and FORBIDDEN. Would that be a problem?
> 
> We could add an additional int *request_status parameter, too, though.
> Or put the error logging/generating of FORBIDDEN into core's
> open_htaccess hook function and make the hook return HTTP status and
> not apr_status_t.
> 
> What do you think?
> 
> +1
>  
> 
> Cheers,
> Stefan
> 
> 
> 
> -- 
> Born in Roswell... married an alien...
> http://emptyhammock.com/