On Wed, 9 Nov 2022 at 13:18, Jeff King <peff@xxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxxxx/ > > 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