Re: [PATCH 0/3] Wildcard matching for credentials

From: brian m. carlson <sandals@crustytoothpaste.net>
Date: Feb 15, 2020, 00:13 UTC
Message-ID: <20200215001357.GB6134@camp.crustytoothpaste.net>
In-Reply-To: <20200214235822.GA11400@syl.local>

On 2020-02-14 at 23:58:22, Taylor Blau wrote:

Show 15 quoted lines

> Hi brian,
> 
> On Fri, Feb 14, 2020 at 10:59:26PM +0000, brian m. carlson wrote:
> > This series introduces wildcard matching (that is, urlmatch support) for
> > credential config options, just like for the http options.  This is
> > helpful in corporate environments where custom credentials should be
> > used across a wide variety of subdomains.
> >
> > In addition, there's an additional test for urlmatch behavior with
> > multiple subdomains and a mailmap update for the email address used in
> > this series.
> 
> I can imagine that this is perhaps for Git LFS, which I could see
> benefiting from this change. My review has nothing to do with my
> affiliation (or lack thereof) to LFS.

It was originally prompted by a discussion that someone started on the Git LFS issue tracker, yes. I pointed out that it's actually Git that controls credential helper matching and so I sent a patch.

I've also worked somewhere with multiple internal Git servers, so I can imagine this would also be valuable in such an environment.

> I gave your patches a review, and they all look quite good to me. Thanks
> especially for 2/3, which I would have suggested were it not already
> there ;-).

I think there was some discussion on the list about whether allowing multiple wildcards and wildcards in any part of the domain name was intentional or not, and it was decided that it was. I promised to follow up with a patch to the testsuite, and here I am, some time later.

> This looks good to me, so please have my:
> 
>   Reviewed-by: Taylor Blau <me@ttaylorr.com>

Thanks.

-- 
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204