375880 - Append default domain to user name in ISP RDF File ignores SMTP setting

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Chris Conklin]

User-Agent:       Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.04506.30; InfoPath.1)
Build Identifier: version 2.0.0.0pre (20070329)

related to bug 366439 - if IncomingServer hostName and SMTP hostname are the same, smtpUserNameRequiresDomain setting is ignored.  Instead the value for incomingServerUserNameRequiresDomain sets the authentication string for both incoming server and smtp server. 
If 'incoming' set to false and 'smtp' set to true, the account wizard summary screen correctly shows "Outgoing User Name: [e-mail]" but the effective setting is "user" - i.e. your prefs.js file has:
user_pref("mail.smtpserver.smtp1.username", "user");


Reproducible: Always

Steps to Reproduce:
1.
2.
3.



My ISP RDF file does contain a domain <NC:nsIMsgAccount about="domain:foo.edu">

As long as your incoming mail server and SMTP host names differ, the settings incomingServerUserNameRequiresDomain and smtpUserNameRequiresDomain work independently.  In my situation, mail.foo.edu is the hostname for IMAP, POP3 and SMTP.  POP3 authenticates as "user" SMTP authenticates as "[e-mail]" (luckily IMAP will take either string).  ISP RDF file available on request.

Comment 1

[Chris Conklin]

examining AccountWizard.js this behavior is expected (if not by design).  See bug 107953.  AccountWizard.js commented lines 974-976: "if incoming hostname is same is smtp hostname use the server username (instead of the email username)"

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Reporter, does the issue still occur in the latest supported 2.0.0.x / Shredder trunk nightlies?

(1.5.0.x is end-of-life and the latest supported 2.0.0.x is 2.0.0.17)

Comment 3

[Magnus Melin [:mkmelin]]

Yes.

Comment 4

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

Comment 5

[[e-mail]]

Comment on attachment 343929 [details] [diff] [review]
proposed fix

Looks reasonable.

Comment 6

[Magnus Melin [:mkmelin]]

changeset:   647:2de8eaab9cfb
http://hg.mozilla.org/comm-central/rev/2de8eaab9cfb

->FIXED