124170 - Trim leading and trailing spaces from server names in account manager

Status: RESOLVED WORKSFORME

(MailNews Core :: Backend, defect)

Description

[staubli.net informatics]

==PROBLEM:==
if der is a space " " in front of the outgoing mailservername (SMTP), mozilla is
not able to connect to the server.

(this happened cause i copy pasted the adress of my mailserver and didn't
recognize that there is a space in front of the entry...)

==SOLUTION:==
remove unused spaces (trim) in front and end of the mailserver entry.

==P.S.==
don't know if problem also exists with incoming mailserver


mfg
stefan staubli

Comment 1

[R.K.Aa.]

Bug 99133 resembles, but is more relevant since it is harder to spot a trailing
space than a leading one. Added comment about this bug there.

Comment 2

[Ninoschka Baca]

Trunk build 2002-02-13-03: Winme
Reproduced the problem.

Comment 3

[(not reading, please use [e-mail] instead)]

mass re-assign.

Comment 5

[Phil Ringnalda (:philor)]

Fixed by bug 99133 for the place where I would expect to run into trouble, the new account wizard, and by something else for changing an existing SMTP server, but not for the incoming server in the account manager for an existing account.

STR:
1. Create a working mail account.
2. Open account manager, server settings pane.
3. Add a space before and after incoming server name.
4. Save settings, try to get mail.

Comment 6

[Magnus Melin [:mkmelin]]

Moving to Core - as Phil pointed out in the dupe, the (shared code) is around here: http://mxr.mozilla.org/mozilla/source/mailnews/base/prefs/resources/content/AccountManager.js#340

Comment 7

[:Hb]

The SMTP host name is now checked for spaces before saving. Space characters lead to the error message "Please enter a valid hostname."

Comment 8

[Magnus Melin [:mkmelin]]

->WFM (FIXED is only used when known code changes resolved the issue)