Hi,

I would +1 for this solution which is, IMHO, much better.
However, changing the parser itself would require checking the potential impact as it is used in many places.

CJ

Le 17/04/2015 13:15, Yann Ylavic a écrit :
On Fri, Apr 17, 2015 at 11:54 AM, Yann Ylavic <ylavic....@gmail.com> wrote:
How about handling the case in the Require directive parser itself
(add_authz_provider)?
Or even maybe in the expr parser itself:

Index: server/util_expr_eval.c
===================================================================
--- server/util_expr_eval.c    (revision 1674046)
+++ server/util_expr_eval.c    (working copy)
@@ -455,6 +455,10 @@ AP_DECLARE(ap_expr_info_t*) ap_expr_parse_cmd_mi(c
      info->line_number = cmd->directive->line_num;
      info->flags = flags;
      info->module_index = module_index;
+    if (expr && (expr[0] == '"' || expr[0] == '\'') && (expr[1] != '\0')
+             && (expr[strlen(expr) - 1] == expr[0])) {
+        expr = ap_getword_conf(cmd->temp_pool, &expr);
+    }
      *err = ap_expr_parse(cmd->pool, cmd->temp_pool, info, expr, lookup_fn);

      if (*err)
--


Reply via email to