Closed Bug 107953 Opened 23 years ago Closed 21 years ago

when incoming server and outgoing server are the same the username for the SMTP server should be the same a the username for the incoming server.

Categories

(SeaMonkey :: MailNews: Account Configuration, defect)

Product:
SeaMonkey
Component:
MailNews: Account Configuration
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla1.4beta

People

(Reporter: bugzilla, Assigned: iannbugzilla)

References

Details

Attachments

(1 file, 2 obsolete files)

updated patch
926 bytes, patch
Details | Diff | Splinter Review 
I always get into this problem:

I create an IMAP server account that has the following info:
incoming server mail.bla.net
outgoing server mail.bla.net
mailaddress = spammer@bla.net
username = me

then a IMAP server is created with the correct username BUT the username for my 
SMTP server is set to spammer! The correct username is me

The username seems to be taken from the mailadress. The is ok, but not when the 
incoming and outgoing server are the same!
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.0.1
The same happens with the first (off all accounts) POP server account you make.
*** Bug 139828 has been marked as a duplicate of this bug. ***
Taking as I have a fix
Assignee: racham → ian
Status: ASSIGNED → NEW
OS: Windows 2000 → All
Hardware: PC → All
Target Milestone: mozilla1.0.1 → mozilla1.4beta
Accepting bug
Status: NEW → ASSIGNED
Attached patch Patch v1.0 (obsolete) — — Splinter Review 
Adds some simple checking to AccountWizard.js
Attached patch Patch v1.0a (obsolete) — — Splinter Review 
Corrects slight typo
Attachment #119916 - Attachment is obsolete: true
Comment on attachment 119916 [details] [diff] [review]
Patch v1.0

Not obsolete afterall, it's LXR playing up
Attachment #119916 - Attachment is obsolete: false
Comment on attachment 119918 [details] [diff] [review]
Patch v1.0a

There wasn't a typo, so this is obsolete
Attachment #119918 - Attachment is obsolete: true
*** Bug 182754 has been marked as a duplicate of this bug. ***
Attachment #119916 - Flags: review?(racham)
Adding alecf to cc seeing it's his code I'm tweaking. Any comments?
Adding sspitzer to the loop.
Any comments? Can someone review this patch?
Comment on attachment 119916 [details] [diff] [review]
Patch v1.0

r=dmose@mozilla.org
Attachment #119916 - Flags: review?(racham) → review+
Attachment #119916 - Flags: superreview?(sspitzer)
Comment on attachment 119916 [details] [diff] [review]
Patch v1.0

I attached what I checked in.
Attachment #119916 - Flags: superreview?(sspitzer)
fixed.  thanks ian.

I wonder using the username (if we have it) would be better than the email
username, even if the hostname doesn't match?

maybe our account wizard should have a username field for smtp?

I'll look to see what OE does.

either way, these issues could be spun off to new bugs.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Blocks: 201469
Well from testing the patch I initially didn't have the hostname comparison
check but on balance decided it was best to have it whilst there wasn't a
separate field for smtp server's username. Spinning this off to bug 201469

v
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: