Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
Am Montag, 22. Juli 2013, 19:58:11 schrieb Steinar H. Gunderson: On Wed, Jul 10, 2013 at 03:07:56PM -0400, Jeff Trawick wrote: I guess it seems to work in the earlier e-mail is the validation that the API is sufficient for MPM-ITK. I see that 2.4.6 has been released, with no mention of open_htaccess in the source code. Was this reverted after 2.4.5? Or was it simply never applied to the 2.4.x branch? Unfortunately, it didn't get the necessary three +1 votes in time and hasn't been applied to the 2.4.x branch, yet.
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On Wed, Jul 24, 2013 at 3:31 PM, Stefan Fritsch s...@sfritsch.de wrote: Am Montag, 22. Juli 2013, 19:58:11 schrieb Steinar H. Gunderson: On Wed, Jul 10, 2013 at 03:07:56PM -0400, Jeff Trawick wrote: I guess it seems to work in the earlier e-mail is the validation that the API is sufficient for MPM-ITK. I see that 2.4.6 has been released, with no mention of open_htaccess in the source code. Was this reverted after 2.4.5? Or was it simply never applied to the 2.4.x branch? Unfortunately, it didn't get the necessary three +1 votes in time and hasn't been applied to the 2.4.x branch, yet. Now it is approved for 2.4.next. -- Born in Roswell... married an alien... http://emptyhammock.com/
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On Wed, Jul 10, 2013 at 03:07:56PM -0400, Jeff Trawick wrote: I guess it seems to work in the earlier e-mail is the validation that the API is sufficient for MPM-ITK. Hi, I see that 2.4.6 has been released, with no mention of open_htaccess in the source code. Was this reverted after 2.4.5? Or was it simply never applied to the 2.4.x branch? /* Steinar */ -- Homepage: http://www.sesse.net/
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On Tue, Jul 09, 2013 at 08:53:03AM -0400, Jeff Trawick wrote: Do you have time to test with this patch on top of 2.4.x and report back? http://people.apache.org/~sf/open_htaccess_hook.patch Hi, I've tried this, adjusted mpm-itk, and it seems to work. Why do I need to return AP_DECLINED and not DECLINED from this hook, though? FWIW, here's my hook: ap_hook_open_htaccess(itk_open_htaccess, NULL, NULL, APR_HOOK_REALLY_FIRST); [...] static apr_status_t itk_open_htaccess(request_rec *r, const char *dir_name, const char *access_name, ap_configfile_t **conffile, const char **full_name) { int status; if (!ap_has_irreversibly_setuid || r-main != NULL) { return AP_DECLINED; } *full_name = ap_make_full_path(r-pool, dir_name, access_name); status = ap_pcfg_openfile(conffile, r-pool, *full_name); if (APR_STATUS_IS_EACCES(status)) { ap_log_rerror(APLOG_MARK, APLOG_WARNING, errno, r, Couldn't read %s, closing connection., *full_name); ap_lingering_close(r-connection); exit(0); } return status; } I don't like all that much having to duplicate the “official” hook (in particular the ap_make_full_path() call), but I guess it's better than what used to be there, and it's only two lines. /* Steinar */ -- Homepage: http://www.sesse.net/
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On 10.07.2013 13:14, Steinar H. Gunderson wrote: On Tue, Jul 09, 2013 at 08:53:03AM -0400, Jeff Trawick wrote: Do you have time to test with this patch on top of 2.4.x and report back? http://people.apache.org/~sf/open_htaccess_hook.patch Hi, I've tried this, adjusted mpm-itk, and it seems to work. Why do I need to return AP_DECLINED and not DECLINED from this hook, though? http://mail-archives.apache.org/mod_mbox/httpd-dev/201307.mbox/%3calpine.deb.2.00.1307031352000.14...@eru.sfritsch.de%3E Regards, Rainer
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On Wednesday 10 July 2013, Steinar H. Gunderson wrote: I don't like all that much having to duplicate the “official” hook (in particular the ap_make_full_path() call), but I guess it's better than what used to be there, and it's only two lines. Yes, that's the price to pay for the more flexible hook which supports additional use cases. I think these two lines are acceptable. Cheers, Stefan
Re: Hey Steinar... Re: Revisiting the pre_htaccess hook
On Wed, Jul 10, 2013 at 2:30 PM, Stefan Fritsch s...@sfritsch.de wrote: On Wednesday 10 July 2013, Steinar H. Gunderson wrote: I don't like all that much having to duplicate the “official” hook (in particular the ap_make_full_path() call), but I guess it's better than what used to be there, and it's only two lines. Yes, that's the price to pay for the more flexible hook which supports additional use cases. I think these two lines are acceptable. I think this issue was the original objection to solving the problem with this particular API (voiced or unvoiced), but I agree that the additional use cases make it worthwhile. I guess it seems to work in the earlier e-mail is the validation that the API is sufficient for MPM-ITK. Cheers, Stefan -- Born in Roswell... married an alien... http://emptyhammock.com/
Hey Steinar... Re: Revisiting the pre_htaccess hook
On Sun, Jun 9, 2013 at 8:00 AM, Steinar H. Gunderson sgunder...@bigfoot.com wrote: On Sun, Jun 09, 2013 at 11:57:54AM +0200, Stefan Fritsch wrote: 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: Either way is fine by me; the only thing mpm-itk needs is really DECLINED or exit(). :-) Do you have time to test with this patch on top of 2.4.x and report back? http://people.apache.org/~sf/open_htaccess_hook.patch Thanks! /* Steinar */ -- Homepage: http://www.sesse.net/ -- Born in Roswell... married an alien... http://emptyhammock.com/