Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some folders are not removed from K9 after a refresh folder list action #5661

Closed
gazzawazza opened this issue Sep 11, 2021 · 6 comments · Fixed by #5687
Closed

some folders are not removed from K9 after a refresh folder list action #5661

gazzawazza opened this issue Sep 11, 2021 · 6 comments · Fixed by #5687
Labels
type: bug Something is causing incorrect behavior or errors

Comments

@gazzawazza
Copy link

gazzawazza commented Sep 11, 2021

hi all

firstly, i have searched for "folders not removed". I've looked at both open and closed issues and can't seem to find anything that matches my problem. This of course makes me wonder if it's something specific to me.

Describe the bug

when i've renamed a folder in gmail, i would expect the k9 refresh folder list action to:

a) produce the newly named folder

b) remove the old / missing folder

As per your FAQ here https://k9mail.app/documentation/faq.html#anchor16

However, I've got K9 folders that no longer exist in the browser gmail experience (because i've renamed them), yet they are still visible in K9.

Refresh Folder List does not remove them.

To Reproduce

i can only partially reproduce it now :(

I've literally just tested with a new folder in browser gmail (no emails in test folder), just so as to not waste your time.

  • I refreshed the k9 folder list. New folder appeared.
  • i then renamed in gmail (browser) and k9 folder refreshing changed the folder name. The previous folder name no longer exists in K9.
  • As a final test, i deleted the gmail folder ("remove label" command) and K9 folder refreshed. Folder disappears.

I retested the above with an email in it (admittedly a draft email).

  • Test folder with emails in it appears in K9.
  • Renamed test folder with email in it appears in K9.
  • Old test folder with email disappears from K9.
  • Removing test folder from gmail removes from K9 after folder refresh.

However, i DO have existing K9 folders which don't exist on gmail any more, yet i can't delete them from K9 because there's no delete function and refresh folder list doesn't remove them.

  • Although there were no emails showing the redundant K9 folders, i tried "clear local messages", in case there was some data marked against them. This didn't help.
  • I've got 8 folders in K9 like this, which don't exist on gmail.
  • I did think about the possibility of compacting the account but i think that functionality has been removed.

Expected behaviour

all folders that no longer exist in gmail should get removed by K9's refresh folder list action.

Additional context

using K9 v5.806 on Samsung S8 with official OTA android release 9 i.e. pie

Am using IMAP for gmail to K9 operations.

Logs

didn't attach any as i get no errors

debug log attached for "Refresh Folder List" action on my the account in question.
k9-log.txt

regards,

Gary

@gazzawazza gazzawazza added the type: bug Something is causing incorrect behavior or errors label Sep 11, 2021
@cketti
Copy link
Member

cketti commented Sep 13, 2021

Please record a debug log while refreshing the folder list. See https://github.com/k9mail/k-9/wiki/LoggingErrors

This will log the client-server communication so we can see what data Gmail is delivering to the app.

@gazzawazza
Copy link
Author

Please record a debug log while refreshing the folder list. See https://github.com/k9mail/k-9/wiki/LoggingErrors

This will log the client-server communication so we can see what data Gmail is delivering to the app.

tyvm

presumably i attach the log to my opening thread?

@cketti
Copy link
Member

cketti commented Sep 13, 2021

presumably i attach the log to my opening thread?

Please create a new comment when attaching the log. This will trigger a notification email.

@gazzawazza
Copy link
Author

presumably i attach the log to my opening thread?

Please create a new comment when attaching the log. This will trigger a notification email.

tyvm. done.

@gazzawazza
Copy link
Author

logs for Refresh Folder List action (as suggested by cketti)

k9-log.txt

@cketti
Copy link
Member

cketti commented Sep 14, 2021

The log indicates that some folders have been removed locally. Could it be possible that the folder list in the UI is just not updated? Can you test creating/removing a folder on the server, refreshing the folder list, killing the app, then checking if the local folder list properly reflects the server state when the app is restarted?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is causing incorrect behavior or errors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants