BTW, I noted that some of your differentials cannot be easily used. arc patch returns "Base commit is not in local repository". Then I get:

Checking patch docker/synapse/rootfs/opt/app-root/src/init.sh...
error: docker/synapse/rootfs/opt/app-root/src/init.sh: does not exist in index
Checking patch docker/synapse/rootfs/opt/app-root/src/homeserver.yaml...
error: docker/synapse/rootfs/opt/app-root/src/homeserver.yaml: does not exist in index
Checking patch docker/synapse/Dockerfile...
error: docker/synapse/Dockerfile: does not exist in index
Checking patch config.prod/src/database/seeds/PassportSeeder.php...
Checking patch config.demo/src/database/seeds/PassportSeeder.php...
Checking patch ci/testctl...
Hunk #1 succeeded at 111 (offset -1 lines).
Checking patch ci/env...
error: while searching for:
MEET_SERVER_TOKEN=simple123
PASSPORT_PROXY_OAUTH_CLIENT_ID=5909ca4f-df7e-45fe-b355-e7c195aef117
PASSPORT_PROXY_OAUTH_CLIENT_SECRET=3URb+3JGJM9wPuDnlUSTPOw2mqmHsoOV8NXanx9xwQM=
DES_KEY=kBxUM/53N9p9abusAoT0ZEAxwI2pxFz/

I'm going to write tests.

• Better const name

• EWS: Fix some contact properties in exported vCard
• EWS: Fix lists handling
• EWS: Improved tasks handling

Nope. If you do this the header will be send to every recipient. Note that we're unsetting the header in $smtp_headers only here. The copy of the message saved to Sent folder should contain Bcc header intact.

Thanks. Fixed.

In D4845: IMAP data migrator and other improvements I took this differential of yours, but I re-implemented some things differently. I think it's cleaner. Still a lot of TODO lines there. So, we might improve while writing tests. I have also an idea to try to do mail streaming for best performance.

Throwable catches that fatal error. Now looking at this I found that we could change the catch in https://git.kolab.org/diffusion/RPK/browse/master/plugins/libcalendaring/lib/libcalendaring_vcalendar.php$158

• It's a debug not a warning

• Move sorting to later place

• Fix sorting callback

• Fix sign

Indeed. I just noticed that the actual error wasn't an exception. It was `PHP Fatal error: Uncaught Error: __clone method called on non-object in /usr/share/roundcubemail/plugins/libcalendaring/lib
/libcalendaring_vcalendar.php:633`, so to catch it we'd have to catch Throwable, I suppose, I'm not sure.

• phpdoc

• Missing DAV/Search

Good progress, but I was about to work on DAV, and I already have some conflicting code. Please, don't work on DAV at this moment.

Short term, we should update the php-http-request2 package. 2.6.0 supports PHP >= 5.6 still, so there's no issue with that.

• Merge branch 'master' into dev/dav-folders-init

• Small improvements

• Fix case

• Remove unused $response variable

Almost there. tests/Browser/WalletTest.php needs an update too.

• Workaround KTOR client bug, add development commands to send push notifications

Maybe something like this:

--- a/cyruslib.py
+++ b/cyruslib.py
@@ -885,6 +885,7 @@ class CYRUS:
             annotation = annotation.strip()
             tokens = tokenize(annotation)
             folder = tokens[0]
+            mailbox = ensure_binary(mailbox)

See my take in D4794: Fold WalletCharge and WalletCheck into one