Closed Bug 130986 Opened 24 years ago Closed 24 years ago

when logon redirection fails, we shouldn't logout of the connection

Categories

(MailNews Core :: Networking: IMAP, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

Attachments

(1 file)

When an external logon redirection fails, we shouldn't log out of the connection, since we might have been previously logged on in another part of the client. Patch upcoming from mscott, r=bienvenu, sr=mscott.
Attached patch proposed fixSplinter Review
the problem is that when we have a logon redirector (a generic logon extension mechanism, shocking only used by AOL webmail servers :-) ), we should not logoff from the session when we get a redirection error from the logon server. Although this change is in the mozilla tree, it only affects AOL-branded webmail since they're the only ones using the logon redirector service. The reason behind this is that different components can have different levels of security, but use the same connection. The challenge for the higher security can fail, but that doesn't mean we should log off the component using the lower security connection. If we didn't log on, there's no need to log off.
Comment on attachment 74139 [details] [diff] [review] proposed fix a=asa (on behalf of drivers) for checkin to the 1.0 trunk patch from http://bugscape.netscape.com/show_bug.cgi?id=3688 written by mscott, r=beinvenu, sr=sspitzer
Attachment #74139 - Flags: approval+
fix checked in (fix by mscott, r=me, sr=sspitzer, in bugscape bug this was spun off from)
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
QA Contact: huang → gchan
verified, see tests case for teseting this with specific type mail account in bugscape bug.
Status: RESOLVED → VERIFIED
QA Contact: gchan → esther
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: