doveadm-server: Fix potential hangs with SSL connections

auth: Don't include LIBSODIUM_LIBS in dependencies

Put it into AUTH_LIBS among other libraries that are used by auth.

lib-smtp: server: STARTTLS command: Call smtp_server_connection_set_ssl_streams() after calling the conn_start_tls() connection callback.

This is called instead smtp_server_connection_set_streams(), so that the connection is now marked as SSL-secured.
This fixes a submission-login bug that prevented client login when ssl=required.

lib-smtp: server: Add smtp_server_connection_set_ssl_streams().

It wraps smtp_server_connection_set_streams() and additionally marks the connection as SSL-secured.

lib-smtp: server: Change conn_start_tls() connection callback to return a success status.

man: Update version number to v2.3

lib-smtp: server: Fixed assertion in smtp_server_connection_next_reply() checking whether command is fully replied.

It did not handle LMTP DATA/BDAT command correctly, for which there can be multiple replies.

This fixes mixing local/remote recipients in LMTP, which used to crash:
Panic: file smtp-server-connection.c: line 610 (smtp_server_connection_next_reply): assertion failed: (cmd->state == SMTP_SERVER_COMMAND_STATE_READY_TO_REPLY && smtp_server_command_is_replied(cmd))

lib-smtp: server: Fix compile error in smtp-server-cmd-helo.c.

Error was:
"smtp-server-cmd-helo.c", line 140: void function cannot return value

lib-smtp: command parser: Fix error recovery.

Input characters were not skipped upon error, meaning that error recovery would encounter these again.
This would make the parser parse the same data over and over again, yielding the same error.
This in turn caused the SMTP server application to hang in an infinite input loop, submitting error replies in the process, thereby filling up the process memory until exhaustion.
Problem seen in submission service, but LMTP is also affected.

lib-smtp: server: Fix panic when disconnecting client aborting commands

This happened at least if the relay server simply disconnected without
sending any output.

Fixes: Panic: file submission-commands.c: line 39: unreached

lib-smtp: smtp_server_connection_pending_command_data() - don't crash if smtp_parser is NULL

doc: example-config: Properly document SSL configuration for submission relay server.

submission: Fix memory leaks at deinit when submission_relay_host setting is missing

lib-smtp: server: Fix panic when io_remove() is called too late

With submission-login the conn_disconnect() callback ends up in
login-common's client_disconnect(), which closes the file descriptor.
The io_remove() was done afterwards, which caused a crash:

submission-login: Panic: epoll_ctl(del, 18) failed: Bad file descriptor

lib-smtp: server: Fix infinite loop when client disconnects in initial state

old-stats plugin: Rename stats_* settings to old_stats_*

auth: Install stats plugin to old-stats/

Fixes using auth_stats=yes

lib: Fix using appended log prefixes for events when full prefix isn't replaced

lib: Fix using failure_context.log_prefix with internal log process communication

log: Add "disable log prefix" flag to log lines

lib: Add event_send()

lib: Rename event_send() to event_vsend()

lmtp: local: Properly handle a user being over quota.

Recent changes caused quota enforcement to be ineffective.

lmtp: local: Make sure pending recipient is always destroyed.

Before, the recipient was not freed when the server/connection ended prematurely.

login-proxy: Add back support for ssl_client_cert/key

Forgot to add these while rewriting login-proxy to use lib-ssl-iostream.

lib-ssl-iostream: If allow_invalid_cert=TRUE, skip verifying hostname

lib-master: Fix master_service_ssl_settings_to_iostream_set() for client settings

ssl_verify_client_cert setting applies only to server side. For client side
we always verify the SSL certificate validity.

lib-http: Add http_client_init_private()

This allows creating a HTTP client without a shared context, in case it's
needed for some reason.

lib-http: Change http_client_init() to use the global shared client context

This is most likely what is usually wanted.

lib-http: Add http_client_get_global_context()

fts: Code cleanup

fts: Don't try to retry parsing if indexing already fails

fts: Use consistent naming for retriable_err_msg

It's only for retriable errors, not errors in general

fts: Make sure may_need_retry is reset between attempts.

It wasn't if the second attempt failed early.

fts: Add assert to make sure fts_parser.deinit() returns retriable_error_msg when needed

fts: fts-parser-tika - Add assert to make sure retriable_error_msg is set

fts: fts-parser-script - Fix deinit return value

fts: fts-parser-html - Fix deinit return value

fts: fts_parser_deinit() - Fix default return value

fts: Add comment to fts_parser_deinit() describing its API

lib: Fix static analyzer warning about uninitialized va_list

lib-storage: Delay error handing for invalid postmaster_address setting

cassandra: Fix crash when using prepared statements

Broken by cae3c6903c10682429f9849ab1a25093e3f538e4

*-login: Avoid unnecessary "OK Waiting for authentication process to respond"

These happened with SSL connections when the process was starting up.
The ssl-istream was triggering the IO callback, which was thinking that the
client was sending a command. If this happened early on before auth process
connection was finished, this caused several unnecessary notifications to
the client.

global: Finish fixing oldstats plugin/service rename

lib-http: client: Check whether client is running in a wait loop with the correct condition.

This used to check client->ioloop != NULL, but now it should check client->waiting.
Previous commit (e8a1b62fe4a81b211dcccd1a58b44f254074eab6) omitted a few instances where this should have been changed.

lda: Do not replace the NULL sender with a default <MAILER-DAEMON> address.

Only do this substitution locally where a NULL sender would cause problems, not everywhere else.
This e.g. causes problems when determining whether to send a rejection somewhere; attempts would be made to mail <MAILER-DAEMON>.
Also other envelope-based (Sieve) tests would get confused.

