Closed Bug 1694266 Opened 5 years ago Closed 3 years ago

Intermittent comm/mail/test/browser/newmailaccount/browser_newmailaccount.js | Uncaught exception - [Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIURI.host]"]

Categories

(Thunderbird :: Account Manager, defect, P5)

defect

Tracking

(thunderbird_esr102 unaffected)

RESOLVED FIXED
106 Branch
Tracking Status
thunderbird_esr102 --- unaffected

People

(Reporter: intermittent-bug-filer, Assigned: darktrojan)

References

(Depends on 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

I must say that this bug may have the potential to undermine trust in Thunderbird somewhat. On first account setup, right after entering just your email address, pressing Tab into password field and then delaying a bit (feed the cat, walk the dog, you name it), Thunderbird goes in a loop of throwing this error until error console is painted all red :-/

Why would it start doing things with just the email address and no password entered before I choose "Next"?

STR

91.0.1 (64-Bit), DE, Win10

  • Install TB, first account setup (not sure if it happens for every account setup)
  • Enter Name and Email address (gmail)
  • Press Tab into password field, but do not enter a password
  • Move focus elsewhere, and refocus password input

Actual

  • Thunderbird throws above error instantly and again every time you focus the password input.
  • Error console turns red after just entering my email and focusing password input

Expected

  • As a confidence building measure, let's not throw at this stage. We can always toss errors later if we deem that necessary...

Code associated with the error

This is about "LoginAutoComplete", so iiuc, we're allowing password manager to interfere here (as when you fill credentials online and FF will try to save or fill your password). Which makes no sense at all and should be prevented because typically we will not have the password for an account which the user is just adding...

   getLoginOrigin(uriString, allowJS = false) {`
[snip]
      log.warn("Couldn't parse origin for", uriString, e);

Comment on attachment 9237097 [details]
Screenshot 1: Error console painted red after just entering an email address in Account Setup and typing password...

.

Attachment #9237097 - Attachment description: Screenshot 1: Error console painted red after just entering an email address in Account Setup and waiting a bit... → Screenshot 1: Error console painted red after just entering an email address in Account Setup and focusing password input a couple of times...

Can we prevent LoginAutoComplete.jsm / password manager from acting on password input when it receives focus?

STR / code starting points in my comment 10.

This is about "LoginAutoComplete", so iiuc, we're allowing password manager to interfere here (as when you fill credentials online and FF will try to fill your password). Which makes no sense at all and should be prevented.

As bug 1724302 mentions, we're saving the password directly.
Which URI are we checking for?

This is a bit out of our control.
We asked for support tot he m-c folks in bug 1724302.

Depends on: 1724302

Comment on attachment 9237097 [details]
Screenshot 1: Error console painted red after just entering an email address in Account Setup and typing password...

Turns out the error is re-generated when focusing the password input, and for every single password character which you type.

Attachment #9237097 - Attachment description: Screenshot 1: Error console painted red after just entering an email address in Account Setup and focusing password input a couple of times... → Screenshot 1: Error console painted red after just entering an email address in Account Setup and typing password...

Thomas, this is a bug about an intermittent failure in the CI tests. Why are you hijacking it?

See Also: → 1700487

I think the exception occurs when .host is called on a URI pointing to about:blank. About URIs don't have a host and they throw this exception.

Assignee: nobody → geoff
Status: NEW → ASSIGNED
Target Milestone: --- → 106 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/1f3973f22a4d
Avoid exception in browser_newmailaccount.js. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: