* [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
@ 2020-08-17 12:54 Vasyl Vavrychuk
2020-08-17 17:16 ` Eric Sunshine
0 siblings, 1 reply; 9+ messages in thread
From: Vasyl Vavrychuk @ 2020-08-17 12:54 UTC (permalink / raw)
To: git; +Cc: Vasyl Vavrychuk
Looks like Google changed gmail security and now less secure app access
needs to be explicitly enabled if 2-factor authentication is not in
place, otherwise send-mail fails with:
5.7.8 Username and Password not accepted. Learn more at
5.7.8 https://support.google.com/mail/?p=BadCredentials v5sm13756502ede.13 - gsmtp
---
Documentation/git-send-email.txt | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 0a69810147..5aed3f5b1d 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -494,6 +494,10 @@ edit ~/.gitconfig to specify your account settings:
smtpServerPort = 587
----
+If you do not have multifactor authentication setup on your gmail account, you
+need to allow less secure app access. Visit
+https://myaccount.google.com/lesssecureapps to enable it.
+
If you have multifactor authentication setup on your gmail account, you will
need to generate an app-specific password for use with 'git send-email'. Visit
https://security.google.com/settings/security/apppasswords to create it.
@@ -505,7 +509,7 @@ following commands:
$ edit outgoing/0000-*
$ git send-email outgoing/*
-The first time you run it, you will be prompted for your credentials. Enter the
+The first time you run it, you will be prompted for your credentials. Enter the
app-specific or your regular password as appropriate. If you have credential
helper configured (see linkgit:git-credential[1]), the password will be saved in
the credential store so you won't have to type it the next time.
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-08-17 12:54 [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable Vasyl Vavrychuk
@ 2020-08-17 17:16 ` Eric Sunshine
2020-08-29 15:39 ` Vasyl Vavrychuk
0 siblings, 1 reply; 9+ messages in thread
From: Eric Sunshine @ 2020-08-17 17:16 UTC (permalink / raw)
To: Vasyl Vavrychuk; +Cc: Git List
On Mon, Aug 17, 2020 at 8:54 AM Vasyl Vavrychuk <vvavrychuk@gmail.com> wrote:
> Looks like Google changed gmail security and now less secure app access
> needs to be explicitly enabled if 2-factor authentication is not in
> place, otherwise send-mail fails with:
>
> 5.7.8 Username and Password not accepted. Learn more at
> 5.7.8 https://support.google.com/mail/?p=BadCredentials v5sm13756502ede.13 - gsmtp
> ---
Missing sign-off. Please see Documentation/SubmittingPatches.
> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
> @@ -494,6 +494,10 @@ edit ~/.gitconfig to specify your account settings:
> +If you do not have multifactor authentication setup on your gmail account, you
> +need to allow less secure app access. Visit
> +https://myaccount.google.com/lesssecureapps to enable it.
Thanks. Make sense. A few comments:
I think multi-factor is usually hyphenated, so perhaps:
s/multifactor/multi-factor/
Since you are not using it as a noun: s/setup/set up/
Also, Gmail is normally capitalized, so: s/gmail/Gmail
> @@ -505,7 +509,7 @@ following commands:
> -The first time you run it, you will be prompted for your credentials. Enter the
> +The first time you run it, you will be prompted for your credentials. Enter the
This unnecessary whitespace change just adds noise to the patch, thus
should be dropped.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-08-17 17:16 ` Eric Sunshine
@ 2020-08-29 15:39 ` Vasyl Vavrychuk
2020-08-29 18:33 ` Junio C Hamano
2021-01-07 11:11 ` [PATCH v3] git-send-email.txt: mention less secure app access with Gmail Denton Liu
0 siblings, 2 replies; 9+ messages in thread
From: Vasyl Vavrychuk @ 2020-08-29 15:39 UTC (permalink / raw)
To: Eric Sunshine; +Cc: Git List, Vasyl Vavrychuk
Looks like Google changed Gmail security and now less secure app access
needs to be explicitly enabled if 2-factor authentication is not in
place, otherwise send-mail fails with:
5.7.8 Username and Password not accepted. Learn more at
5.7.8 https://support.google.com/mail/?p=BadCredentials v5sm13756502ede.13 - gsmtp
Signed-off-by: Vasyl Vavrychuk <vvavrychuk@gmail.com>
---
Documentation/git-send-email.txt | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 0a69810147..06953fd1e0 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -494,7 +494,11 @@ edit ~/.gitconfig to specify your account settings:
smtpServerPort = 587
----
-If you have multifactor authentication setup on your gmail account, you will
+If you do not have multi-factor authentication set up on your Gmail account, you
+will need to allow less secure app access. Visit
+https://myaccount.google.com/lesssecureapps to enable it.
+
+If you have multi-factor authentication set up on your Gmail account, you will
need to generate an app-specific password for use with 'git send-email'. Visit
https://security.google.com/settings/security/apppasswords to create it.
--
2.23.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-08-29 15:39 ` Vasyl Vavrychuk
@ 2020-08-29 18:33 ` Junio C Hamano
2020-08-29 19:29 ` Junio C Hamano
2021-01-07 11:11 ` [PATCH v3] git-send-email.txt: mention less secure app access with Gmail Denton Liu
1 sibling, 1 reply; 9+ messages in thread
From: Junio C Hamano @ 2020-08-29 18:33 UTC (permalink / raw)
To: Vasyl Vavrychuk; +Cc: Eric Sunshine, Git List
Vasyl Vavrychuk <vvavrychuk@gmail.com> writes:
> Looks like Google changed Gmail security and now less secure app access
> needs to be explicitly enabled if 2-factor authentication is not in
> place, otherwise send-mail fails with:
'send-email' I presume?
> 5.7.8 Username and Password not accepted. Learn more at
> 5.7.8 https://support.google.com/mail/?p=BadCredentials v5sm13756502ede.13 - gsmtp
Is the v5sm1375... part stay constant that everybody would get the
same thing?
Otherwise,
... otherwise send-email fails with an error message from Gmail
server telling the user to "Learn more" at
https://support.google.com/mail/?p=BadCredentials
would be preferrable.
> Signed-off-by: Vasyl Vavrychuk <vvavrychuk@gmail.com>
> ---
> Documentation/git-send-email.txt | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
> index 0a69810147..06953fd1e0 100644
> --- a/Documentation/git-send-email.txt
> +++ b/Documentation/git-send-email.txt
> @@ -494,7 +494,11 @@ edit ~/.gitconfig to specify your account settings:
> smtpServerPort = 587
> ----
>
> -If you have multifactor authentication setup on your gmail account, you will
> +If you do not have multi-factor authentication set up on your Gmail account, you
> +will need to allow less secure app access. Visit
> +https://myaccount.google.com/lesssecureapps to enable it.
> +
> +If you have multi-factor authentication set up on your Gmail account, you will
> need to generate an app-specific password for use with 'git send-email'. Visit
> https://security.google.com/settings/security/apppasswords to create it.
In 2020, we should probably list the instruction for those with 2
factor first and then give "less secure app access" as a fallback.
IOW, I'd suggest the order of these two paragraphs swapped.
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-08-29 18:33 ` Junio C Hamano
@ 2020-08-29 19:29 ` Junio C Hamano
2020-09-06 21:52 ` Junio C Hamano
0 siblings, 1 reply; 9+ messages in thread
From: Junio C Hamano @ 2020-08-29 19:29 UTC (permalink / raw)
To: Vasyl Vavrychuk; +Cc: Eric Sunshine, Git List
Junio C Hamano <gitster@pobox.com> writes:
> Vasyl Vavrychuk <vvavrychuk@gmail.com> writes:
Another thing I forgot to say.
Subject: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
Especially with grammofix s/to enable/to be enabled/ applied, the
above is way too long as a title and would stand out like a sore
thumb in "git shortlog --no-merges v2.28.0..v2.29.0" output.
Something like
Subject: [PATCH] send-email doc: mention less secure app access with GMail
perhaps.
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-08-29 19:29 ` Junio C Hamano
@ 2020-09-06 21:52 ` Junio C Hamano
2020-12-22 19:58 ` Junio C Hamano
0 siblings, 1 reply; 9+ messages in thread
From: Junio C Hamano @ 2020-09-06 21:52 UTC (permalink / raw)
To: Vasyl Vavrychuk; +Cc: Eric Sunshine, Git List
Junio C Hamano <gitster@pobox.com> writes:
> Junio C Hamano <gitster@pobox.com> writes:
>
>> Vasyl Vavrychuk <vvavrychuk@gmail.com> writes:
>
> Another thing I forgot to say.
>
> Subject: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
>
> Especially with grammofix s/to enable/to be enabled/ applied, the
> above is way too long as a title and would stand out like a sore
> thumb in "git shortlog --no-merges v2.28.0..v2.29.0" output.
> Something like
>
> Subject: [PATCH] send-email doc: mention less secure app access with GMail
>
> perhaps.
>
> Thanks.
Anything new on this topic? No rush, but I'd hate to see a
basically good topic to be left in the stalled state too long.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
2020-09-06 21:52 ` Junio C Hamano
@ 2020-12-22 19:58 ` Junio C Hamano
0 siblings, 0 replies; 9+ messages in thread
From: Junio C Hamano @ 2020-12-22 19:58 UTC (permalink / raw)
To: Vasyl Vavrychuk; +Cc: Eric Sunshine, Git List
Junio C Hamano <gitster@pobox.com> writes:
> Junio C Hamano <gitster@pobox.com> writes:
>
>> Junio C Hamano <gitster@pobox.com> writes:
>>
>>> Vasyl Vavrychuk <vvavrychuk@gmail.com> writes:
>>
>> Another thing I forgot to say.
>>
>> Subject: [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable.
>>
>> Especially with grammofix s/to enable/to be enabled/ applied, the
>> above is way too long as a title and would stand out like a sore
>> thumb in "git shortlog --no-merges v2.28.0..v2.29.0" output.
>> Something like
>>
>> Subject: [PATCH] send-email doc: mention less secure app access with GMail
>>
>> perhaps.
>>
>> Thanks.
>
> Anything new on this topic? No rush, but I'd hate to see a
> basically good topic to be left in the stalled state too long.
Just pinging old threads that we have been expecting a reroll, but
it has been more than 3 months. Has an update been sent that I
missed?
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v3] git-send-email.txt: mention less secure app access with Gmail
2020-08-29 15:39 ` Vasyl Vavrychuk
2020-08-29 18:33 ` Junio C Hamano
@ 2021-01-07 11:11 ` Denton Liu
2021-01-07 20:37 ` Eric Sunshine
1 sibling, 1 reply; 9+ messages in thread
From: Denton Liu @ 2021-01-07 11:11 UTC (permalink / raw)
To: Git Mailing List; +Cc: Vasyl Vavrychuk, Eric Sunshine, Junio C Hamano
Google may have changed Gmail security and now less secure app access
needs to be explicitly enabled if two-factor authentication is not in
place, otherwise send-email fails with:
5.7.8 Username and Password not accepted. Learn more at
5.7.8 https://support.google.com/mail/?p=BadCredentials
Document steps required to make this work.
Original-patch-by: Vasyl Vavrychuk <vvavrychuk@gmail.com>
Signed-off-by: Denton Liu <liu.denton@gmail.com>
---
Range-diff against v2:
1: 2bcfd1403f < -: ---------- Documentation/git-send-email.txt: Mention less secure app access might need to enable.
-: ---------- > 1: 274475559a git-send-email.txt: mention less secure app access with Gmail
Documentation/git-send-email.txt | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 0a69810147..0db0783f2f 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -494,10 +494,14 @@ edit ~/.gitconfig to specify your account settings:
smtpServerPort = 587
----
-If you have multifactor authentication setup on your gmail account, you will
+If you have multi-factor authentication set up on your Gmail account, you will
need to generate an app-specific password for use with 'git send-email'. Visit
https://security.google.com/settings/security/apppasswords to create it.
+If you do not have multi-factor authentication set up on your Gmail account,
+you will need to allow less secure app access. Visit
+https://myaccount.google.com/lesssecureapps to enable it.
+
Once your commits are ready to be sent to the mailing list, run the
following commands:
--
2.30.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-01-07 20:38 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-17 12:54 [PATCH] Documentation/git-send-email.txt: Mention less secure app access might need to enable Vasyl Vavrychuk
2020-08-17 17:16 ` Eric Sunshine
2020-08-29 15:39 ` Vasyl Vavrychuk
2020-08-29 18:33 ` Junio C Hamano
2020-08-29 19:29 ` Junio C Hamano
2020-09-06 21:52 ` Junio C Hamano
2020-12-22 19:58 ` Junio C Hamano
2021-01-07 11:11 ` [PATCH v3] git-send-email.txt: mention less secure app access with Gmail Denton Liu
2021-01-07 20:37 ` Eric Sunshine
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).