324309 2013-08-31 07:28:32 +0000 No mailbox is selected at startup -> "Disconnected for inactivity" when GUI is immediately hidden 2024-09-23 18:51:09 +0000 1 1 2 Applications trojita IMAP unspecified unspecified Linux RESOLVED UNMAINTAINED NOR normal --- 1 pali.rohar trojita-bugs 0 oldest_to_newest 1393972 0 pali.rohar 2013-08-31 07:28:32 +0000 I get this error after 30-40 minutes of inactivity "The underlying socket is having troubles when processing connection to imap.gmail.com:993: Remote host closed connection" I'm connecting to imap gmail server and it looks like that after some time of inactivity, gmail closing connection. After that trojita not working and restart of application is required. Reproducible: Always Steps to Reproduce: 1. Configure gmail imap account 2. Start Trojita and do nothing (no folder opening) 3. Wait about 30-40 minutes Actual Results: Trojita show error message "The underlying socket is having troubles when processing connection to imap.gmail.com:993: Remote host closed connection" And all imap folders are empty. Trojita cannot connect to imap server again, restart of application is required. Expected Results: At least Trojita should try to reconnect after server close connection. Maybe some periodical NOP command could be usefull to prevent disconnecting due to inactivity. 1393976 1 jkt 2013-08-31 07:48:16 +0000 Do you have a log of that situation? Disk based logging is probably useful in this situation (IMAP -> Debugging -> Log into ...). > After that trojita not working and restart of application is required. This is strange -- Trojita should notice that it's offline, indicate that in the network mode indicator, and a simple switch to online should fix this. What happens when you try to do this? Could you please attach the log of *that* as well? > Maybe some periodical NOP command could be usefull to prevent disconnecting > due to inactivity. Trojita will periodically ask for a STATUS of the visible mailboxes ("visible" == shown on the left). The IMAP standard guarantees that INBOX shall always be present, so there will always be at least one mailbox to poll. However, this polling does not happen when no mailboxes are "visible" (It's based on calls to QAbstractItemModel::data() via QTreeView after the model emits dataChanged; I know for sure that those mailboxes which are not visible in the view's viewport are not polled, and I don't know whether this hapens also when the window is minimized or in the systray, for example. Should be pretty easy to test -- check the m_periodicMailboxNumbersRefresh in Model.cpp.). There is also no polling when in the Expensive Network mode. 1394047 2 thomas.luebking 2013-08-31 14:02:34 +0000 (In reply to comment #1) > Trojita will periodically ask for a STATUS of the visible mailboxes > ("visible" == shown on the left). The IMAP standard guarantees that INBOX > shall always be present, so there will always be at least one mailbox to > poll. However, this polling does not happen when no mailboxes are "visible" > (It's based on calls to QAbstractItemModel::data() via QTreeView after the > model emits dataChanged; I know for sure that those mailboxes which are not > visible in the view's viewport are not polled, and I don't know whether > this hapens also when the window is minimized or in the systray, for > example. I didn't see any explicit calls to data() for "visible" items in the code, so this probably relies on the access through the repaint in return of the protected dataChanged() slot of the view - and in this case the answer is "no". All widget repaints are compressed for invisible widgets since "show" will trigger a repaint anyway, so repaint won't cause a paintEvent here. This should fail if - the "left" splitter part is collapsed - the "one item" layout is used and on the "wrong" view too long - the window is minimized (and not kept mapped by the compositor for a "live thumbnail") - trojita "minimized" (actually "closed" ;-) to systray This is also the pending main issue (resp. it's trigger) with multiple accounts and there needs resolution to either explicitlly disconnecting collapsed resp. completely invisible (out of widget bounds) accounts or by explicitly calling to the data of one index (per account) when focusing on merging that. 1394691 3 jkt 2013-09-03 06:28:44 +0000 > - the "left" splitter part is collapsed Only if nothing is shown anywhere else in Trojita. If that is the case, the user has bigger problems than Trojita disconnecting after half an hour, IMHO. > - the "one item" layout is used and on the "wrong" view too long In that case, either IDLE or the periodic NOOP polling is active, and the connection won't come offline. > - trojita "minimized" (actually "closed" ;-) to systray Interesting; the systray code actually only checks for messageCountPossiblyChanged(QModelIndex), a more specific signal than dataChanged(). That's likely a bug; patch for this is at https://git.reviewboard.kde.org/r/112474/ . 1394765 4 pali.rohar 2013-09-03 13:11:42 +0000 (In reply to comment #1) > Do you have a log of that situation? Disk based logging is probably useful > in this situation (IMAP -> Debugging -> Log into ...). > I will generate one. > > After that trojita not working and restart of application is required. > > This is strange -- Trojita should notice that it's offline, indicate that > in the network mode indicator, and a simple switch to online should fix > this. What happens when you try to do this? Could you please attach the log > of *that* as well? > How to switch it back to online? When I click on INBOX folder nothing happen and folder is empty. I have configured trojita to do not use offline cache and network access is free access. 1394769 5 jkt 2013-09-03 13:33:31 +0000 On Tuesday, 3 September 2013 15:11:42 CEST, Pali Rohár wrote: > How to switch it back to online? IMAP -> Network Access -> Free Access There's also an icon in the lower right for this purpose. 1394800 6 82132 pali.rohar 2013-09-03 15:03:34 +0000 Created attachment 82132 trojita-connection-log It is not so easy to reproduce this bug and provide log file. But it is possible. I needed to modify trojita code to automatically start logging to file at startup, because it is not possible to start logging via GUI button and reproducing this bug (see steps why). I think this is race condition bug. I will update steps to reproduce: 1. configure trojita to do not store IMAP password and ask it at startup and enable systray icon 2. start trojita and wait until password dialog appear 3. enter password, click OK and immediately click on systray icon which hide trojita main window 4. it is needed that trojita main window will be hidden before it connect to imap server 5. wait 30 minutes and you will see that error message If I clicked on systray icon later (not immediately after entering password), then I'm not able to reproduce this bug. 1394804 7 pali.rohar 2013-09-03 15:18:58 +0000 (In reply to comment #5) > On Tuesday, 3 September 2013 15:11:42 CEST, Pali Rohár wrote: > > How to switch it back to online? > > IMAP -> Network Access -> Free Access > > There's also an icon in the lower right for this purpose. I cannot do that from standalone trojita version, because of bug #324458. Will try it in Kontact version. 1395011 8 jkt 2013-09-04 08:00:39 +0000 OK, thanks for reporting this -- it made me fix a long-standing bug in mailbox discovery at least :). Your steps to reproduce were also very helpful -- indeed, even when the QTreeView on the left is collapsed so that it is not shown at all, its headers are still rendered which request a call to QAIM::rowCount() and the resulting activity gets noticed and a call to QAIM::data gets made which ultimately results in STATUS being sent. (The widgets are collapsed after a short delay when restoring their sizes.) This is tricky to fix properly -- the mailbox discovery is asynchronous in nature and there is no component which would "asynchronously wait for mailbox to become available". Some code is at src/XtConnect/MailboxFinder.cpp; it will have to be moved to a better location, changed so that it handles disconnects and unit-tested. When that is in place, we can also fix bug #321078, and *perhaps* also unconditionally open INBOX at the beginning. 1395341 9 jkt 2013-09-05 12:14:58 +0000 Git commit ba2b2b0f4a17cb04e6265be71da2bd29ea8b907a by Jan Kundrát. Committed on 03/09/2013 at 06:22. Pushed by jkt into branch 'master'. Model: emit messageCountPossiblyChanged when periodically invalidating the message counts The code which takes care of periodically refreshing the view of mailboxes emits dataChanged() whenever it invalidates the cached message counts for a particular mailbox. The systray code was, however, only listening to the messageCountPossiblyChanged signal. During normal operation, this is not an issue because the GUI's views would intercept the dataChanged and ask for the actual data, which triggers an IMAP STATUS which in turn updates the message counts and systray therefore notices that the numbers might have changed. However, when the views are hidden for some reason, this process does not happen and the systray icon is not updated. REVIEW: 112474 M +1 -2 src/Imap/Model/Model.cpp http://commits.kde.org/trojita/ba2b2b0f4a17cb04e6265be71da2bd29ea8b907a 2012107 10 justin.zobel 2021-03-09 07:26:10 +0000 Thank you for the bug report. As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists. If this bug is no longer persisting or relevant please change the status to resolved. 2358786 11 christoph 2024-09-23 18:51:09 +0000 Trojitá is no longer maintained, please switch to a maintained alternative like https://apps.kde.org/kmail2/ Sorry for the inconveniences. 82132 2013-09-03 15:03:34 +0000 2013-09-03 15:03:34 +0000 trojita-connection-log trojita-connection-log text/plain 3639 pali.rohar MTY6MTc6MzYuNDQ2MSAgW3Rhc2tdIEltYXA6Ok1haWxib3g6Ok9wZW5Db25uZWN0aW9uVGFzayBP cGVuQ29ubmVjdGlvblRhc2s6ICBBY3RpdmF0ZWQKMTY6MTc6MzYuNTEyMSAgPDw8ICAqKiogTG9v a2luZyB1cCBpbWFwLmdtYWlsLmNvbS4uLgoxNjoxNzozNi41MTIxICBbdGFza10gY29ubiBSZXNv bHZpbmcgaG9zdG5hbWUuLi4KMTY6MTc6MzYuNTE1MSAgPDw8ICAqKiogQ29ubmVjdGluZyB0byBp bWFwLmdtYWlsLmNvbTo5OTMgKFNTTCkuLi4KMTY6MTc6MzYuNTE1MSAgW3Rhc2tdIGNvbm4gQ29u bmVjdGluZyB0byB0aGUgSU1BUCBzZXJ2ZXIuLi4KMTY6MTc6MzYuNTM0MSAgPDw8ICAqKiogTmVn b3RpYXRpbmcgZW5jcnlwdGlvbi4uLgoxNjoxNzozNi41MzQxICBbdGFza10gY29ubiBTdGFydGlu ZyBlbmNyeXB0aW9uIChTU0wpLi4uCjE2OjE3OjM2LjU3NjEgIDw8PCAgKioqIFtTb2NrZXQgaXMg ZW5jcnlwdGVkIG5vdzsgMiBlcnJvcnM6IFRoZSBpc3N1ZXIgY2VydGlmaWNhdGUgb2YgYSBsb2Nh bGx5IGxvb2tlZCB1cCBjZXJ0aWZpY2F0ZSBjb3VsZCBub3QgYmUgZm91bmQgKENOOiBHb29nbGUg SW50ZXJuZXQgQXV0aG9yaXR5KSBUaGUgcm9vdCBDQSBjZXJ0aWZpY2F0ZSBpcyBub3QgdHJ1c3Rl ZCBmb3IgdGhpcyBwdXJwb3NlIChDTjogR29vZ2xlIEludGVybmV0IEF1dGhvcml0eSkgXQoxNjox NzozNi41NzYxICBbdGFza10gY29ubiBDaGVja2luZyBjZXJ0aWZpY2F0ZXMgKFNTTCkuLi4KMTY6 MTc6MzYuNTc2MSAgW3Rhc2tdIGNvbm4gQ2hlY2tpbmcgY2FwYWJpbGl0aWVzLi4uCjE2OjE3OjM2 LjU3NzEgIDw8PCAgKiBPSyBHaW1hcCByZWFkeSBmb3IgcmVxdWVzdHMgZnJvbSA3OC4xMjguMTkz LjIwMiBrNDRpZjExOTM0NjcyZWVuLjEzMQoxNjoxNzozNi41NzcxICBbdGFza10gY29ubiBXYWl0 aW5nIGZvciBlbmNyeXB0aW9uLi4uCjE2OjE3OjM2LjU3NzEgID4+PiAgeTAgQ0FQQUJJTElUWQox NjoxNzozNi41OTgxICA8PDwgICogQ0FQQUJJTElUWSBJTUFQNHJldjEgVU5TRUxFQ1QgSURMRSBO QU1FU1BBQ0UgUVVPVEEgSUQgWExJU1QgQ0hJTERSRU4gWC1HTS1FWFQtMSBYWVpaWSBTQVNMLUlS IEFVVEg9WE9BVVRIIEFVVEg9WE9BVVRIMiBBVVRIPVBMQUlOIEFVVEg9UExBSU4tQ0xJRU5UVE9L RU4KMTY6MTc6MzYuNTk4MSAgPDw8ICB5MCBPSyBUaGF0cyBhbGwgc2hlIHdyb3RlISBrNDRpZjEx OTM0NjcyZWVuLjEzMQoxNjoxNzozNi41OTgxICBbdGFza10gY29ubiBMb2dnaW5nIGluLi4uCjE2 OjE3OjUyLjI3ODEgID4+PiAgW0xPR0lOIGNvbW1hbmQgZ29lcyBoZXJlXQoxNjoxNzo1Mi41OTcx ICA8PDwgICogQ0FQQUJJTElUWSBJTUFQNHJldjEgVU5TRUxFQ1QgSURMRSBOQU1FU1BBQ0UgUVVP VEEgSUQgWExJU1QgQ0hJTERSRU4gWC1HTS1FWFQtMSBVSURQTFVTIENPTVBSRVNTPURFRkxBVEUg RU5BQkxFIE1PVkUgQ09ORFNUT1JFIEVTRUFSQ0gKMTY6MTc6NTIuNTk3MSAgPDw8ICB5MSBPSyBw YWxpLnJvaGFyQGdtYWlsLmNvbSBQYWxpIFJvaMODwqFyIGF1dGhlbnRpY2F0ZWQgKFN1Y2Nlc3Mp CjE2OjE3OjUyLjU5NzEgIFt0YXNrXSBjb25uIEFjdGl2YXRpbmcgY29tcHJlc3Npb24uLi4KMTY6 MTc6NTIuNTk3MSAgPj4+ICB5MiBDT01QUkVTUyBERUZMQVRFCjE2OjE3OjU1LjU3MDEgIDw8PCAg eTIgT0sgU3VjY2VzcyBbVEhST1RUTEVEXQoxNjoxNzo1NS41NzAxICBbdGFza10gY29ubiBMb2dn ZWQgaW4uCjE2OjE3OjU1LjU3MDEgIFt0YXNrXSBJbWFwOjpNYWlsYm94OjpPcGVuQ29ubmVjdGlv blRhc2sgT3BlbkNvbm5lY3Rpb25UYXNrOiBMb2dnZWQgaW4uIENvbXBsZXRlZAoxNjoxNzo1NS41 NzAxICBbdGFza10gSW1hcDo6TWFpbGJveDo6R2V0QW55Q29ubmVjdGlvblRhc2sgQWN0aXZhdGVk CjE2OjE3OjU1LjU3MDEgIFt0YXNrXSBJbWFwOjpNYWlsYm94OjpHZXRBbnlDb25uZWN0aW9uVGFz ayBDb21wbGV0ZWQKMTY6MTc6NTUuNTcxMSAgW3Rhc2tdIEltYXA6Ok1haWxib3g6Okxpc3RDaGls ZE1haWxib3hlc1Rhc2sgTGlzdGluZyBzdHVmZiBiZWxvdyBtYWlsYm94ICBBY3RpdmF0ZWQKMTY6 MTc6NTUuNTcxMSAgW3Rhc2tdIEltYXA6Ok1haWxib3g6OkdldEFueUNvbm5lY3Rpb25UYXNrIENv bXBsZXRlZAoxNjoxNzo1NS41NzExICBbdGFza10gSW1hcDo6TWFpbGJveDo6TGlzdENoaWxkTWFp bGJveGVzVGFzayBMaXN0aW5nIHN0dWZmIGJlbG93IG1haWxib3ggIEFjdGl2YXRlZAoxNjoxNzo1 NS41NzExICBbdGFza10gSW1hcDo6TWFpbGJveDo6SWRUYXNrIEFjdGl2YXRlZAoxNjoxNzo1NS44 NTQxICA+Pj4gIHkzIExJU1QgIiIgIiUiCjE2OjE3OjU1Ljg1NDEgID4+PiAgeTQgTElTVCAiIiAi JSIKMTY6MTc6NTUuODU0MSAgPj4+ICB5NSBJRCAoIm5hbWUiICJUcm9qaXRhIiAib3MiICJRdC80 LjguMzsgWDExOyBMaW51eDsgS3VidW50dSAxMi4xMCIgInZlcnNpb24iICJ2MC4zLjkzLTMxOS1n OGU1ZTJkMC1kaXJ0eSIpCjE2OjE3OjU4LjcwNTEgIDw8PCAgKiBMSVNUIChcSGFzTm9DaGlsZHJl bikgIi8iICJJTkJPWCIKMTY6MTc6NTguNzA1MSAgPDw8ICAqIExJU1QgKFxIYXNDaGlsZHJlbikg Ii8iICJbR21haWxdIgoxNjoxNzo1OC43MDUxICA8PDwgIHkzIE9LIFN1Y2Nlc3MgW1RIUk9UVExF RF0KMTY6MTc6NTguNzA1MSAgW3Rhc2tdIEltYXA6Ok1haWxib3g6Okxpc3RDaGlsZE1haWxib3hl c1Rhc2sgTGlzdGluZyBzdHVmZiBiZWxvdyBtYWlsYm94ICBDb21wbGV0ZWQKMTY6MTc6NTguNzA2 MSAgPDw8ICAqIExJU1QgKFxIYXNOb0NoaWxkcmVuKSAiLyIgIklOQk9YIgoxNjoxNzo1OC43MDYx ICA8PDwgICogTElTVCAoXEhhc0NoaWxkcmVuKSAiLyIgIltHbWFpbF0iCjE2OjE3OjU4LjcwNjEg IDw8PCAgeTQgT0sgU3VjY2VzcyBbVEhST1RUTEVEXQoxNjoxNzo1OC43MDYxICBbdGFza10gSW1h cDo6TWFpbGJveDo6TGlzdENoaWxkTWFpbGJveGVzVGFzayBMaXN0aW5nIHN0dWZmIGJlbG93IG1h aWxib3ggIENvbXBsZXRlZAoxNjoxNzo1OC43MDcxICA8PDwgICogSUQgKCJuYW1lIiAiR0ltYXAi ICJ2ZW5kb3IiICJHb29nbGUsIEluYy4iICJzdXBwb3J0LXVybCIgImh0dHA6Ly9zdXBwb3J0Lmdv b2dsZS5jb20vbWFpbCIgInZlcnNpb24iICJnbWFpbF9pbWFwXzEzMDgyNy4wMF9wMSIgInJlbW90 ZS1ob3N0IiAiNzguMTI4LjE5My4yMDIiKQoxNjoxNzo1OC43MDcxICA8PDwgIHk1IE9LIFN1Y2Nl c3MgW1RIUk9UVExFRF0KMTY6MTc6NTguNzA3MSAgW3Rhc2tdIEltYXA6Ok1haWxib3g6OklkVGFz ayBDb21wbGV0ZWQKMTY6NDc6NTguNjYwMSAgPDw8ICAqKiogU29ja2V0IGRpc2Nvbm5lY3RlZDog VGhlIHVuZGVybHlpbmcgc29ja2V0IGlzIGhhdmluZyB0cm91YmxlcyB3aGVuIHByb2Nlc3Npbmcg Y29ubmVjdGlvbiB0byBpbWFwLmdtYWlsLmNvbTo5OTM6IFZ6ZGlhbGVuw4PCvSBwb8OEwo3Dg8Kt dGHDhMKNIHV6YXZyZWwgc3BvamVuaWUKMTY6NDc6NTguNjYwMSAgW2Vycl0gIFRoZSB1bmRlcmx5 aW5nIHNvY2tldCBpcyBoYXZpbmcgdHJvdWJsZXMgd2hlbiBwcm9jZXNzaW5nIGNvbm5lY3Rpb24g dG8gaW1hcC5nbWFpbC5jb206OTkzOiBWemRpYWxlbsO9IHBvxI3DrXRhxI0gdXphdnJlbCBzcG9q ZW5pZQoxNjo0Nzo1OC42NjAxICA+Pj4gICoqKiBDb25uZWN0aW9uIGNsb3NlZC4K