Re: [PATCH 1/1] curl: anonymize URLs in error messages and warnings

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mar 5, 2019, 22:55 UTC
Message-ID: <nycvar.QRO.7.76.6.1903051709330.41@tvgsbejvaqbjf.bet>
In-Reply-To: <20190305051126.GL19800@sigill.intra.peff.net>

Hi Peff,

On Tue, 5 Mar 2019, Jeff King wrote:

Show 13 quoted lines

> On Mon, Mar 04, 2019 at 07:33:46AM -0800, Johannes Schindelin via GitGitGadget wrote:
> 
> > @@ -442,17 +443,23 @@ static struct discovery *discover_refs(const char *service, int for_push)
> >  		break;
> >  	case HTTP_MISSING_TARGET:
> >  		show_http_message(&type, &charset, &buffer);
> > -		die("repository '%s' not found", url.buf);
> > +		die("repository '%s' not found",
> > +		    transport_anonymize_url(url.buf));
> 
> Sort of orthogonal to your patch, but these could probably stand to be
> marked for translation (I'm happy for that to come on top later,
> though).

Good idea!

Ciao, Dscho