Re: [RFC PATCH] credential: minor documentation fixes
- From
Jeff King <peff@peff.net>
- Date
- May 4, 2020, 16:13 UTC
- Message-ID
- <20200504161342.GD12842@coredump.intra.peff.net>
- In-Reply-To
- <20200504155837.GD86805@Carlos-MBP>
On Mon, May 04, 2020 at 08:58:37AM -0700, Carlo Marcelo Arenas Belón wrote:
> what I meant is that we don't REALLY allow those modification to take place > since the credential helper suggestions can't really affect what git assumes > is the right host or protocol (or FWIW path), unlike username/password and > so they don't seem worth having IMHO.
I guess it depends on your definition of REALLY. Sure, we won't access a different host via curl (or a different protocol!), but I wouldn't expect that. This is the credential subsystem.
If you're proposing to document it more clearly, I'm fine with that.
And if you want to tighten it up in a major release to ignore such input from helpers, I'm fine with that, too. But I'd be worried pushing it out in a maintenance release and finding out that somebody _is_ depending on it somehow.
-Peff