Bug 2379 - prompt for password after authentication timeout
Summary: prompt for password after authentication timeout
Status: NEW
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: POP3 (show other bugs)
Version: 3.8.0
Hardware: PC Linux
: P3 enhancement
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2011-03-10 21:38 UTC by Robert
Modified: 2012-02-21 15:30 UTC (History)
0 users

See Also:


Attachments
patch for non-password auth errors (1.36 KB, patch)
2012-02-15 15:49 UTC, Robert
no flags Details | Diff
updated patch (2.69 KB, patch)
2012-02-21 15:28 UTC, Robert
no flags Details | Diff
example poprc with non-password error strings (164 bytes, text/plain)
2012-02-21 15:30 UTC, Robert
no flags Details

Description Robert 2011-03-10 21:38:09 UTC
Once a week or so my POP3 server times out during authentication. When this happens, claws will ask me for my password again.  The error message for a bad password is different, so it seems like this shouldn't be necessary.

Timeout:
[16:09:42] POP3< -ERR Connect attempt failed: server may be down or too busy to respond.

Bad password:
[16:35:24] POP3< -ERR Logon failure: unknown user name or bad password.
Comment 1 Robert 2012-02-15 15:49:18 UTC
Created attachment 1077 [details]
patch for non-password auth errors

I've recently started using DavMail to interface with an Exchange server via EWS, and the EWS server is a little flaky. So sometimes I get "EWS end point not available". This causes the same issue as the connect attempt failure - I get prompted for my password on the next attempt.  This happens much more frequently than the previous error, which finally drove me to fix it.

The attached patch adds a hardcoded list of string which are compared against the ERR string during authentication. If the error matches a string in the list, the error code is changed from PS_AUTHFAIL to PS_ERROR. The error is logged, but the user will not be prompted for their password on the next connection attempt.

Patch is against 3.8.0.
Comment 2 Robert 2012-02-21 15:28:06 UTC
Created attachment 1081 [details]
updated patch

A new non-error error popped up today. I don't want to have to rebuild claws every time this happens, so I rewrote the patch to read a config file for non-password errors instead of hardcoding them.
Comment 3 Robert 2012-02-21 15:30:01 UTC
Created attachment 1082 [details]
example poprc with non-password error strings

example poprc config file with the 3 non-password auth errors I've collected so far.

Note You need to log in before you can comment on or make changes to this bug.