stats: openmetrics - Set field always when used

Satisfied static analysers.

Broken in ba19a18d54cb7cf7de93d6a235862a408cfe5828

lib: md4 - Fix violation of strict aliasing.

Fix miscompilation when LTO is enabled.

imap: Improve disconnection log message if it happens immediately after unhibernation

The "No commands sent after unhibernation" can mean that imap-hibernate
process saw a timeout while unhibernating and shutdown the connection.
Or it can also mean just that the client itself disconnected.

imap-hibernate: If unhibernation fails, make sure imap process won't finish it later on

Especially if unhibernation fails due to a connection timeout to imap
process due to high load, it's possible that the imap process will
eventually finish the unhibernation and continue with the client. This is
rather confusing, since imap-hibernate process already logged that the
client got disconnected.

imap: Disconnect imap-master client if it's not sending anything for 25 seconds

lib-imap-urlauth: imap-urlauth - Make sure a host name is available in imap_urlauth_check_hostport().

Addresses FIXME.

lib-imap-urlauth: imap-urlauth - Make struct imap_url parameters const.

imap-urlauth: imap-urlauth-worker-client - Use the input_args handler of the connection API.

imap-urlauth: imap-urlauth-worker-client - Use generic line input handling from connection API.

imap-urlauth: imap-urlauth-worker - Fully perform client-to-server VERSION handshake with connection API.

imap-urlauth: imap-urlauth-worker - Perform client-to-server VERSION handshake in separate function.

imap-urlauth: imap-urlauth-worker - Enable connection API's server-to-client VERSION handling.

imap-urlauth: imap-urlauth-worker - Use connection API's client-to-server VERSION handling.

Requires reordering sending FDs with VERSION line.

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

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

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.