Closed
Bug 262584
Opened 20 years ago
Closed 20 years ago
Infinite logon retry loop of 10/01 build when POP3 server(APOP) responds "-ERR [AUTH] CRAM-MD5 authentication is disabled." to "AUTH CRAM-MD5" request
Categories
(MailNews Core :: Networking: POP, defect)
MailNews Core
Networking: POP
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: World, Assigned: ch.ey)
Details
Attachments
(3 files, 2 obsolete files)
(deleted),
text/plain
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
patch
|
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a4) Gecko/20040928
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a4) Gecko/20040928
When POP3 server(APOP) responds "-ERR [AUTH] CRAM-MD5 authentication is
disabled." to "AUTH CRAM-MD5" request, 2004100105-trunk/Win-2K retried logon
forever.
No probelm on 2004092805-trunk.
Reproducible: Always
Steps to Reproduce:
Reporter | ||
Comment 1•20 years ago
|
||
Reporter | ||
Comment 2•20 years ago
|
||
Reporter | ||
Updated•20 years ago
|
Attachment #160833 -
Attachment description: POP3 NSPR.LOG - Logon retry loop by 2004-01-10-05 → POP3 NSPR.LOG - Logon retry loop by 2004-10-01-05 build
Comment 3•20 years ago
|
||
Christian: This is to 99,9% caused by your Patch from Bug 257299.
Assignee | ||
Comment 4•20 years ago
|
||
(In reply to comment #1)
> 2004100105-trunk/Win-2K retried logon forever.
> No probelm on 2004092805-trunk.
Right Frank, this is caused by my patch.
The "[AUTH]" means according to RFC 3206 "any authentication error caused by a
problem with the user's credentials."
Before the patch we just continued trying other mechanisms, now we go the
password failed way.
Besides the fact that the server is lying (not the credentials are wrong but
CRAM-MD5 disabled - besides he still advertises it ...), normaly password failed
should ask for a new password.
Because of to many false password discards we never do this anymore if sending
the password succeeded once in this session.
Can this have been the case, Wada?
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All
Assignee | ||
Comment 5•20 years ago
|
||
While in the case Wada logged it's not really true, we could trust the server
if it sends the [AUTH] code.
This patch forces the password dialog for this case.
Assignee: sspitzer → ch.ey
Status: NEW → ASSIGNED
Assignee | ||
Comment 6•20 years ago
|
||
Comment on attachment 160850 [details] [diff] [review]
proposed patch
Sorry, wrong patch
Attachment #160850 -
Attachment is obsolete: true
Assignee | ||
Comment 7•20 years ago
|
||
This is what I meant.
Assignee | ||
Comment 8•20 years ago
|
||
That's much better. Instead faking around, just use POP3_AUTH_FAILURE as
indicator to forget password in GetPassword().
Also clearing the flag should not be forgotten.
Attachment #160851 -
Attachment is obsolete: true
Assignee | ||
Comment 9•20 years ago
|
||
David, IIRC you implemented this isAuthenticated and
m_pop3ConData->logonFailureCount. So what do you think?
Comment 10•20 years ago
|
||
Comment on attachment 160856 [details] [diff] [review]
patch v2
looks good to me - sorry I didn't comment earlier; I never saw the bugzilla
mail notification
Attachment #160856 -
Flags: superreview+
Assignee | ||
Comment 11•20 years ago
|
||
Patch v2 has been checked in as part of patch for bug 249240.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 12•20 years ago
|
||
Tested on Mozilla Suite 2004102805-trunk/Win-2K.
Problem still occurs.
If password was deleted from password manager and "use password manager" was not
requested on any password prompt, retry loop could be canceled.
But once "use password manager" was requested on a password prompt, password was
not prompted anymore, and uncancelable infinite retry loop occured again.
Assignee | ||
Comment 13•20 years ago
|
||
(In reply to comment #12)
> Tested on Mozilla Suite 2004102805-trunk/Win-2K.
>
> Problem still occurs.
Strange. While I don't have such a server like you, my simulated one should
behave like it. And since my patch it works well here.
Note, it will still loop if
a) the server doesn't answer with [AUTH] (although it does in your posted logs).
b) your username contains a dot (.), see bug 263542.
If nothing of both applies to you I can't help with a more detailed description
and new log (if it differs from the others).
BTW, you can cancel the loop if you don't click "OK" but press ESC and hold it
until the dialog stops.
Reporter | ||
Comment 14•20 years ago
|
||
> (In reply to comment #12)
> b) your username contains a dot (.), see bug 263542.
Gotcha!
My username contains multiple dots - userid@server.subdomain.company.ne.jp
> BTW, you can cancel the loop if you don't click "OK" but press ESC and hold it
> until the dialog stops.
I closed pop up dialog by clicking close button("X" of top/right corner).
This probably means I pressed "OK" button.
Anyway, my original problem of this bug has been fixed by you.
And my latest loop in comment #12 is DUP of bug 263542.
So I change this bug "VERIFIED".
Thanks a lot, Christian.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 15•20 years ago
|
||
No problem. Let's hope the dot-escape-bug will be fixed soon.
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•