Add 'Existing Email Account' does not have OAuth2 when configuring an account manually
Categories
(Thunderbird :: Account Manager, defect)
Tracking
(Not tracked)
People
(Reporter: raysatiro, Assigned: david.ward)
References
()
Details
(Whiteboard: [patchlove])
Attachments
(9 files, 2 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
BenB
:
review+
|
Details |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details |
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:76.0) Gecko/20100101 Firefox/76.0
Steps to reproduce:
I added a gmail account in Thunderbird 68.4.2 (32-bit) several days ago using the 'Existing Email Account' UI and OAuth2 was not listed as an option.
Actual results:
See above.
Expected results:
OAuth2 should be listed as an option when adding an existing e-mail account.
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment hidden (obsolete) |
Comment 5•5 years ago
|
||
I can confirm that OAuth2 isn't a selection when manually configuring a gmail address using 68.7.0 on Ubuntu 18.04.4 LTS Linux, or 77.0b3 or 78.0a1.
Assignee | ||
Comment 6•4 years ago
|
||
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D98009
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D98010
Assignee | ||
Comment 9•4 years ago
|
||
Depends on D98011
Assignee | ||
Comment 10•4 years ago
|
||
The SMTP server may have been configured using dynamic OAuth2 settings for an
otherwise unknown hostname. For now, simply make the behavior consistent with
editing incoming server settings, where the OAuth2 menu item is never hidden.
Depends on D98012
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 11•4 years ago
|
||
Comment on attachment 9189858 [details]
Bug 1622181 - Part 1: Test correctly for OAuth2 settings during backend account creation.
This code change is wrong. If the user selects another auth method, or autoconfig found another auth method, that choice would be completely ignored and we'd use OAuth2 contrary to choices that were made.
createInBackend()
doesn't implement policy. Rather, it saves what was specified elsewhere. This would be like a spelling correction after you press Save in Word.
Assignee | ||
Comment 12•4 years ago
|
||
(In reply to Ben Bucksch (:BenB) from comment #11)
Comment on attachment 9189858 [details]
Bug 1622181 - Part 2: Test correctly for OAuth2 settings during backend account creation.This code change is wrong. If the user selects another auth method, or autoconfig found another auth method, that choice would be completely ignored and we'd use OAuth2 contrary to choices that were made.
Please take a closer look. This change has no effect on the auth method that is selected, now or later.
This prevents a crash from occurring when trying to store preferences from values that haven't been set (and which shouldn't be set if OAuth2 isn't selected). These preferences are only used if OAuth2 is the selected auth method, and otherwise they are ignored.
Comment 13•4 years ago
|
||
auth method
(Answered in Phab.)
This prevents a crash
Can you fix the crash instead, please?
Assignee | ||
Comment 14•4 years ago
|
||
(In reply to Ben Bucksch (:BenB) from comment #13)
This prevents a crash
Can you fix the crash instead, please?
Please see the commit description. This is the only place to avoid it.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 15•4 years ago
|
||
Comment on attachment 9189857 [details]
Bug 1622181 - Part 1: Improve logging in verifyConfig() and verifyLogin().
Revision D98009 was moved to bug 1681002. Setting attachment 9189857 [details] to obsolete.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 16•4 years ago
|
||
Can you fix the crash instead, please?
For the record: David wasn't talking about a crash, but a JS exception.
This is the only place to avoid it.
Actually, it isn't. This is an invalid state that was introduced by your other patch D98011. That means the other patch D98011 was buggy. See https://phabricator.services.mozilla.com/D98010 .
Comment 18•3 years ago
|
||
Increasing support requests about this as well.
Do we need a SUMO kb article?
Comment 20•3 years ago
|
||
This is WFM, oauth is offered immediately after switching from auto to manual config - I tested both 91.8.0 and nightly.
Do you agree? Because of bug 1757713?
Comment 21•3 years ago
|
||
(In reply to Wayne Mery (:wsmwk) from comment #20)
Created attachment 9270618 [details]
gmail oauth nightly manual configThis is WFM, oauth is offered immediately after switching from auto to manual config - I tested both 91.8.0 and nightly.
Do you agree? Because of bug 1757713?
I tried today to configure Gmail account in other domain than gmail.com domain name in Thunderbird 91.8.0 and there was no option OAuth2 in manual settings account configuration to choose from.
Comment 22•3 years ago
|
||
Based on the original screenshot, it looks like the reporter attempted an invalid configuration. (Note that the hostnames are ".gmail.com", not "imap.gmail.com" and "smtp.gmail.com".) Based on that, I'd be tempted to mark this as invalid, but it's unclear whether the patches listed here are intended to address this bug by creating a more robust/less provider-specific OAuth2 implementation. That may be worth breaking into a separate ticket.
Reporter | ||
Comment 23•3 years ago
|
||
Based on the original screenshot, it looks like the reporter attempted an invalid configuration.
OAuth2 still isn't an option for me in 91.9.0. Try manually configuring a new account test@gmail.com. A screenshot of what I see when I do that is attached.
Comment 24•3 years ago
|
||
This is what the settings should be and are when I attempt a manual configuration.
You need the full hostname and port numbers.
Reporter | ||
Comment 25•3 years ago
|
||
You need the full hostname and port numbers.
You mean the port number and subdomain are automatically filled in for you? They aren't for me. Even if I add them, it makes no difference:
Reporter | ||
Comment 26•3 years ago
|
||
Oops I captured the wrong dropdown in the previous reply. This reply should have a screenshot of the right dropdown.
Comment 27•3 years ago
|
||
(In reply to Ray Satiro from comment #25)
Created attachment 9275619 [details]
Thunderbird 91.9.0 missing OAuth2 in new account - second tryYou need the full hostname and port numbers.
You mean the port number and subdomain are automatically filled in for you? They aren't for me. Even if I add them, it makes no difference:
The whole account is set up for me if I use the account setup wizard, enter my full name and email address, then click Continue
.
I tested with a fresh profile using 91.9.0, and the Account Setup wizard tab opens in the window.
- Entered my full name
- Entered my email address and left the password field empty
- After entering the email address, I clicked
Configure manually
- I did get the result that you posted in comment 23
- Edited the Hostname fields
- Entered the correct Port numbers, and Connection security settings.
- The Authentication method list did not show OAuth2 as a choice
- Clicked the `Re-test' button and "Normal Password" appeared as the Authentication method
- Checked the drop-down list and "OAuth2" was there for each server
- Selected it and finished creating the account
Reporter | ||
Comment 28•3 years ago
|
||
Ok thanks for the detail. IMO it's a bug that OAuth2 does not initially show as a choice. I wouldn't have thought to do that.
Comment 29•3 years ago
|
||
(In reply to Ray Satiro from comment #28)
Ok thanks for the detail. IMO it's a bug that OAuth2 does not initially show as a choice. I wouldn't have thought to do that.
Agree it's a bug, but I would use the Account Setup tab and not use Manual configuration for an existing account, unless it wasn't auto discovered.
Comment 30•3 years ago
|
||
What WaltS said.
Comment 31•2 years ago
|
||
Assignee seems to be gone.
So where do we go from here?
Comment 32•2 years ago
|
||
The original problem was fixed in bug 1779518. Looks like the problem remains for editing SMTP server (Part 4)
Description
•