imap-urlauth: imap-urlauth-client - Connect worker client last in client_create().

In later commit the worker client will depend on full unitialization of main
client struct.

imap-urlauth: imap-urlauth-worker-client - Use the connection.h API.

imap-urlauth: imap-urlauth-worker-settings - Use socket name definition for default.

imap-urlauth: imap-urlauth-worker-client - Use version definitions to compose handshake.

imap-urlauth: imap-urlauth-client - Move protocol version definitions to imap-urlauth-common.h.

imap-urlauth: imap-urlauth-worker - Move protocol version definitions to imap-urlauth-worker-common.h.

imap-urlauth: imap-urlauth-worker-client - Create sub-event for the worker.

imap-urlauth: imap-urlauth-worker-client - Make the worker a separate struct.

imap-urlauth: imap-urlauth-worker-client - Rename functions to have the proper prefix.

imap-urlauth: imap-urlauth-worker-client - Rename most client variables to wclient.

imap-urlauth: Make imap-urlauth-worker-client compile as a separate module.

imap-urlauth: Move worker client to separate source file.

imap-urlauth: imap-urlauth-client - Rename to_idle to to_destroy.

Makes more sense.

imap-urlauth: imap-urlauth-login - Send server-to-client VERSION line.

lib-imap-urlauth: imap-urlauth-connection - Use the connection.h API.

lib-imap-urlauth: imap-urlauth-connection - Move sending of authentication command to separate function.

lib: ostream-file - Improve write errors: mention the failing syscall.

imap-urlauth: Reformat imap-urlauth-client.h.

imap-urlauth: Reformat imap-urlauth-client.c.

lib-imap-urlauth: Reformat imap-urlauth-connection.c.

lib-imap-urlauth: Reformat imap-urlauth.h.

lib-imap-urlauth: Reformat imap-urlauth.c.

lib-http: http-server-connection - Fix segfault occurring when output is resumed on closed connection.

login-common: client_get_extra_disconnect_reason() - Add shutting_down and process_full reasons

login-common: Add shutting_down and resource_constraint fields to client struct

login-common: Remove no_extra_disconnect_reason

This got obsolete by previous changes and prevented properly set reason
for the login_aborted event on some internal failures.

login-common: client_disconnect() - Handle NULL reason separately

login-common: login_aborted_event() - Add more event fields

login-common: Change struct client's auth_first_started and auth_finished to timeval

login-common: client_disconnect() - Split off login_aborted_event()

Adds the login_aborted event.

login-common: client_disconnect() - Split off client_disconnected_log()

login-common: client_get_extra_disconnect_reason() - Remove auth_waiting_server and auth_waiting_server_finish

These reasons are not supposed to happen only if a process gets actively
killed. For the shutting_down reason will be added.

login-common: client_get_extra_disconnect_reason() - Increase auth_aborted_by_client reason priority

login-common: Disconnect reason - Add event reasons

login-common: Disconnect reason - clarify client sending untrusted cert

Changed the wording from "invalid cert" to "untrusted cert", which is
is a bit clearer in what the rejection reason usually is.

login-common: Disconnect reason - Return auth counts and secs for most replies

Earlier some reasons weren't returned when there were more than one attempt,
because it would hide if there were a lot of auth attempts. Now that the
auth attempt count is always returned, these less important reasons can also
be returned.

login-common: Use a reason string array for client_auth_fail_code

login-common: Disconnect reason - Add missing CLIENT_AUTH_FAIL_CODE_ANONYMOUS_DENIED

login-common: Add comments

login-common: Disconnect reason - Move auth_process_comm_fail check earlier

Likely doesn't matter, but this is more important reason to be logged than
the ones coming later.

login-common: Disconnect reason - Clarify "auth aborted by client" handling

Rename client.auth_try_aborted to auth_aborted_by_client and clarify the
disconnect reason message.

login-common: Disconnect reason - Check for missing SSL certs before auth attempts

login-common: client_get_extra_disconnect_reason() - Remove parenthesis from returned string

The caller now adds the parenthesis in one place.

login-common: Rename client.auth_waiting to auth_client_continue_pending

login-common: Reset auth_try_aborted if a new auth is started

This only affects the disconnection log message. It is likely better to
forget about this state when a new auth attempt begins, since if the new
auth fails also it is likely something completely different.

stats: openmetrics - Create metrics for all specified fields

lib: fd-util - Try to read link from procfs to see which file leaked

Makes it easier to determine why the leak occured.

lib-storage: Check return status from mailbox_create() in tests

This addresses Coverity complaints.

master: Don't pass config file in two fds to child processes

Broken by 1955e81a2ffe023f937d87f2b555bcec0fb4aee9

lib-storage: Update "user" field in mail_storage_service_user.event after userdb lookup

The userdb lookup may change the username.

stats: Add support for discrete modifiers

stats: Populate group_by field earlier in settings check

This is needed in next commit to avoid crash.

pop3: Add pop3_command_finished event

pop3: Add pop3 command context

