lib-dcrypt: test-crypto - Add missing x9.62 roundtrip test

lib-dcrypt: Rename dcrypt-openssl.c to dcrypt-openssl1.c

lib-dcrypt: Add asserts to key ID and Usage accessors

lib-dcrypt: Make error_r required

global: Call dcrypt always with error_r

lib-dcrytp: istream-decrypt - Report DER decoding issues

Fixes possibility of DER object decode error being reported
as truncation.

lib-dcrypt: Improve OpenSSL error handling

Will now try to get the error more thoroughly and handles out of memory
correctly.

lib-ssl-iostream: Fix compability with OpenSSL 3.0

lib-ssl-iostream: Do not attempt to load empty DH parameters

OpenSSL3 complains about empty DH parameters.

lib-ssl-iostream: Use EVP_PKEY to wrap DH parameters

Simplifies OpenSSL 3.0 compability

lib-ssl-iostream: Remove support for pre-1.0.2 OpenSSL

m4: Update ssl.m4

lib-ssl-iostream: Add test DH parameters

replicator: Fix crash when removing user from queue and debug logging is enabled

Broken by 4b5337667cea6dea3d684540ee79f26292f9b42d

replicator: Fix crash when importing user not listed by userdb iteration

Broken by 31545b4f38ea85615f0111a33d630a03d92d22ab

dsync: ns_mailbox_try_alloc() - Don't set errstr_r when mailbox wasn't found

It's unnecessary and confusing, since the callers can't really use it
anyway.

dsync: Revert logging a more detailed reason why mailbox was lost

This code can't actually work, because the mailbox allocation is attempted
in multiple namespaces. The error would just come from the last namespace,
which might not have been the correct one.

Reverts cdf981514af4eb8af5a459d8e68bed2cccd7f37c

dsync: Remove unnecessary DSYNC_MAILBOX_IMPORT_FLAG_DEBUG

The debug status comes from the parent event.

dsync: Remove unnecessary DSYNC_MAILBOX_TREES_SYNC_FLAG_DEBUG

The debug status comes from the parent event.

dsync: Use event_want_debug() rather than checking for debug flag

dsync: Remove unused DSYNC_MAILBOX_TREES_SYNC_FLAG_MASTER_BRAIN

dsync: Remove duplicate "brain" log prefix

Broken by 3ddb802f3e6faaac4fbeae1b33043bf0046138f4

maildir: If mailbox create fails, don't try to apply mailbox_update changes

This was especially causing problems with mailbox GUID mismatches in dsync.
The mismatch wasn't automatically fixed, because mailbox_create() incorrectly
kept updating the mailbox GUID for the already existing mailbox.

Fixes errors such as:
Remote lost mailbox GUID ... (maybe it was just deleted?)

dsync: Log a more detailed reason why mailbox was lost

Although usually the reason is just that it wasn't found.

doveadm-server: Log dsync's "desync" message with debug level

Previously this was logged only by the CLI doveadm tool as a warning.

replicator: Log dsync error code in debug message

lib-doveadm, doveadm: Change hardcoded exit code 2 to DOVEADM_EX_CHANGED

This also gives it a "CHANGED" error code in the doveadm protocol.

aggregator: Provide name for notify connection

aggregator: Add debug logging

replicator: Add debug logging

replicator: Add "replication" category to events

replication plugin: Use "replication: " log prefix for all log messages

Some of the messages were missing the prefix. Some were using
"replication(user): " as the prefix, which was unnecessary since the
mail_log_prefix already contains the username.

replication plugin: Add "replication" category to events

replicator: Add unit test to replicator-queue

replicator: Add replicator_queue_count() and replicator_queue_peek()

These will be useful for the following unit test.

replicator: Add "next sync secs" field to doveadm replicator status

replicator: Make sure to prevent request starvation

This synchronizes how priority queue is ordered vs what
replicator_queue_want_sync_now() returns. The mismatch might have caused
request starvation in some situations because they behaved differently.

Also this change makes sure that higher priority requests don't infinitely
block lower priority requests. Instead, they get a temporary boost time
(hardcoded 15-45 minutes). Afterwards lower priority requests are started
to be handled as well.

replicator: Don't update user's last_update if priority doesn't change

This didn't actually matter, since it was only used to check if importing
should update an existing user. However, importing is only run at startup
when all users have last_update=0 anyway.

The following changes bring new use cases for last_update.

replicator: Change replicator_queue_add_sync_callback() to take user struct

This is just to clean up the API to match the previous change.

replicator: Fix user placement in replication queue

Especially replicator queue importing and NOTIFY command could have
placed the user to wrong place in the queue, because they modified the
last sync timestamps afterwards.

Fixed by splitting replicator_queue_add() into get/update/add(), so all
the necessary changes can be done before replicator_queue_add() is used
to place the user into the queue.

quota: Drop rquota-pragmas.h

quota: quota-fs - Ignore boolean errors with rquota.h

quota: Fix rquota generation rules

lib-storage: mailbox_attribute_get*() - Fail early on with empty key lookup

This fixes using mail_attribute_dict with dict-sql.

stats: Replace i_<log>() with e_<log>()

replication: replicator - Replace i_<log>() with e_<log>()

replication: struct replicator_queue - Move to replicator-queue-private.h

