Subject: Re: [PATCH v3] http: do not ignore proxy path

From: Junio C Hamano <[email-masked]>
Date: Wed, 31 Jul 2024 22:21:13 -0700

Ryan Hendrickson <[email-masked]> writes:

>> We insist that it must be "localhost", so let's not do strcasecmp()
>> but just do strcmp().
>
> I don't see the wisdom of being more restrictive than curl is in this
> respect.

Ah, if curl is doing case insensitivity, then matching its behaviour
is perfectly fine.  I was just reacting to this message ...

> +	die("Invalid proxy URL '%s': host must be localhost if a path is present",

... that results when strcasecmp() does not like it.

Thanks.