Re: svn commit: r1646724 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_fcgi.c

2014-12-21 Thread Christophe JAILLET
Hi, Le 19/12/2014 15:03, cove...@apache.org a écrit : Author: covener Date: Fri Dec 19 14:03:52 2014 New Revision: 1646724 URL: http://svn.apache.org/r1646724 Log: tweak SCRIPT_FILENAME passed to fastcgi backends when a balancer is used. Modified: httpd/httpd/trunk/CHANGES

Re: svn commit: r1642154 - in /httpd/httpd/trunk: docs/manual/expr.xml include/ap_expr.h server/util_expr_eval.c server/util_expr_parse.y

2014-12-21 Thread Graham Leggett
On 27 Nov 2014, at 3:46 PM, jkal...@apache.org wrote: Author: jkaluza Date: Thu Nov 27 13:46:11 2014 New Revision: 1642154 URL: http://svn.apache.org/r1642154 Log: * ap_exr: Add replace(string, from, to) function. Modified: httpd/httpd/trunk/docs/manual/expr.xml

Re: svn commit: r1642154 - in /httpd/httpd/trunk: docs/manual/expr.xml include/ap_expr.h server/util_expr_eval.c server/util_expr_parse.y

2014-12-21 Thread Eric Covener
On Sun, Dec 21, 2014 at 6:37 AM, Graham Leggett minf...@sharp.fm wrote: I am currently getting the error “sss” (not sure what that means?) when attempting to use named regex variables in LocationMatch - can you confirm this works as expected? When I saw this I was sure it was my own

[Patch] Simplifying mod_alias

2014-12-21 Thread Graham Leggett
On 27 Jan 2014, at 12:11 AM, GRAHAM LEGGETT minf...@sharp.fm wrote: A look at mod_alias shows it has 7 directives: • Alias • AliasMatch • Redirect • RedirectMatch • RedirectPermanent • RedirectTemp • ScriptAlias • ScriptAliasMatch In theory we only need these three: • Alias •

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread André Malo
Hi, I think, most of the directives are compatibility ones. IMHO the best way to handle a transition to a different configuration concept would be to introduce a new module (mod_alias_ng or mod_fs_map or so...) instead of patching the current one and producing a lot of anger (Alias* and

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Graham Leggett
On 21 Dec 2014, at 4:18 PM, André Malo n...@perlig.de wrote: I think, most of the directives are compatibility ones. IMHO the best way to handle a transition to a different configuration concept would be to introduce a new module (mod_alias_ng or mod_fs_map or so...) instead of patching

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Helmut K. C. Tessarek
On 21.12.14 9:41 , Graham Leggett wrote: What we should do in future is remove all the *Match directives and move them into a mod_alias_compat module, leaving just Alias/Redirect/ScriptAlias in mod_alias, same as we did with authnz in httpd v2.4. +1. In SW development a lot of people mixup

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Tim Bannister
On 21 Dec 2014, at 13:48, Graham Leggett minf...@sharp.fm wrote: This patch implements the above. The idea is that the existing syntaxes remain unaltered (and can be deprecated in future), while we introduce new Location syntaxes with a single argument, like so: Location /image

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Eric Covener
On Sun, Dec 21, 2014 at 9:41 AM, Graham Leggett minf...@sharp.fm wrote: What we should do in future is remove all the *Match directives and move them into a mod_alias_compat module, leaving just Alias/Redirect/ScriptAlias in mod_alias, same as we did with authnz in httpd v2.4. I am -1 on

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Graham Leggett
On 21 Dec 2014, at 7:37 PM, Eric Covener cove...@gmail.com wrote: I am -1 on moving them to a _compat module. I don't see a technical reason, there's nothing wrong with the current code or function, and I think there has to be a stronger case to justify adding more upgrade speedbumps. The

Re: [Patch] Simplifying mod_alias

2014-12-21 Thread Eric Covener
On Sun, Dec 21, 2014 at 1:57 PM, Graham Leggett minf...@sharp.fm wrote: I am -1 on moving them to a _compat module. I don't see a technical reason, there's nothing wrong with the current code or function, and I think there has to be a stronger case to justify adding more upgrade speedbumps.