lib: events - Use const event pointer for "get" functions

lib-storage: mail_user_init_fs_settings() - Use user's event as fs parent event

auth: test-mech - Fix type mismatch

auth: test-mech - Fix APOP challenge format

submission: submission-backend-relay - Handle relay server replies consistently.

Initial connection setup errors were handled differently when the reply came
into the ready callback first.

lib-smtp: test-smtp-client-errors - Add test for server presenting invalid SSL certificate.

lib-smtp: smtp-client-connection - If SSL connection gets lost, log the last SSL error.

lib-smtp: smtp-client-connection - Handle connection loss uniformly at a single location.

Also amend the logged error message with SSL information when relevant.

lib-smtp: smtp-client-connection - Properly log SSL handshake failure occuring during input read as an error.

lib-smtp: smtp-client-connection - Restructure handling of immediate connection failure.

The delayed error callback is now started at a single location, much like
lib-http/client deals with this situation.

master: Make sure a listener isn't added to a throttled service

This fixes master process using 100% CPU in some situations when a service
is being throttled.

auth: mech-gssapi - GSSAPI can contain NUL.

imap: Add "mailbox" field to successful imap_client_unhibernated event

imap-hibernate: Add "mailbox" field to events

imap, imap-hibernate: Track how long hibernation lasted

imap-hibernate: Add imap_client_unhibernate_retried event

imap-hibernate: Add imap_client_unhibernated event for errors

Successful unhibernation is logged by imap process.

imap-hibernate: Handle unhibernation errors more consistently.

Log an error with the exact reason, and just log "Failed to unhibernate
client" as the info-level disconnect reason.

imap-hibernate: Fix error handling when unhibernation retry timeout is reached

An error message wasn't logged, and a wrong kind of message was logged in
the info line.

imap-hibernate: Split off imap_move_has_reached_timeout()

imap: Add imap_client_unhibernated event

imap: Send unhibernation OK reply to imap-hibernate only after creating client

The following commit adds unhibernation event, which would be much more
difficult to implement for this error path without this change.

The OK reply is still sent before client_create_finish(), which is the part
that could be slow.

imap: Set unhibernation IDLE state earlier

This simplifies the following changes.

imap: Add imap_client_hibernated event

imap-hibernate: Log via imap_client.event or connection.event

imap-hibernate: Add imap_client.event

imap-hibernate: Preserve local_port and remote_port during hibernation

imap: Log ID and logout info lines via imap_client.event

This doesn't change the logging output at all.

imap: Log imap-master errors/debugs via connection.event

imap: Log errors via imap_client.event when possible

imap: imap_hibernate_process_send() - Return error instead of logging it

lib: Add str_to_timeval()

director: Use director_connection.event wherever possible

This adds the "director(name): " log prefix to many log messages where it
didn't previously exist. The director name is now sometimes duplicated in
the log messages, but maybe that's good for clarity.

director: Avoid explicit log prefixes by logging via director_connection.event

director: Add director_connection.event

director: Add director_connection_set_name() change connection name

director: Replace most of the i_error() logging with e_error()

director: Replace i_warning() and i_info() logging with e_*()

director: Add auth_connection.dir

director: Add user_directory.director

director: Add mail_host_list.dir

director: Remove dir_debug() and director_debug

director: director-request: Add event and replace dir_debug() with e_debug()

director: Replace most of the dir_debug() with e_debug()

director: Add struct director.event

lib: Fix compilation with OpenSSL deprecated APIs disabled

OpenSSL with no deprecated APIs does not implicitly include header files.

lib: Always use the representative when comparing event's categories

We need to be consistent about using the representative category pointer.
Otherwise, we may end up with duplicates in an event's categories array.

Since an event's categories are conceptually a set, it is reasonable to
write event using code that assumes no duplicates.  If duplicates are
present, it may lead to incorrect behavior or even crashes.

lib: event-filter-parser - Use str_begins() to compare unit test error strings

Different versions of bison generate slightly different error strings, but
they always begin the same way.  So, instead of comparing them let's only
compare the beginning.

lib: event-filter-parser - YYERROR_VERBOSE has been deprecated since 2003

The proper way to enable verbose error messages is to use the parse.error
define.

lib: Add unit test for long log line splitting

lib-index: Remove log prefix from cache events

Index events don't have a prefix either. The parent event likely already
contains the mailbox name, which should be enough. It's especially bad
using the full filesystem path in the event prefix, because it could be
very long.

global: Make sure event log prefixes are sanitized and not excessively long

It doesn't make sense to allow log prefixes to grow to kilobytes long,
especially from user-given input.

lib: Fix sending log lines when prefix is larger than PIPE_BUF

This caused the log line to be sent repeatedly to the log process, possibly
causing hundreds of duplicate log lines.

log: log_connection_handshake() - Handle i_stream_read()'s -2 return value properly

Although this shouldn't happen anymore after the previous change.

log: Fix busy loop when a long line is logged

If the line exceeded PIPE_MAX (4 kB usually), the process got into a busy
loop. It would still break out of it every 100ms and maybe handle some
other services' logging, but the service that logged the long line would
be stuck. This resulted in all those service processes also hanging because
they were waiting to be able to write to log.

auth: test-mech - Add tests for RPA and NTLM bug

lib-ntlm: Check buffer length on responses

Add missing check for buffer length.

If this is not checked, it is possible to send message which
causes read past buffer bug.

