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.

dict-redis: Fix crash when using synchronous commit

Redis is usually used via dict proxy process, which always uses async
commit, so this didn't matter so much.

lib-storage: Store mailbox names unescaped in box-name header

This header is used when rebuilding mailbox list indexes with LAYOUT=index.
The escape character is currently hardcoded for LAYOUT=index, so this
couldn't have broken anything unless the hardcoded separator was changed.
Still, this was wrong/confusing behavior.

lib-storage: Fix memory leak for last error's mailbox

Broken by 0a09f7321f00253dfacd87453462e6c43ea926bf

lib-storage: mailbox_get_last_internal_error() - Don't report mailbox prefix

Don't report mailbox prefix anymore unless box vname is different from last
error's one mail_storage_get_last_internal_error() will continue to report
it as previously.

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

client-connection-http
client-connection-tcp
doveadm-auth-server
doveadm-dict
doveadm-fs
doveadm-mailbox-list-iter
doveadm-master
doveadm-mutf7
doveadm-penalty
doveadm-pw
doveadm-replicator
doveadm-stats
doveadm-sys

doveadm: dsync - Replace i_<log>() with e_<log>()

dsync-brain
dsync-brain-mailbox-tree-sync
dsync-brain-mailbox-tree
dsync-brain-mailbox
dsync-brain-mails
dsync-mailbox-tree-sync
dsync-mailbox

doveadm: dsync-transaction-log-scan - Replace i_<log>() with e_<log>()

doveadm: dsync-mailbox-tree-fill - Replace i_<log>() with e_<log>()

doveadm: dsync-mailbox-import -  Replace i_<log>() with e_<log>()

doveadm: dsync-mailbox-export - Replace i_<log>() with e_<log>()

doveadm: doveadm-mail-* - Replace i_<log>() with e_<log>()

doveadm-mail
doveadm-mail-altmove
doveadm-mail-copymove
doveadm-mail-deduplicate
doveadm-mail-expunge
doveadm-mail-fetch
doveadm-mail-flags
doveadm-mail-import
doveadm-mail-index
doveadm-mail-iter
doveadm-mail-mailbox
doveadm-mail-mailbox-cache
doveadm-mail-mailbox-metadata
doveadm-mail-mailbox-status
doveadm-mail-save
doveadm-mail-search
doveadm-mail-server

doveadm: doveadm-who - Replace i_<log>() with e_<log>()