lib-lda: Properly check whether MAIL FROM is the NULL sender <>.

There's a function for that, since comparison with NULL pointer is not sufficient.

lib-master: stats-client - Fix reconnection if handshake wait fails

Fixes:
Warning: Timeout leak: 0x5555556ac8d0 (stats-client.c:132)

submission-login: Fix another NULL dereference occurring at an invalid reply from the backend server.

The text variable was still being used when it could be NULL. Made the offending code dependent on the reply validity check.

Problem found by Coverity.

oldstats plugin: Fix init/deinit function names

lib-storage: mailbox_list_index - Don't overwrite INBOX's flags with prefix/INBOX's

This made the real INBOX appear as if it was \NoSelect

lib-storage: mailbox_list_index - indentation cleanup

lib-storage: mailbox_list_index - List prefix/INBOX if it necessary

LAYOUT=fs: Fix listing prefix/INBOX

Removed some confusing special case code that didn't seem to work very well.
Implemented this now properly so that prefix/INBOX is listed as \NoSelect
mailbox whenever it has children.

It's not actually possible to differentiate between INBOX and prefix/INBOX
in the storage for a inbox=yes namespace, because they both are converted
into the same storage_name=INBOX.

LAYOUT=fs: List INBOX as \NoInferiors when necessary

LAYOUT=index: List INBOX as \NoInferiors when necessary

LAYOUT=Maildir++: List INBOX as \NoInferiors when necessary

lib-storage: Add mail_namespace_is_inbox_noinferiors()

imapc: Don't skip listing "INBOX" when INBOX/ namespace prefix is used.

The INBOX was still listed as part of the autocreated mailboxes, unless
MAILBOX_LIST_ITER_NO_AUTO_BOXES was used.

lib: mempool-allocfree - make static analyzer happier with pool clearing

It was thinking that already freed memory was being accessed.

dbox: Fix creating dbox-alt-root symlink when mail root directory doesn't exist

Broken by 74664ac6dcf88cfc2ae6378c87c20ba694385a23

lib-http: client: peer: Properly handle claiming an existing connection.

If no other connections were being created, request handling was not restarted.
This problem sometimes caused the client to hang.
Fixed by calling http_client_peer_connection_success() upon claiming an existing connection.

lib-http: test-http-payload: Terminate the test if it is hanging for some reason.

lib-http: test-http-payload: Amend shared client tests to do both download and echo tests.

Also, the original download tests were named wrong.

lib-http: test-http-payload: Fixed indent problems.

doveadm stats dump: Add -r parameter to reset stats after dumping

stats: Add DUMP-RESET command to reset stats after dumping them

lib-storage: Delay creating mail root directory until it's necessary

lib-storage: Make sure index root is created when it's the same as root directory

This is required by the following commit.

lib-http: client: Made http_client record the current ioloop it is switched to.

This prevents http_client_wait() from switching the client to an ioloop it was never explicitly switched to.

imap: Fix o_stream_is_corked() assert check

All the callers cork the stream, so this can only mean that the corking
failed because the connection was already closed. Continue handling the
input anyway.

Fixes:
Panic: file imap-client.c: line 1236 (client_handle_input): assertion failed: (o_stream_is_corked(client->output))

LAYOUT=index: Fix crash in doveadm force-resync if storage doesn't implement list_index_rebuild()

Broken by recent changes.

LAYOUT=index: Fix updating STATUS changes in mailbox list index

Mailbox list index backend code was overriding sync_init and sync_deinit
methods, which STATUS handling had already already overridden. They both
used the same super struct, so STATUS's sync_* were never called.

lib-master: Fix logging "Expected FILTER" error

stats: Fix logging unknown client command error

.gitignore: Ignore src/old-stats/old-stats

lib-http: Add named event: http_request_finished

doveadm: Add new "stats dump" command

global: Enable MASTER_SERVICE_FLAG_SEND_STATS for mail processes

lib: connection - When receiving invalid VERSION line, log the line contents

log: Fix restoring global log prefix

It was allocated from data stack, which was freed by
master_service_init_finish(). This may have caused wrong prefix to be
used for log's internal error messages, or restoring the global log
prefix could have crashed:

Panic: file strfuncs.c: line 147 (t_noalloc_strdup_vprintf): assertion failed: ((unsigned int)ret == *size_r-1)

lib-http: client: Replace http_*_debug() with e_debug()

lib-http: client: Remove redundant debug logging checks

lib-http: client: Add support for event API

lib-http: Cleanup - Add name parameter to http_client_host_create()

lib-fs: Replace i_error() calls with e_error()

lib-fs: Add fs_file/iter_init_with_event()

Use the event for logging critical errors

lib-fs: Add fs_file/iter_init_parent()

lib-fs: Set fs_iter.flags immediately in fs_iter_init()

This way backends don't need to set it.

lib-fs: Split allocation from fs.iter_init() to a separate fs.iter_alloc()

lib-fs: Split allocation from fs.file_init() to a separate fs.file_alloc()

lib-master: Add stats-client

Enable it if MASTER_SERVICE_FLAG_SEND_STATS is set

stats: Add new stats service

imap: Add imap_command_finished event.

This can be used to get per-command statistics.

imap: Add client_command_context.event and use it as global event while running

imap: Add client.event and use it as mail_*user's parent event

lib-index: mail_index_set_error() - log error using index's event

lib-index: Add mail_index.event

lib-storage: mail_storage_set_critical() - log error using user's event

Fix test-storage unit test to also initialize mail_user.event so it won't
crash.