Subject: Re: [PATCH 0/2] For v2.37.0: rename fetch.credentialsInUrl to transfer.credentialsInUrl

From: Junio C Hamano <[email-masked]>
Date: Wed, 15 Jun 2022 11:35:55 -0700

Derrick Stolee <[email-masked]> writes:

> This rename makes sense and is good to handle before the release.
> Thank you for updating the release notes.
>
> The expanded details in the docs are good.

Generally, transfer.$name hierarchy is there to give defaults to
fetch.$name and push.$name so that the knob can be set separately
for different directions of the transfer, but for this particular
one, I do not see a reason why we want to redact fetch requests
without redacting push requests.

One thing that worries me is that this will probably burn our early
adopters, those who are willing to help our development by running
versions based on 'next' and already have fetch.* configuration.
But the breakage this change brings on them hopefully will not be
large enough to discourage them from being guinea pigs ;-)

Thanks, all.