Closed Bug 218766 Opened 21 years ago Closed 21 years ago

SASL mechanism PLAIN not supported

Categories

(MailNews Core :: Networking: POP, defect)

Product:
MailNews Core
Component:
Networking: POP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: ch.ey, Assigned: ch.ey)

Details

Attachments

(1 file)

suggested patch
3.58 KB, patch
Bienvenu
: review+
mscott
: superreview+
Details | Diff | Splinter Review 
In contrast to SMTP, we don't support the SASL mechanism PLAIN in POP3.
Attached patch suggested patch — — Splinter Review 
This patch adds support for SASL/PLAIN to our POP3 protocol. The code is
basically the same like in nsSmtpProtocol, so very little risk.

This is a -puw patch, -pu for checkin on request.
Attachment #131137 - Flags: review?(bienvenu)
Comment on attachment 131137 [details] [diff] [review]
suggested patch

r=bienvenu - are we giving auth plain the right precedence when multiple
mechanisms are supported by the server, i.e., preferring it to auth login?
Attachment #131137 - Flags: review?(bienvenu) → review+
Yep, PLAIN before LOGIN.
Attachment #131137 - Flags: superreview?(scott)
Attachment #131137 - Flags: superreview?(scott) → superreview+
fix checked in. Thx, Christian.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
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

Creator:
Created:
Updated:
Size: