Re: [PATCH] Documentation: increase example cache timeout to 1 hour
- From
Jeff King <peff@peff.net>
- Date
- Nov 9, 2022, 13:18 UTC
- Message-ID
- <Y2uoko+ho9c5sfKf@coredump.intra.peff.net>
- In-Reply-To
- <pull.1412.git.1667989181611.gitgitgadget@gmail.com>
On Wed, Nov 09, 2022 at 10:19:41AM +0000, M Hickford via GitGitGadget wrote:
> From: M Hickford <mirth.hickford@gmail.com> > > Previously, the example *decreased* the cache timeout compared to the > default, nudging users to make cache less usable.
I don't mind at all changing this as your patch does. The existing example was mostly just to illustrate the syntax. But...
> Instead, nudge users to make cache more usable. Currently many users > choose store over cache for usability. See > https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/
I don't see how my email argues for this. The only thing I mentioned about credential-cache there is that it's not available on all platforms.
But if you want my opinion on its usability, the main problem is not that the cache timeout. It's that entering the credential at all is a pain, either because it's a semi-automated environment that needs to operate without user input, or because the credential itself is awkward for the user to enter (like a long token). And that's what pushes people to "store" over "cache".
Show 15 quoted lines
> diff --git a/Documentation/git-credential-cache.txt b/Documentation/git-credential-cache.txt > index 0216c18ef80..432e159d952 100644 > --- a/Documentation/git-credential-cache.txt > +++ b/Documentation/git-credential-cache.txt > @@ -69,10 +69,10 @@ $ git push http://example.com/repo.git > ------------------------------------ > > You can provide options via the credential.helper configuration > -variable (this example drops the cache time to 5 minutes): > +variable (this example increases the cache time to 1 hour): > > ------------------------------------------------------- > -$ git config credential.helper 'cache --timeout=300' > +$ git config credential.helper 'cache --timeout=3600' > -------------------------------------------------------
The patch itself is obviously correct.
-Peff