lmtp: Add smtp server statistics to user event

lib-smtp: Include net_in_bytes, net_out_bytes in event

Ensure they end up in connection finished event

lib-smtp: Move struct smtp_server_stats

This is needed for next commit

lmtp: Remove useless cast

submission: Include net_in_bytes, net_out_bytes in event

These fields are mainly intended for the mail_user_session_finished event.

pop3: Include net_in_bytes, net_out_bytes in event

These fields are mainly intended for the mail_user_session_finished event.

imap: Include net_in_bytes, net_out_bytes in event

These fields are mainly intended for the mail_user_session_finished event.

pop3: Use i_stream_get_absolute_offset() in stats

Consistent with other protocols

imap: Rename bytes_in,_out to net_in_,out_bytes in cmd stats

login-common: Rename bytes_in,_out to net_in_,out_bytes

lib-http: Rename bytes_in,_out to net_in_,out_bytes

lib: connection - Rename bytes_in,_out to net_in_,out_bytes

lib: connection - Expose connection_update_counters()

dbox: dbox_cleanup_temp_files() - Fuzz scan interval adding 0..+30% based on username hash

dbox: dbox_cleanup_temp_files() - Reformat code

dbox: dbox_cleanup_temp_files() - Remove unnecessary else/else if after returns

dbox: dbox_mailbox_open() - Drop unused ctime

dbox: dbox_mailbox_open() - Move dbox_cleanup_temp_files() to dbox_mailbox_close()

dbox: dbox_mailbox_open() - Infer last_temp_file_scan from dir's atime if 0

dbox: dbox_mailbox_create() - Update last_temp_file_scan

lib-storage: dbox_cleanup_temp_files() - Fix inverted check for ENOENT

doveadm: Panic if doveadm_print_header() is called too late

Otherwise it's not noticed unless the doveadm command is called for multiple
users (-A, -F).

doveadm: Panic if doveadm_print_init() is called too late

Otherwise it's not noticed unless the doveadm command is called for multiple
users (-A, -F).

fts-flatcurve: Move doveadm_print_header() eariler

Otherwise the newly introduced checks trigger faults

fts: fts tokenize - Move doveadm_print_init() eariler

Otherwise the newly introduced checks trigger faults

mail-crypt: Move doveadm_print_init/header() eariler

Otherwise the newly introduced checks trigger faults

doveadm: pw - Allow weak schemes in verification

auth: mech-gssapi - Remove KRB5_USEROK check

It should be there always these days. Enables cross-realm
and cross-identity authentication always.

sdbox: Fix panic due to leaked mail_index_view if sync fails

Fixes:
Panic: Leaked view for index .../dbox-Mails/dovecot.index: Opened in mail-index-sync.c:445

lib-index: Don't update log_file_tail_offset on sync if reading dovecot.index.log failed

This is more of a theoretical issue, since dovecot.index.log read failure is
very unlikely.

lib-index: Don't update max_tail_offset if write to dovecot.index.log fails

The max_tail_offset didn't actually increase in that situation.
This fixes wrongly updating mail_index_header.log_file_tail_offset after
write to dovecot.index.log failed (due to out of disk space), which in
turn can cause some unexpected errors.

lib-index: Fix event leak if purging dovecot.index.cache fails

lib: event-filter-parser.y - Add workaround for nerrs being unused warning

Fixes with clang-17:
warning: variable 'event_filter_parser_nerrs' set but not used

global: Remove dead code

lib: istream - Don't call memcpy() with NULL parameter --with-devel-checks

lib: lib - Add value for defines that don't have it

Fixes compilation with FreeBSD that requires a value for
defines.

Broken in e6f2682b32ff5b366ee7242841c3194425202880

lib: lib - Remove memcpy macro in C++

This breaks C++ compilation due to too wide impact.

Broken in 38dbeec767ad10021020e3833b1dbc3d04442f85

lib-auth: Remove external libraries from dependencies

This just breaks compilation when they contain -L.

Broken in 5d632abbf571f8e9157b1526533d677a1b8eb829

lib-mail: Remove obsolete mail-user-hash

login-common: Remove obsolete include of mail-user-hash.h

fts: cmd_fts_expand_run() - Fix for crash invoking doveadm fts expand -A

imap: imap_parser_read_next_atom() - Reject special character DEL 0x7F

atom-specials includes CTL, which in turn contains \x00-\x1F and \x7F DEL

imap: imap_parser_read_next_atom() - Accept "]" in cmd-tag(s)

despite "]" being in atom-specials, a specific exception is made
for that in ASTRING-CHAR, which is used by tag

lib-imap-client: Fix crash if SSL initialization fails when connecting

This could be for example due to invalid SSL certificate settings.

lib-imap-urlauth: imap-urlauth - Don't return errors from imap_msgpart_url*() as internal errors.

These are explicitly suitable as client errors; e.g. about nonexistent folders.

lib-imap-urlauth: imap-urlauth - Don't replace error returned by imap_msgpart_url_open_mailbox()

The original error is intended to be sent to the client.