No, actually it's valid for server to just ignore any non-permanent flag

updates.

--HG--
branch : HEAD

diff --git a/src/lib-storage/index/index-update-flags.c b/src/lib-storage/index/index-update-flags.c
index 29e833dd3a60bf0a81def098b01dd59d801e3817..4e4ae102c249b684ff55e609519add009671b714 100644 (file)
--- a/src/lib-storage/index/index-update-flags.c
+++ b/src/lib-storage/index/index-update-flags.c
@@ -67,9 +67,11 @@ int index_storage_update_flags(struct mailbox *box, const char *messageset,
        int ret, ret2;
 
        if (box->readonly) {
-               /* FIXME: we should use memory */
-               mail_storage_set_error(box->storage, "Mailbox is read-only");
-               return FALSE;
+               box->storage->callbacks->
+                       notify_no(&ibox->box,
+                                 "Mailbox is read-only, ignoring store",
+                                 box->storage->callback_context);
+               return TRUE;
        }
 
        mail_flags = flags->flags;