Closed
Bug 1138172
Opened 10 years ago
Closed 10 years ago
Three strings about SMTP errors tell user to change connection/password/authentication method in Account settings > Server settings, but should be in Outgoing server (SMTP)
Categories
(MailNews Core :: Composition, defect)
MailNews Core
Composition
Tracking
(seamonkey2.36 fixed)
RESOLVED
FIXED
Thunderbird 39.0
Tracking | Status | |
---|---|---|
seamonkey2.36 | --- | fixed |
People
(Reporter: aryx, Assigned: aceman)
References
Details
Attachments
(2 files)
8.11 KB,
patch
|
neil
:
review+
mkmelin
:
review+
|
Details | Diff | Splinter Review |
5.69 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Three strings about SMTP errors tell user to change connection/password/authentication method in Account settings > Server settings, but should be in Outgoing server (SMTP)
smtpAuthChangeEncryptToPlainNoSsl=The Outgoing server (SMTP) %S does not seem to support encrypted passwords. If you just set up the account, try changing the 'Authentication method' in 'Account Settings | Server settings' to 'Password, transmitted insecurely'. If it used to work but now doesn't, you may be susceptible to getting your password stolen.
smtpAuthChangeEncryptToPlainSsl=The Outgoing server (SMTP) %S does not seem to support encrypted passwords. If you just set up the account, try changing the 'Authentication method' in 'Account settings | Server settings' to 'Normal password'.
smtpAuthChangePlainToEncrypt=The Outgoing server (SMTP) %S does not allow plaintext passwords. Please try changing the 'Authentication method' in 'Account Settings | Server settings' to 'Encrypted password'.
Comment 1•10 years ago
|
||
And I called it out in bug 952493 comment #45 too...
Well, considering the comment:
>>+# LOCALIZATION NOTE (smtpAuthChangeEncryptToPlainSsl): %S is the server hostname
>>+smtpAuthChangeEncryptToPlainSsl=The SMTP server %S does not seem to support encrypted passwords. If you just >+# LOCALIZATION NOTE (smtpAuthChangePlainToEncrypt): %S is the server hostname
>>+smtpAuthChangePlainToEncrypt=The SMTP server %S does not allow plaintext passwords. Please try changing the 'Authentication method' in 'Account Settings | Server settings' to 'Encrypted password'.
>>+# LOCALIZATION NOTE (smtpAuthFailure): %S is the server hostname
>>+smtpAuthFailure=Unable to authenticate to SMTP server %S. Please check the password, and verify the 'Authentication method' in 'Account Settings | Outgoing server (SMTP)'.
>Huh, shouldn't these all say Outgoing server (SMTP)?
You see Suyash changed the string, but he changed the leading "SMTP server" to "Outgoing server (SMTP)", going by the "SMTP" keyword.
The comment was probably too short or ambiguous to understand you mean to rewrite the whole path to Account manager pane instead :)
Now that seems to be clear, I'll do it.
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Attachment #8571529 -
Flags: review?(neil)
Attachment #8571529 -
Flags: review?(mkmelin+mozilla)
Comment 4•10 years ago
|
||
(In reply to aceman from comment #2)
> The comment was probably too short or ambiguous to understand you mean to
> rewrite the whole path to Account manager pane instead :)
Oops. Sorry, Suyash.
Comment 5•10 years ago
|
||
Bah, these have reached Aurora and 11 localisers have already translated them.
Comment 6•10 years ago
|
||
Comment on attachment 8571529 [details] [diff] [review]
patch [checked in]
(Huh, when did Thunderbird drop the "please" from their two strings?)
Attachment #8571529 -
Flags: review?(neil) → review+
Comment 7•10 years ago
|
||
Is this something that we will need for TB 38?
tracking-thunderbird38:
--- → +
If you r+ it for /mail it could go into TB38. But it is no regression (the bug was in the strings even before Suyash touched them recently), just a bug fix. So you can decide.
Flags: needinfo?(rkent)
Comment 9•10 years ago
|
||
If this was not a regression, then we should not try to push it into TB 38.
tracking-thunderbird38:
+ → ---
Flags: needinfo?(rkent)
Comment 10•10 years ago
|
||
Comment on attachment 8571529 [details] [diff] [review]
patch [checked in]
Review of attachment 8571529 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM!
Attachment #8571529 -
Flags: review?(mkmelin+mozilla) → review+
![]() |
||
Comment 12•10 years ago
|
||
Comment on attachment 8571529 [details] [diff] [review]
patch [checked in]
a=me for SeaMonkey CLOSED TREE
![]() |
||
Updated•10 years ago
|
status-seamonkey2.34:
--- → unaffected
status-seamonkey2.35:
--- → affected
status-seamonkey2.36:
--- → affected
![]() |
||
Comment 13•10 years ago
|
||
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/30f695264e00
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
status-seamonkey2.34:
unaffected → ---
status-seamonkey2.35:
affected → ---
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 39.0
Comment 14•10 years ago
|
||
Nit: the asociated loc notes need to be adjusted as well, the entity names do not match with actual entities now.
![]() |
Assignee | |
Comment 15•10 years ago
|
||
Argh, thanks for noticing.
Attachment #8574859 -
Flags: review?(mkmelin+mozilla)
Updated•10 years ago
|
Attachment #8574859 -
Flags: review?(mkmelin+mozilla) → review+
Attachment #8571529 -
Attachment description: patch → patch [checked in]
Keywords: checkin-needed
Whiteboard: [check in only the remaining patch]
![]() |
||
Comment 16•10 years ago
|
||
Comment on attachment 8574859 [details] [diff] [review]
patch fix notes
a=me for SeaMonkey CLOSED TREE
![]() |
Reporter | |
Comment 17•10 years ago
|
||
Pushed localization comment fixes as https://hg.mozilla.org/comm-central/rev/ad7600eae938
Keywords: checkin-needed
Whiteboard: [check in only the remaining patch]
You need to log in
before you can comment on or make changes to this bug.
Description
•