search

cyrusimap / cassandane

Other
6 stars 11 forks source link

Sieve in UUID mailbox #178

Closed ksmurchison closed 2 years ago

ksmurchison commented 2 years ago

For testing https://github.com/cyrusimap/cyrus-imapd/pull/3487

ksmurchison commented 2 years ago

@elliefm The reason for the bc_only flag for assetsieve[not_]exists() is that install_sieve_script() plops raw sieve right in the sievedir, and doesn't use JMAP or ManageSieve to do so. So, pre-replication we have raw scripts in sievedir, and post-replication we don't because they have been migrated into the #sieve mailbox during replication.

elliefm commented 2 years ago

Ahh that makes sense! I suppose, long term, it would be good if install_sieve_script did it through the proper channels, but that would require every test that might need it to have a JMAP or ManageSieve endpoint listening, so it's not a quick fix in the way the $bc_only flag is.

ksmurchison commented 2 years ago

Moved these commits to https://github.com/cyrusimap/cyrus-imapd/pull/3487