From: Timo Sirainen <timo.sirainen@dovecot.fi>
Date: Wed, 9 Aug 2017 10:17:15 +0000 (+0300)
Subject: dsync: Add missing transaction flags when performing UID renumbering
X-Git-Tag: 2.3.0.rc1~1184
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b0ead8fffe8dfdee92fca875b91fadf3f3f2262b;p=thirdparty%2Fdovecot%2Fcore.git

dsync: Add missing transaction flags when performing UID renumbering

The transaction didn't have MAILBOX_TRANSACTION_FLAG_SYNC or
MAILBOX_TRANSACTION_FLAG_NO_NOTIFY (with DSYNC_BRAIN_FLAG_NO_NOTIFY).

This caused replication-dsync to unnecessarily trigger another dsync
replication notification.
---

diff --git a/src/doveadm/dsync/dsync-mailbox-import.c b/src/doveadm/dsync/dsync-mailbox-import.c
index 7129fb2e73..b4272927e0 100644
--- a/src/doveadm/dsync/dsync-mailbox-import.c
+++ b/src/doveadm/dsync/dsync-mailbox-import.c
@@ -2585,6 +2585,7 @@ reassign_uids_in_seq_range(struct dsync_mailbox_importer *importer,
 {
 	struct mailbox *box = importer->box;
 	const enum mailbox_transaction_flags trans_flags =
+		importer->transaction_flags |
 		MAILBOX_TRANSACTION_FLAG_EXTERNAL |
 		MAILBOX_TRANSACTION_FLAG_ASSIGN_UIDS;
 	struct mailbox_transaction_context *trans;