From: Junio C Hamano Date: Mon, 27 Apr 2020 18:44:06 GMT Subject: Re: [PATCH v5] credential: fix matching URLs with multiple levels in path Message-ID: In-Reply-To: <20200427011809.2494377-1-sandals@crustytoothpaste.net> "brian m. carlson" writes: > 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.