Broken in c7480644202e5451fbed448508ea29a25cffc99c

auth: mech-rpa - Fail on zero len buffer

lib-mail: Fix handling trailing "--" in MIME boundaries

Broken by 5b8ec27fae941d06516c30476dcf4820c6d200ab

lib-mail: message-parser - Support limiting max number of MIME parts

The default is to allow 10000 MIME parts. When it's reached, no more
MIME boundary lines will be recognized, so the rest of the mail belongs
to the last added MIME part.

lib-mail: message-parser - Support limiting max number of nested MIME parts

The default is to allow 100 nested MIME parts. When the limit is reached,
the innermost MIME part's body contains all the rest of the inner bodies
until a parent MIME part is reached.

lib-mail, global: message_parser_init*() - Convert flags to settings structure

lib-mail: message-parser - Don't use memory pool for parser

This reduces memory usage when parsing many MIME parts where boundaries are
being added and removed constantly.

lib-mail: message-parser - Add boundary_remove_until() helper function

lib-mail: message-parser - Optimize boundary lookups when exact boundary is found

When an exact boundary is found, there's no need to continue looking for
more boundaries.

lib-mail: message-parser - Truncate excessively long MIME boundaries

RFC 2046 requires that the boundaries are a maximum of 70 characters
(excluding the "--" prefix and suffix). We allow 80 characters for a bit of
extra safety. Anything longer than that is truncated and treated the same
as if it was just 80 characters.

lib-mail: message-parser - Minor code cleanup to finding the end of boundary line

lib-mail: message-parser - Optimize appending new part to linked list

lib-mail: message-parser - Optimize updating children_count

lib-mail: message-parser - Change message_part_append() to do all work internally

lib-mail: message-parser - Add a message_part_finish() helper function

lib-mail: Move message_parser_init_from_parts() handling to its own file

This helps to see what code they have in common.

lib-mail: test-message-parser - Test that children_count is correct

lib-mail: test-message-parser - Add another test for boundary matching

dict: Fix potential hang when iterating

Broken by 2937287a72cffa8c709c22ede838058d48b61778

lib: Clarify o_stream_set_flush_pending() comment related to corking

lib: When clearing signal handlers, ignore the signals instead of restoring defaults

This is mainly happening in while the process is shutting down. Getting a
signal at a time when the normal signal handlers are gone isn't something
that is expected. It's better to just ignore the signal and let the process
to finish shutting down normally instead of getting killed.

This should also fix some unit tests where the child processes were
sometimes dying with SIGTERM due to these race conditions.

pop3-login: Fix the next command failing after unsuccessful AUTH

If the AUTH command failed (e.g. bad password or proxying failure), the
next command was still processed as AUTH command. For example:

auth plain .
-ERR [AUTH] Invalid base64 data in initial response
quit
-ERR [AUTH] Unsupported authentication mechanism.

lib-compression: zstd - Check version

If we are using older than 1.3.1 we emit warning if there is
library mismatch. This is because error codes will break.

lib-compression: zstd - Repair error codes if necessary

libzstd version numbers were pinned on 1.3.1, if we are
compiled against version before that, and runtime is
newer, we need to repair version numbers.

A horrible hack that only allows using 1.3.1+ with old
code.

lib: Fix potential crash changing signal handlers after destroying ioloop

lib-index: Set event name for "Recreated index"

global: Rely on fs_deinit() automatically freeing parent fs

lib-fs: Free parent fs generically

mail-crypt: Use container_of() instead of casts for fs code

Being more explicit not only makes the code more explict, it makes it safer.

fs-compress: Use container_of() instead of casts

Being more explicit not only makes the code more explict, it makes it safer.

lib-fs: sis - Use container_of() instead of casts

Being more explicit not only makes the code more explict, it makes it safer.

lib-fs: sis-queue - Use container_of() instead of casts

Being more explicit not only makes the code more explict, it makes it safer.

lib-fs: randomfail - Use container_of() instead of casts

Being more explicit not only makes the code more explict, it makes it safer.

lib-fs: metawrap - Use container_of() instead of casts

Being more explicit not only makes the code more explict, it makes it safer.

lib-fs: Add new deinit vfunc

This is useful for tidying up anything just before freeing the struct fs.
For example, flushing dirty data in a write-back cache.

lib-fs: Rename deinit vfunc to free

lib, lib-master: Send updated event to stats if fields/categories change

Based on code by Jeff Sipek

stats: Allow clients to update existing events with UPDATE command

lib: event_import*() - Allow importing duplicate data without using more memory

lib-master: Properly compare unit test output strings with expected values

The I_MIN() mean that only the prefix was getting compared.

imap: cmd-notify - Do not try to notify for removable namespaces.

It is not going to work anyway and it will cause a crash when the IMAP sync code
subsequently drops those namespaces as part of mail_user_drop_useless_namespaces().
This does not drop the earlier created mailbox_list_notify object, causing its
mailbox list index view object to remain open. This causes the following panic:

Panic: Leaked view for index <path>/dovecot.list.index: Opened in mailbox-list-index-notify.c:142

lib-storage: Add mail_namespace_is_removable() and use it.

lib-index: mail-index-view - Record caller source filename and line for mail_index_view_clone/dup_private().

Before, this was only recorded for mail_index_view_open(), which caused the view
leak panic to show a useless source location, e.g.:

Panic: Leaked view for index <path>: Opened in (null):0

lib-fs: Set debug on event from settings