{"thread":{"id":"58776","subject":"[PATCH] Documentation: increase example cache timeout to 1 hour","startedAt":"2022-11-09T10:19:51Z","lastAt":"2022-11-10T02:30:04Z","messageCount":5,"participants":["M Hickford via GitGitGadget","Jeff King","M Hickford","Taylor Blau"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"466944","messageId":"pull.1412.git.1667989181611.gitgitgadget@gmail.com","threadId":"58776","inReplyTo":null,"subject":"[PATCH] Documentation: increase example cache timeout to 1 hour","fromName":"M Hickford via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2022-11-09T10:19:41Z","receivedAt":"2022-11-09T10:19:51Z","isPatch":true,"sender":{"key":"mirth.hickford@gmail.com","avatar":"https://avatars.githubusercontent.com/u/105314?v=4"},"body":"From: M Hickford \n\nPreviously, the example *decreased* the cache timeout compared to the\ndefault, nudging users to make cache less usable.\n\nInstead, nudge users to make cache more usable. Currently many users\nchoose store over cache for usability. See\nhttps://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n\nThe default timeout remains 15 minutes. A stronger nudge would\nbe to increase that.\n\nSigned-off-by: M Hickford \n---\n Documentation: Increase example cache timeout to 1 hour\n \n Previously, the example decreased the cache timeout compared to the\n default, nudging users to make cache less usable.\n \n Instead, nudge users to make cache more usable. Currently many users\n choose store over cache for usability. See\n https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n \n The default timeout remains 15 minutes. A stronger nudge would be to\n increase that.\n \n Signed-off-by: M Hickford mirth.hickford@gmail.com\n\nPublished-As: https://github.com/gitgitgadget/git/releases/tag/pr-1412%2Fhickford%2Fnudge-cache-longer-v1\nFetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1412/hickford/nudge-cache-longer-v1\nPull-Request: https://github.com/gitgitgadget/git/pull/1412\n\n Documentation/git-credential-cache.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-credential-cache.txt b/Documentation/git-credential-cache.txt\nindex 0216c18ef80..432e159d952 100644\n--- a/Documentation/git-credential-cache.txt\n+++ b/Documentation/git-credential-cache.txt\n@@ -69,10 +69,10 @@ $ git push http://example.com/repo.git\n ------------------------------------\n \n You can provide options via the credential.helper configuration\n-variable (this example drops the cache time to 5 minutes):\n+variable (this example increases the cache time to 1 hour):\n \n -------------------------------------------------------\n-$ git config credential.helper 'cache --timeout=300'\n+$ git config credential.helper 'cache --timeout=3600'\n -------------------------------------------------------\n \n GIT\n\nbase-commit: 319605f8f00e402f3ea758a02c63534ff800a711\n-- \ngitgitgadget\n"},{"id":"466948","messageId":"Y2uoko+ho9c5sfKf@coredump.intra.peff.net","threadId":"58776","inReplyTo":"pull.1412.git.1667989181611.gitgitgadget@gmail.com","subject":"Re: [PATCH] Documentation: increase example cache timeout to 1 hour","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2022-11-09T13:18:10Z","receivedAt":"2022-11-09T13:18:15Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Nov 09, 2022 at 10:19:41AM +0000, M Hickford via GitGitGadget wrote:\n\n> From: M Hickford \n> \n> Previously, the example *decreased* the cache timeout compared to the\n> default, nudging users to make cache less usable.\n\nI don't mind at all changing this as your patch does. The existing\nexample was mostly just to illustrate the syntax. But...\n\n> Instead, nudge users to make cache more usable. Currently many users\n> choose store over cache for usability. See\n> https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n\nI don't see how my email argues for this. The only thing I mentioned\nabout credential-cache there is that it's not available on all\nplatforms.\n\nBut if you want my opinion on its usability, the main problem is not\nthat the cache timeout. It's that entering the credential at all is a\npain, either because it's a semi-automated environment that needs to\noperate without user input, or because the credential itself is awkward\nfor the user to enter (like a long token). And that's what pushes people\nto \"store\" over \"cache\".\n\n> diff --git a/Documentation/git-credential-cache.txt b/Documentation/git-credential-cache.txt\n> index 0216c18ef80..432e159d952 100644\n> --- a/Documentation/git-credential-cache.txt\n> +++ b/Documentation/git-credential-cache.txt\n> @@ -69,10 +69,10 @@ $ git push http://example.com/repo.git\n> ------------------------------------\n> \n> You can provide options via the credential.helper configuration\n> -variable (this example drops the cache time to 5 minutes):\n> +variable (this example increases the cache time to 1 hour):\n> \n> -------------------------------------------------------\n> -$ git config credential.helper 'cache --timeout=300'\n> +$ git config credential.helper 'cache --timeout=3600'\n> -------------------------------------------------------\n\nThe patch itself is obviously correct.\n\n-Peff\n"},{"id":"466966","messageId":"CAGJzqsn3-_qM8K8_+5=z7-u-n1Oe-CfgjrSLF84RaFXUDMOChg@mail.gmail.com","threadId":"58776","inReplyTo":"Y2uoko+ho9c5sfKf@coredump.intra.peff.net","subject":"Re: [PATCH] Documentation: increase example cache timeout to 1 hour","fromName":"M Hickford","fromEmail":"mirth.hickford@gmail.com","sentAt":"2022-11-09T16:09:14Z","receivedAt":"2022-11-09T16:10:17Z","isPatch":true,"sender":{"key":"mirth.hickford@gmail.com","avatar":"https://avatars.githubusercontent.com/u/105314?v=4"},"body":"On Wed, 9 Nov 2022 at 13:18, Jeff King wrote:\n>\n> > Instead, nudge users to make cache more usable. Currently many users\n> > choose store over cache for usability. See\n> > https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n>\n> I don't see how my email argues for this. The only thing I mentioned\n> about credential-cache there is that it's not available on all\n> platforms.\n\nI'll amend the commit message.\n\n> But if you want my opinion on its usability, the main problem is not\n> that the cache timeout. It's that entering the credential at all is a\n> pain, either because it's a semi-automated environment that needs to\n> operate without user input, or because the credential itself is awkward\n> for the user to enter (like a long token). And that's what pushes people\n> to \"store\" over \"cache\".\n\nI agree. I have some ideas to help human users; I'll share in the other thread.\n\n> The patch itself is obviously correct.\n\nThanks for the review\n"},{"id":"466967","messageId":"pull.1412.v2.git.1668010273573.gitgitgadget@gmail.com","threadId":"58776","inReplyTo":"pull.1412.git.1667989181611.gitgitgadget@gmail.com","subject":"[PATCH v2] Documentation: increase example cache timeout to 1 hour","fromName":"M Hickford via GitGitGadget","fromEmail":"gitgitgadget@gmail.com","sentAt":"2022-11-09T16:11:13Z","receivedAt":"2022-11-09T16:11:42Z","isPatch":true,"sender":{"key":"mirth.hickford@gmail.com","avatar":"https://avatars.githubusercontent.com/u/105314?v=4"},"body":"From: M Hickford \n\nPreviously, the example *decreased* the cache timeout compared to the\ndefault, making it less user friendly.\n\nInstead, nudge users to make cache more usable. Many users choose\nstore over cache.\nhttps://lore.kernel.org/git/CAGJzqskRYN49SeS8kSEN5-vbB_Jt1QvAV9QhS6zNuKh0u8wxPQ@mail.gmail.com/\n\nThe default timeout remains 15 minutes. A stronger nudge would\nbe to increase that.\n\nSigned-off-by: M Hickford \n---\n Documentation: Increase example cache timeout to 1 hour\n \n Previously, the example decreased the cache timeout compared to the\n default, nudging users to make cache less usable.\n \n Instead, nudge users to make cache more usable. Currently many users\n choose store over cache for usability. See\n https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n \n The default timeout remains 15 minutes. A stronger nudge would be to\n increase that.\n \n Signed-off-by: M Hickford mirth.hickford@gmail.com\n\nPublished-As: https://github.com/gitgitgadget/git/releases/tag/pr-1412%2Fhickford%2Fnudge-cache-longer-v2\nFetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1412/hickford/nudge-cache-longer-v2\nPull-Request: https://github.com/gitgitgadget/git/pull/1412\n\nRange-diff vs v1:\n\n 1: 6c35d68cf98 ! 1: e3e043779dd Documentation: increase example cache timeout to 1 hour\n @@ Commit message\n Documentation: increase example cache timeout to 1 hour\n \n Previously, the example *decreased* the cache timeout compared to the\n - default, nudging users to make cache less usable.\n + default, making it less user friendly.\n \n - Instead, nudge users to make cache more usable. Currently many users\n - choose store over cache for usability. See\n - https://lore.kernel.org/git/Y2p4rhiOphuOM0VQ@coredump.intra.peff.net/\n + Instead, nudge users to make cache more usable. Many users choose\n + store over cache.\n + https://lore.kernel.org/git/CAGJzqskRYN49SeS8kSEN5-vbB_Jt1QvAV9QhS6zNuKh0u8wxPQ@mail.gmail.com/\n \n The default timeout remains 15 minutes. A stronger nudge would\n be to increase that.\n\n\n Documentation/git-credential-cache.txt | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/Documentation/git-credential-cache.txt b/Documentation/git-credential-cache.txt\nindex 0216c18ef80..432e159d952 100644\n--- a/Documentation/git-credential-cache.txt\n+++ b/Documentation/git-credential-cache.txt\n@@ -69,10 +69,10 @@ $ git push http://example.com/repo.git\n ------------------------------------\n \n You can provide options via the credential.helper configuration\n-variable (this example drops the cache time to 5 minutes):\n+variable (this example increases the cache time to 1 hour):\n \n -------------------------------------------------------\n-$ git config credential.helper 'cache --timeout=300'\n+$ git config credential.helper 'cache --timeout=3600'\n -------------------------------------------------------\n \n GIT\n\nbase-commit: 319605f8f00e402f3ea758a02c63534ff800a711\n-- \ngitgitgadget\n"},{"id":"467024","messageId":"Y2xiJEhmcf3KDK1p@nand.local","threadId":"58776","inReplyTo":"CAGJzqsn3-_qM8K8_+5=z7-u-n1Oe-CfgjrSLF84RaFXUDMOChg@mail.gmail.com","subject":"Re: [PATCH] Documentation: increase example cache timeout to 1 hour","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-10T02:29:56Z","receivedAt":"2022-11-10T02:30:04Z","isPatch":true,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Wed, Nov 09, 2022 at 04:09:14PM +0000, M Hickford wrote:\n> > The patch itself is obviously correct.\n>\n> Thanks for the review\n\nThanks, both. I picked up the updated round.\n\nThanks,\nTaylor\n"}]}