Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated mailer docs with option for google password auth , issue #4510 #5427

Merged
merged 2 commits into from
Nov 14, 2019

Conversation

akochnev
Copy link
Contributor

By default, the vertx mail client and google both default to XOAUTH2 and that method needs to be disabled in order for password authentication (with app passwords to work)

Addresses issue #4510

Otherwise, gmail fails with the dreaded:

AUTH XOAUTH2 failed 535-5.7.8 Username and Password not accepted. Learn more at
535 5.7.8 https://support.google.com/mail/?p=BadCredentials n5sm9432005qke.74 - gsmtp

@cescoffier
Copy link
Member

Thanks LGTM!

I've restarted the CI that broke because of a timeout.

@cescoffier cescoffier added area/documentation area/vertx triage/waiting-for-ci Ready to merge when CI successfully finishes labels Nov 13, 2019
akochnev and others added 2 commits November 14, 2019 16:54
…kusio#4510

By default, the vertx mail client and google both default to XOAUTH2 and that method needs to be disabled in order for password authentication (with app passwords to work)

Addresses issue quarkusio#4510
@gsmet
Copy link
Member

gsmet commented Nov 14, 2019

I rebased and added an additional commit.

@machi1990
Copy link
Member

Ci green, let's merge. Thanks @akochnev @gsmet @cescoffier

@machi1990 machi1990 merged commit 55847c0 into quarkusio:master Nov 14, 2019
@gsmet gsmet added this to the 1.1.0 milestone Nov 15, 2019
@gsmet gsmet removed the backport? label Nov 15, 2019
@gsmet gsmet modified the milestones: 1.1.0, 1.0.0.Final Nov 15, 2019
@akochnev akochnev deleted the mailer-doc-fix branch November 17, 2019 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation area/vertx triage/waiting-for-ci Ready to merge when CI successfully finishes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants