[autoconfig] Wrong error msg: Says "Username or password invalid", although it could be any kind of error (verifyConfig)

NEW
Unassigned

Status

P1
major
9 years ago
2 years ago

People

(Reporter: BenB, Unassigned)

Tracking

Bug Flags:
wanted-thunderbird +

Firefox Tracking Flags

(blocking-thunderbird3.1 -)

Details

(Whiteboard: [GS], URL)

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

9 years ago
Reproduction:
1. File | New | Mail account...
2. Enter foo@gmx.com and continue
3. Disconnect network, or any other error
4. Click on "Create account"

Actual result:
Dialog says "username and/or password invalid"

Expected result:
The actual error is is shown, whatever was the cause for the failed verification.
There are thousands of reasons why the check may have failed: network down, server down, server malfunctioning, our network code malfunctioning, the dialog code having some exception (just add "throw 'foo';" somewhere in verifyConfig.js).
"username and/or password invalid" MUST NOT be shown unless we are certain that it was the username or password, or at least that the server refused the authentication. If the server gave an error message (even as response to authentication), that message must be shown. Sometimes, authentication fails simply because only one POP check within 15 minutes is allowed, and the server then says that as error message in response to the login.

Importance:
Misleading error messages which tell the *wrong* cause are worse than none. They make users try to fix the wrong thing (here: try other username permutations or hunt for the right password, maybe it was changed?) and therefore cause severe problems.
(Reporter)

Updated

9 years ago
Priority: -- → P2
(Reporter)

Updated

9 years ago
blocking-thunderbird3.1: --- → ?
(Reporter)

Comment 1

9 years ago
Note: this may well need new strings.
(Reporter)

Updated

9 years ago
Summary: Account creation wizard says "username and/or password invalid", although it could be any kind of error → Account creation wizard says "Username or password invalid", although it could be any kind of error
This would be great to get, but we wouldn't block on it unless we had reason to believe that it was being experienced by an extremely significant percentage of our users, and they weren't able to recover from it.  Adding folks who might know whether that's true, they're welcome to renominate.
blocking-thunderbird3.1: ? → -
Flags: wanted-thunderbird+
(Reporter)

Updated

9 years ago
Priority: P2 → P1
(Reporter)

Comment 3

9 years ago
Created attachment 440029 [details] [diff] [review]
Add error strings only, v1

This patch adds only the needed error messages to the locale, before the string freeze.

I think this bug is serious (see initial description), so I'd like to fix it in 3.1, if I can. If we don't add the strings now, I can't fix it later anymore, though. Just adding the strings shouldn't do much harm, just a tiny little bit more work for our translators.

Please review before the beta2 deadline (with enough time for me to still check it in)!
Attachment #440029 - Flags: ui-review?(clarkbw)
Attachment #440029 - Flags: review?(bwinton)
(Reporter)

Updated

9 years ago
Status: NEW → ASSIGNED
Summary: Account creation wizard says "Username or password invalid", although it could be any kind of error → [Account creation] Says "Username or password invalid", although it could be any kind of error (verifyConfig)
(Reporter)

Comment 4

9 years ago
Created attachment 440032 [details] [diff] [review]
Add error strings only, v2

Add "unexpectedly"
Attachment #440029 - Attachment is obsolete: true
Attachment #440032 - Flags: ui-review?(clarkbw)
Attachment #440032 - Flags: review?(bwinton)
Attachment #440029 - Flags: ui-review?(clarkbw)
Attachment #440029 - Flags: review?(bwinton)
Comment on attachment 440032 [details] [diff] [review]
Add error strings only, v2

>+++ b/mail/locales/en-US/chrome/messenger/accountCreationUtil.properties
>@@ -13,8 +13,14 @@ boolean.error=Not a boolean
>+#verifyConfig.js
>+auth_failed_generic.error=Login failed. Are username/email address and password correct?
>+auth_failed_with_reason.error=Login failed. The server said: %S

Needs a translation note.

>+verification_failed.error=Login verification failed for an unknown reason.
>+verification_failed_with_exception.error=Login verification failed unexpectedly: %S

Needs a translation note.  Also, I'ld like that to read a little better, like "Login verification failed unexpectedly with message: %S"

Other than those two, r=me.
Attachment #440032 - Flags: review?(bwinton) → review+
(Reporter)

Comment 6

9 years ago
Created attachment 440097 [details] [diff] [review]
Add error strings only, v3
Attachment #440097 - Flags: ui-review?(clarkbw)
Attachment #440097 - Flags: review+
(Reporter)

Updated

9 years ago
Attachment #440032 - Attachment is obsolete: true
Attachment #440032 - Flags: ui-review?(clarkbw)
(In reply to comment #3)
> If we don't add the strings now, I can't fix it later anymore,
> though. Just adding the strings shouldn't do much harm, just a tiny little bit
> more work for our translators.
> 
> Please review before the beta2 deadline (with enough time for me to still check
> it in)!

Localisers hate this type of string landing, and it is actively discouraged and we've been complained at before for doing this. Generally, I believe, it is because they are unable to test the strings although they have to translate them.

I'm cc'ing the l10n guys here. The only option may be to land it with a couple of l10n postings. This isn't a blocker, but very much wanted.
(Reporter)

Comment 8

9 years ago
> Generally, I believe, it is because they are unable to test the strings

These are just error strings. Generally, they couldn't even test with a patch. Same in other bugs.
We can do this, but *only* if we give localisers enough backgroud information. 
That would include:

- a much more descriptive localization note. Basically a short form of the 
  original bug issue together with a short explanation on how to test and 
  if testing isn't possible, why it is not.
- A post to mozilla.dev.l10n with a short heads-up for localizers when you 
  check this in.
(Reporter)

Comment 10

9 years ago
Sure, I can add descriptions, no problem. Thanks.
Comment on attachment 440097 [details] [diff] [review]
Add error strings only, v3

I don't usually like using the word "server" in messages to the user but I'm not sure what else to replace it with.  If we had the host name that would be a lot better.  Is that possible?

In general it looks fine, so I'll give a ui-r+ but will look for comments on that.
Attachment #440097 - Flags: ui-review?(clarkbw) → ui-review+
(Reporter)

Comment 12

9 years ago
Yes, I think I can add the hostname. I'd still prefer to keep the word "server" in there. I modeled it after these msgs:
http://mxr.mozilla.org/comm-central/source/mail/locales/en-US/chrome/messenger/messengercompose/composeMsgs.properties#78

An error occurred while sending mail. The mail server responded:  %s

(and similar msgs in IMAP, POP3 etc.) The msgs here are very similar.
(Reporter)

Comment 13

9 years ago
Created attachment 440307 [details] [diff] [review]
Add error strings only, v4 - commited

Here's an example of all 4 errors:
Login failed. Are username/email address and password correct?
Login failed. The server imap.web.de said: Only one login per 15 minutes
Login verification failed for an unknown reason.
Login verification failed with message: TCP timeout
Attachment #440097 - Attachment is obsolete: true
Attachment #440307 - Flags: ui-review+
Attachment #440307 - Flags: review+
Attachment #440307 - Flags: feedback?(bugzilla)
(Reporter)

Comment 14

9 years ago
Commited as <http://hg.mozilla.org/comm-central/rev/b9de88d28d49>
(with one duplicated line in v4 removed)
(Reporter)

Updated

9 years ago
Attachment #440307 - Attachment description: Add error strings only, v4 → Add error strings only, v4 - commited
Comment on attachment 440307 [details] [diff] [review]
Add error strings only, v4 - commited

Looks great (with the duplicate removed). Thanks for your work.
Attachment #440307 - Flags: feedback?(bugzilla) → feedback+

Comment 16

8 years ago
Related bugs:
Bug 224032
Bug 435306
(Reporter)

Comment 17

8 years ago
Not related, this bug is in the new account creation wizard.
(Reporter)

Updated

7 years ago
Summary: [Account creation] Says "Username or password invalid", although it could be any kind of error (verifyConfig) → [autoconfig] Wrong error msg: Says "Username or password invalid", although it could be any kind of error (verifyConfig)
(Reporter)

Comment 18

7 years ago
I just noticed that we published a bad config for tiscali.it: IMAP, port 143, normal SSL. This will surely go wrong. The user sees a message: "Username/password invalid!" next to the password. Very bad! :-((( This is sure to horribly confuse users and waste tons of their time, we must fix this.
Severity: normal → major
(Reporter)

Comment 19

7 years ago
(If anybody wants to take and fix this bug, be my guest. I likely won't have time in the near future.)
Hi gents,

The work here seems great, it is too bad if it cannot be finished...

We have a lot of user reporting this kind of error on GSFN, see https://getsatisfaction.com/mozilla_messaging/tags/username_or_password_wrong

However, it doesn't only happened during autoconfiguration. Should we open a new bug for this kind of problems, which certainly need a similar fix?
(Reporter)

Comment 21

7 years ago
I agree this is very important, but I won't get to it in the mid-term future.

bienvenu, could you please fix this?
Assignee: ben.bucksch → dbienvenu

Updated

4 years ago
Assignee: mozilla → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.