Re: [PATCH v5] credential: fix matching URLs with multiple levels in path
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 27, 2020, 18:44 UTC
- Message-ID
- <xmqqblnc93bd.fsf@gitster.c.googlers.com>
- In-Reply-To
- <20200427011809.2494377-1-sandals@crustytoothpaste.net>
"brian m. carlson" <sandals@crustytoothpaste.net> writes:
Show 7 quoted lines
> I talked to Carlo off-list because he was concerned about the potential > security implications of the patch, and as part of that discussion he > suggested an additional test for our UTF-8 handling. Since adding some > additional tests for that case seemed like a good idea, I squashed in a > patch of his to handle that case. > > That's the only change between v4 and v5.
With the polarity flip of the option done in v4, the resulting code is quite easy to follow. Thanks, will queue.