71121 - news-identity defaults to <nospam@nospam>

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[Ben Bucksch (:BenB)]

Reproduce:
1. Click on an NNTP links, e.g. <news://news.mozilla.org/netscape.test>. Account
Wizard comes up.
2. Click Cancel. The server and newsgroup will appear nevertheless in the folder
pane.
3. Check Identity in the Account Manager.

Actual result:
Name empty, email address <nospam@nospam>.

Expected results:
- No account added at all or
- Account disabled for posting.

Additional Comments:
Comm-on, don't encourage invalid email addresses! GNKSA-blocker.

*If* you really need to use a dummy address, anywhere, use <[e-mail]>
(e.g. this case) or <[e-mail]> (e.g. in docs). See RFC2606.

Comment 1

[Ben Bucksch (:BenB)]

Code is at
<http://lxr.mozilla.org/seamonkey/source/mailnews/base/prefs/resources/content/AccountWizard.js#161>.

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Patch

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Cool, so I read the RFC and .invalid is the right TLD to use for clear cases
like this one, it seems.  Thanks, Ben.

Comment 4

[Ben Bucksch (:BenB)]

r=BenB

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

Seth, can I get an sr= on this puppy?  Thanks.

Comment 6

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

sr=sspizer

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

Fix checked in.  Thanks to all for speedy reviews.

Comment 8

[[e-mail] (gone - use [e-mail] instead)]

Fix checked in.  Thanks to all for speedy reviews.

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

The easiest way to get this working: click on the link, wait for the newsgroup
to populate, hit new msg, then cancel the account wizard, then just verify that
the [e-mail] is in the from widget.

Comment 10

[Ninoschka Baca]

Build 2001-04-02-04: NT4
Build 2001-04-02-08: Linux RH 6.2, Mac 9.04
Verified Fixed. The From: entry in a new message compose window now states 
[e-mail] after cancelling out of the Wizard.