Check via unauthenticated OPTIONS request

In D5322#68733, @machniak wrote:

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

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

• Include max_password_age setting too

• Fix getPolicySetting() default value

In D5310#68598, @machniak wrote:

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.

However, because kolab is a plugin itself, it can add plugins, but it cannot remove already loaded ones.

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.

Remove accidentally comitted line

no change

check the hostname via middleware to return 404 on non-implemented routes

Remove the domain requirement completely instead

• Small CS fix

• Support delegation in LDIF import
• Optimize user creation
• Optimize aliases import
• Support import of password history
• Rename migration file

We should do the same in syncroton_folder table

Let's try with replacing the index then...

BTW, we have the same situation in syncroton_folder table. The table is smaller, so it probably is not a problem, but we should use the same index there.

Composite key with device_id makes a lot more sense

In D5295#68265, @mollekopf wrote:

Seems to work:

MariaDB [roundcube]> ANALYZE SELECT * FROM `syncroton_content` WHERE `device_id` = 'b64bcab5169887ba3318728b5174be9958cf3a00' AND `folder_id` = '75e3e4a5c192280e5d6f62281453edbeb9d8252b' AND `is_deleted` = 1;
+------+-------------+-------------------+------+---------------------------------------------------------------+------+---------+------+-------+----------+----------+------------+-------------+
| id   | select_type | table             | type | possible_keys                                                 | key  | key_len | ref  | rows  | r_rows   | filtered | r_filtered | Extra       |
+------+-------------+-------------------+------+---------------------------------------------------------------+------+---------+------+-------+----------+----------+------------+-------------+
|    1 | SIMPLE      | syncroton_content | ALL  | device_id--folder_id--contentid,syncroton_contents::device_id | NULL | NULL    | NULL | 18487 | 17559.00 |    50.00 |       0.00 | Using where |
+------+-------------+-------------------+------+---------------------------------------------------------------+------+---------+------+-------+----------+----------+------------+-------------+

This is just a draft for potential further improvements. Untested, and unclear if actually an improvement.

Use KEY which is a synonym for INDEX

Seems to work:

Did you test whether it indeed does sped up the query?

This is completely untested, and the syntax may be wrong, so not ready to be merged.

I didn't actually plan on removing auth.

• Lowercase kolabAllowSMTPSender value

• Lower-case addresses in shared folders' acl

• Fix domains import

• Prevent an error when a line reminder is null, not a string
• Prefer displayName over CN for contacts
• phpstan error fix