Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Message-ID: <3F12FE44.2020709@remulak.net>
Date: Mon, 14 Jul 2003 15:02:28 -0400
From: "Paul J. Reder" <rederpj@remulak.net>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030312
MIME-Version: 1.0
To: dev@httpd.apache.org
Subject: Re: cvs commit: httpd-2.0/modules/mappers mod_rewrite.c
References: <20030714164014.51197.qmail@icarus.apache.org>
 <n2m-g.1940505.r7ovG9ReDJ@news.perlig.de>
In-Reply-To: <n2m-g.1940505.r7ovG9ReDJ@news.perlig.de>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit

Quite right. In the original submitted patch there were several extra
bits which I intended to remove while updating the patch. I removed
one, but forgot the other. You are correct that this code is not part
of the fix. Thanks for the catch. It'll be gone in a sec...

Andr� Malo wrote:
> * rederpj@apache.org wrote:
> 
> 
>>  Index: mod_rewrite.c
>>  ===================================================================
> 
> ...
> 
>>  @@ -2816,6 +2832,13 @@
>>           }
>>           else if (!strncasecmp(uri, "ntp://", 6)) {  /* nntp://   */
>>               return 7;
>>  +        }
>>  +        break;
>>  +
>>  +    case 'p':
>>  +    case 'P':
>>  +        if (!strncasecmp(uri, "roxy:", 5)) {       /* proxy:   */
>>  +            return 6;
>>           }
>>           break;
>>       }
> 
> 
> uhmm. Why that? I bet that breaks some things, since proxy: is not a real
> scheme.
> 
> nd
> 

-- 
Paul J. Reder
-----------------------------------------------------------
"The strength of the Constitution lies entirely in the determination of each
citizen to defend it.  Only if every single citizen feels duty bound to do
his share in this defense are the constitutional rights secure."
-- Albert Einstein