Closed
Bug 211105
Opened 22 years ago
Closed 22 years ago
advertisement of CRAM-MD5 not always recognize
Categories
(MailNews Core :: Networking: SMTP, defect)
MailNews Core
Networking: SMTP
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ch.ey, Assigned: ch.ey)
Details
Attachments
(1 file)
|
953 bytes,
patch
|
Bienvenu
:
review+
|
Details | Diff | Splinter Review |
In patch to bug 202162 I made a mistake by changing the start offsets for
m_responseText.Find("CRAM-MD5", PR_TRUE, 8)
and
m_responseText.Find("EXTERNAL", PR_TRUE, 8)
That now leads to only recognize mechanisms CRAM-MD5 and EXTERNAL if they're not
the first one in the AUTH line.
|
Assignee | |
Comment 1•22 years ago
|
||
Patch to fix this problem.
|
|
Assignee | |
Updated•22 years ago
|
Attachment #126745 -
Flags: review?(bienvenu)
|
||
Comment 2•22 years ago
|
||
Comment on attachment 126745 [details] [diff] [review]
patch
r=bienvenu - let me know if you want me to check this in.
Attachment #126745 -
Flags: review?(bienvenu) → review+
|
|
Assignee | |
Comment 3•22 years ago
|
||
Yes, that would be nice, thanks.
|
|
||
Comment 4•22 years ago
|
||
OK, thx for fixing this; I'll check it in the next time I have stuff to check in.
Assignee: sspitzer → ch.ey
|
|
||
Comment 5•22 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
||
Updated•21 years ago
|
Product: MailNews → Core
|
||
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•