From: M Hickford <mirth.hickford@gmail.com>
Date: Wed, 09 Nov 2022 16:09:14 GMT
Subject: Re: [PATCH] Documentation: increase example cache timeout to 1 hour
Message-ID: <CAGJzqsn3-_qM8K8_+5=z7-u-n1Oe-CfgjrSLF84RaFXUDMOChg@mail.gmail.com>
In-Reply-To: <Y2uoko+ho9c5sfKf@coredump.intra.peff.net>

On Wed, 9 Nov 2022 at 13:18, Jeff King <peff@peff.net> wrote:
>
> > 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.

I'll amend the commit message.

> 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".

I agree. I have some ideas to help human users; I'll share in the other thread.

> The patch itself is obviously correct.

Thanks for the review