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

From: Junio C Hamano <gitster@pobox.com>
Date: Jun 15, 2022, 18:35 UTC
Message-ID: <xmqqy1xxzsdw.fsf@gitster.g>
In-Reply-To: <e53a247d-1d12-e34c-6cc8-3813a2fa34dc@github.com>

Derrick Stolee <derrickstolee@github.com> 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.