replication: aggregator - Replace i_<log>() with e_<log>()

pop3: Replace i_<log>() with e_<log>()

pop3: struct client - Add event

master: Replace i_<log>() with e_<log>()

This reverts commit 4c6bd967436db31b5a050a55a3a35d83c70e299c.

master: Replace service_error() with e_error()

This reverts commit e9ea9531521abf1e1a8456b30bf9d3668186b26a.

master: Add struct service.event, struct service_list.event

login-common: Replace i_<log>() with e_<log>()

log: Replace i_<log>() with e_<log>()

lda: Replace i_<log>() with e_<log>()

imap-urlauth: Replace i_<log>() with e_<log>()

imap: Replace i_<log>() with e_<log>()

auth: Replace i_<log>() with e_<log>()

lib-sql: Replace i_<log>() with e_<log>()

lib-program-client: Replace i_<log>() with e_<log>()

lib-mail: istream-attachment-extractor - Return read failure on unexpected errors

If the istream content unexpectedly changes or there are any syscall failures, the
istream reading should fail rather than handle it the same as if the base64 input
was invalid.

The way astream_part_finish() reports errors been modified:
Instead of returning -1 (failure) and 0(success) now it returns
-1 (failure, propagate *error and abort), 0 (invalid base64), 1(success)

lib-ldap: Replace i_<log>() with e_<log>()

lib-ldap: struct ldap_client_settings - Add event

lib-lda: Replace i_<log>() with e_<log>()

lib-index: Replace i_<log>() with e_<log>()

lib-imap-urlauth: imap_urlauth_fetch_local() - Reformat code

lib-imap-urlauth: Replace i_<log>() with e_<log>()

lib-imap-client: Replace i_<log>() with e_<log>()

lib-imap-client: Expose struct imapc_connection.event

lib-http: Replace i_<log>() with e_<log>()

lib-fts: Update error message for the case where a stopword list is empty.

Now this condition is no longer allowed.

lib-dict-backend: ldap_dict_lookup_callback() - Reformat code

lib-dict-backend: Replace i_<log>() with e_<log>()

lib-dict-backend: struct ldap_dict - Add event

lib-login: Replace i_<log>() with e_<log>()

lib-login: struct login_server_connection  - Move to header file and add event field

lib-master: Replace i_<log>() with e_<log>()

lib-auth: auth_user_fields_parse() - Fail on parse errors rather than just log it

login-common: Remove access-lookup.[ch]

leftover from 97815ea9

global: Remove stray whitespaces

imap-login: Amend ID cmd response with trust-state

lib-storage: Finish renaming mail_stream_opened to mail_stream_accessed in comment

Forgotten in d7b8c4097185e2ab40acf43524f48fcde80f615d

doveadm: Make some functions public to allow plugins to send proxy commands

doveadm: Remove global cmd_ctx variable

lib: Always evaluate e_error(), e_warning() and e_info() parameters

e_debug() still delays evaluation until it's known that the event won't
simply be discarded. Since there are lots of e_debug() calls, this is a
good performance optimization. Other types of logging are much more rare
though and it's not so useful as an optimization. It can also be
unexpected that the evaluation doesn't happen. Especially with e_error()
and e_warning() it is expected, since it's only some special circumstances
where they don't log the output.

README.m4: Update contact info

README.md: Update http links to https

lib-storage: mail-storage-list-index-rebuild - Don't use continue inside T_BEGIN..T_END

Coverity doesn't like using continue there.

lib-storage: mailbox_set_index_error() - Fix last_internal_error_mailbox leak

Broken by 0a09f7321f00253dfacd87453462e6c43ea926bf

imapc: imapc_mailbox_get_status() - Implement STATUS_UNSEEN with INDEXPVT

lib-storage: LAYOUT=index rebuild - Fix detecting duplicate mailbox GUIDs in index

Otherwise duplicate folders are not properly cleared out.

Broken in 2fc8d7024bb161aab08031b8212a736301844858

lib-storage: LAYOUT=index rebuild - Remove existence check

It's not needed.

lib-storage: LAYOUT=index rebuild - Sort mailbox tree before creating

Try to load the mailbox original name already from storage
so we can sort & create the mailbox tree in correct order.

lib-storage: LAYOUT=index rebuild - Remove first_list usage

Instead, keep track of the list where box was found.

lib-storage: LAYOUT=index rebuild - Remove useless check for LAYOUT=index

This is already done when collecting namespaces.

Simplifies next change.

lib-storage: LAYOUT=index rebuild - Fix rebuilds when using multiple namespaces

The old code did not properly populate the mailbox list for each
namespace, but only for the first namespace. This caused mailbox
storage existence checks to always fail.

This applies for LAYOUT=index only.

Broken in 2fc8d7024bb161aab08031b8212a736301844858

lib-storage: list - Add separators to mailbox names when decoding from index

Mailbox name header decode did not add separators to name,
producing only FooBarBar, when it was supposed to give
Foo/Bar/Baz.

Broken in 8216a079d8aac93e2537ed79485e301350ff3887

lib-storage: Add assert to help scan-build

doveadm sis: Fix "Uninitialized argument value" scan-build complaint

The code was a bit ugly in that it didn't set error_r on ENOENT. There was
no bug because the caller did check for this, but scan-build didn't
understand it.