You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by tr...@apache.org on 2002/01/17 03:22:23 UTC

cvs commit: httpd-2.0/modules/proxy mod_proxy.c

trawick     02/01/16 18:22:23

  Modified:    .        CHANGES
               modules/proxy mod_proxy.c
  Log:
  Fix a problem in the parsing of the <Proxy foo> directive.
  
  Revision  Changes    Path
  1.523     +4 -0      httpd-2.0/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/CHANGES,v
  retrieving revision 1.522
  retrieving revision 1.523
  diff -u -r1.522 -r1.523
  --- CHANGES	17 Jan 2002 00:54:32 -0000	1.522
  +++ CHANGES	17 Jan 2002 02:22:22 -0000	1.523
  @@ -1,4 +1,8 @@
   Changes with Apache 2.0.31-dev
  +
  +  *) Fix a problem in the parsing of the <Proxy foo> directive.
  +     [Jeff Trawick]
  +
     *) rewrite of mod_ssl input filter for better performance and less
        memory usage [Doug MacEachern]
   
  
  
  
  1.65      +2 -2      httpd-2.0/modules/proxy/mod_proxy.c
  
  Index: mod_proxy.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/modules/proxy/mod_proxy.c,v
  retrieving revision 1.64
  retrieving revision 1.65
  diff -u -r1.64 -r1.65
  --- mod_proxy.c	29 Nov 2001 21:09:31 -0000	1.64
  +++ mod_proxy.c	17 Jan 2002 02:22:22 -0000	1.65
  @@ -873,10 +873,10 @@
       cmd->path = ap_getword_conf(cmd->pool, &arg);
       cmd->override = OR_ALL|ACCESS_CONF;
   
  -    if (strncasecmp(cmd->path, "proxy:", 6))
  +    if (!strncasecmp(cmd->path, "proxy:", 6))
           cmd->path += 6;
   
  -    /* XXX Ignore case?  What if we proxy a case-insenstive server?!? 
  +    /* XXX Ignore case?  What if we proxy a case-insensitive server?!? 
        * While we are at it, shouldn't we also canonicalize the entire
        * scheme?  See proxy_fixup()
        */