Bug 2028 - Attempts to connect to all accounts on exit
Summary: Attempts to connect to all accounts on exit
Status: RESOLVED FIXED
Alias: None
Product: Claws Mail (GTK 2)
Classification: Unclassified
Component: Folders/IMAP (show other bugs)
Version: 3.7.3
Hardware: PC Linux
: P3 normal
Assignee: users
URL: http://bugs.debian.org/552295
Depends on:
Blocks:
 
Reported: 2009-10-25 13:46 UTC by Ricardo Mones
Modified: 2009-11-02 07:24 UTC (History)
0 users

See Also:


Attachments
possible fix (832 bytes, patch)
2009-10-26 10:07 UTC, Colin Leroy
no flags Details | Diff

Description Ricardo Mones 2009-10-25 13:46:20 UTC
Quoting original submitter:
"
Since upgrading to claws-mail-3.7.3-1, it attempts to connect to all 
accounts that I have defined. All of these are IMAP accounts. This 
includes accounts that have not been used during the session, and that 
do not get automatically checked for new messages.

This is problematic for me since I do not have access to all of my
accounts all of the time. As a result, when I try to close claws-mail, I
either have to wait for all of the connection attempts to time out, or kill
claws-mail.
"

BTW, I've observed also this behaviour, but as I don't save passwords I can dismiss all password dialogs by pressing escape key several times :-)
Comment 1 Colin Leroy 2009-10-25 15:23:25 UTC
This is due to "Warn if there are queued messages" on exit, Preferences/Other/Miscellaneous.

HTH
Comment 2 Michael Williamson 2009-10-25 16:07:35 UTC
I have none of the checkboxes ticked under "On exit", nor do I get any password dialogs (I don't save any passwords).
Comment 3 JG 2009-10-25 17:17:26 UTC
the same happens here too (and there's at least one other user (phx) on irc who's got the same problem) but not (only) on exit but also when cm is running. 
after some time (say 3-4 times of auto-checking all activated accounts) CM suddenly starts to check (IMAP) accounts which are disabled (for auto "get mail").

colin, i've sent you --debug output via email (Subject: CM 3.7.3 auto-account check buggy) this morning.

please reopen.
Comment 4 Ricardo Mones 2009-10-25 21:07:19 UTC
Great, thanks Colin!

Certainly in my case unchecking that checkbox fixed the problem, will ask original submitter whether is the same case or not.
Comment 5 JG 2009-10-26 09:43:49 UTC
did not fix the problem here. cm still wants to fetch emails from imap accounts that are "disabled" in the account properties.

i just started cm and let it run in the background without clicking any buttons and doing nothing at all in the gui and after some time (maybe 10-15 minutes) it starts checking those accounts..
Comment 6 Ricardo Mones 2009-10-26 09:51:37 UTC
Quoting original reporter:
"
>   Do you have the checkbox "Warn if there are queued messages"
> checked? It's on Configuration menu, Preferences option,
> Other/Miscellaneous panel.
> 
>   In my case unchecking this fixes claws-mail asking for those
> accounts.

I have this unchecked, and the problem persists.
"

So I guess is a different issue.
Comment 7 Colin Leroy 2009-10-26 10:07:57 UTC
Created attachment 739 [details]
possible fix

Indeed, it's probably due to scanning queue anyway... Can you test the attached patch?
Comment 8 JG 2009-10-26 12:26:21 UTC
cm+patch has been running for 1-2 hours now without checking disabled accounts. it seems fixed, thanks very much! if other problems arise (like not checking any accounts or similar) i'll report back.
Comment 9 Paul 2009-10-27 07:30:29 UTC
this ain't fixed til the attached patch is in cvs
Comment 10 users 2009-10-27 08:35:34 UTC
Changes related to this bug have been committed.
Please check latest CVS and update the bug accordingly.
You can also get the patch from:
http://www.colino.net/claws-mail/

2009-10-27 [colin]	3.7.3cvs10

	* src/folder.c
		Fix bug 2028, 'Attempts to connect to all accounts on exit'
		Don't try to check whether the folder is up-to-date if
		the folder class doesn't implement set_mtime, as we don't
		care in this case. This code exists since a long time, but
		the bug was triggered by more scanning of queue folders 
		since recently (when we added Deleted handling).

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