diff options
author | ARaspiK <araspik@protonmail.com> | 2020-08-18 20:27:23 +0000 |
---|---|---|
committer | Reto Brunner <reto@labrat.space> | 2020-08-19 11:38:57 +0200 |
commit | fe1cabb077cf6c6cb3de122b3f5532acbeba8c85 (patch) | |
tree | 6f57a28a507c500df82f991cefd67910eca973f0 /worker/maildir | |
parent | f4dc7e1f746582d42462ec56347dd354756203b0 (diff) | |
download | aerc-fe1cabb077cf6c6cb3de122b3f5532acbeba8c85.tar.gz |
Add support for :rmdir
The `:rmdir` command removes the current directory (`-f` is required if the directory is not empty). This is not supported on the notmuch backend. An issue with the maildir backend is that some sync programs (e.g. offlineimap) may recover the directory after it is deleted. They need to specifically be configured to accept deletions, or special commands need to be executed (e.g. `offlineimap --delete-folder`) to properly delete folders. A danger of using this on the IMAP backend is that it is possible for a new message to be added to the directory and for aerc to not show it immediately (due to a slow connection) - using `:rmdir` at this moment (with `-f` if the directory already contains messages) would delete the directory and the new message that just arrived (and all other contents). This is documented in aerc(1) so that users are aware of possible risks.
Diffstat (limited to 'worker/maildir')
-rw-r--r-- | worker/maildir/worker.go | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/worker/maildir/worker.go b/worker/maildir/worker.go index d1ff3c2..4a7ae51 100644 --- a/worker/maildir/worker.go +++ b/worker/maildir/worker.go @@ -186,6 +186,8 @@ func (w *Worker) handleMessage(msg types.WorkerMessage) error { return w.handleFetchDirectoryContents(msg) case *types.CreateDirectory: return w.handleCreateDirectory(msg) + case *types.RemoveDirectory: + return w.handleRemoveDirectory(msg) case *types.FetchMessageHeaders: return w.handleFetchMessageHeaders(msg) case *types.FetchMessageBodyPart: @@ -362,6 +364,16 @@ func (w *Worker) handleCreateDirectory(msg *types.CreateDirectory) error { return nil } +func (w *Worker) handleRemoveDirectory(msg *types.RemoveDirectory) error { + dir := w.c.Dir(msg.Directory) + if err := os.RemoveAll(string(dir)); err != nil { + w.worker.Logger.Printf("could not remove directory %s: %v", + msg.Directory, err) + return err + } + return nil +} + func (w *Worker) handleFetchMessageHeaders( msg *types.FetchMessageHeaders) error { for _, uid := range msg.Uids { |