The slow_log becomes kinda redundant, but it's a "warning" level, so we can keep that.

• More debug logs

• Simplify

New diff created D5376: Include shared folders and resources when looking for a mail recipient

• Use walletOwner() where appropriate

Ok, maybe it's indeed less problematic this way.

This probably breaks tests/Feature/Console/Status/HealthTest.php

Fixed.

I was unable to reproduce the issue on Kolab's Cyrus DAV server, but I could when I used % character in an UID.

Seems middleware() is not removing group middleware, we need withoutMiddleware(). Test added.

• Fix middleware on some routes
• Merge branch 'master' into dev/routes-cleanup

I suppose handling of 'skin' option could be moved to applyConfiguration()

• Reset relation state in addCOntroller() and removeController()

Maybe we could just use DAV::options() when user/pass is not provided.

• Include max_password_age setting too

• Fix getPolicySetting() default value

That would mean that if connection to cockpit fails the plugin should throw a big error, otherwise use may get access to plugins he should not. ALso, plugin already have support for merge: prefix, so this should be remove:plugins.