lib-storage: Override service->name with input->service when available

lib-storage: mail_storage_service_lookup_real() - Reformat very long line

lib-storage: mail_storage_service_read_settings() - Fix typo in comment

lib-storage: Remove stray whitespaces

lib: test-file-cache - Use RLIMIT_AS only if it's available

doveadm-cmd: Use PRId64 instead of %ld

int64_t is not guaranteed to be long.

doc: Use am__untar to get correct invocation across systems

Current one does not work always.

lib: ioloop-kqueue - Include time-util.h

Needed to compile on MacOS

global: Use ioloop_time32 more when ioloop_time is used as uint32_t

These were missing from 138bee2f638a53415a1729b6a9476eb342b40831.

plugins: Use container_of() in doveadm functions instead of hard casts

doveadm: Use container_of() instead of hard casts

doveadm: Remove doveadm_mail_cmd_context::full_args

doveadm: Refactor doveadm_cmd_ver2_to_mail_cmd_wrapper()

doveadm: Remove doveadm_mail_cmd_vfuncs::parse_arg

doveadm: Remove doveadm_mail_cmd_context::getopt_args

doveadm: Remove doveadm_mail_cmd_context::args

doveadm: Drop parameter args[] from doveadm_mail_cmd_vfuncs::init

doveadm: Remove ::parse_arg() and ::args from doveadm-mail.c handlers

doveadm: Remove ::parse_arg() and ::args from doveadm-dsync.c handlers

doveadm: Remove ::parse_arg() and ::args from most doveadm-mail-*.c files

plugins: Remove ::parse_arg() and ::args from doveadm handlers

doveadm: doveadm_cmd_ver2_to_mail_cmd_wrapper() - Allow v2 args parsing without parse_arg() hook

doveadm: Add doveadm_cmd_param_dump() doveadm_cmd_args_dump() debug functions

doveadm: Add flag CMD_PARAM_FLAG_KEY_VALUE (named positional parameters)

doveadm: Add doveadm_cmd_param_array_append() and doveadm_cmd_param_array_get()

doveadm: Add doveadm_cmd_param_uint64/uint32/int32()

doveadm: Add doveadm_cmd_param_flag()

doveadm: doveadm_fill_param() - Report error on invalid parameters

doveadm: doveadm_cmd_param_*() - Turn type checks into i_assert()s

Attempting to get a paramer by name with a type different from the one declared
in the command block is a logic error. The original behaviors turns that in a
"not found", which can be very misleading to diagnose.

doveadm: Split doveadm-cmd into doveadm-cmd-parse

doveadm: doveadm_cmd_expunge_ver2 - Fix .usage to reflect the actual parsed syntax

doveadm: doveadm_cmd_mailbox_cache_decision - Fix .usage to reflect the actual parsed syntax

lib: Add empty_str_array constant

doveadm: Remove command doveadm batch

No longer relevant and also in the way for next changes.

doveadm: Remove trailing whitespaces

doveadm: Assert-crash if empty referral is attempted to be sent

This shouldn't happen anymore after allocating referral from the context
pool.

doveadm: Allocate doveadm_cmd_context.referral from the context's pool

It's not safe to allocate from data stack.

doveadm: Add doveadm_cmd_context.pool

doveadm: Split off client_handle_command_ctx()

acl: acl_backend_vfile_read() - Drop unused param is_dir_r

acl: acl_backend_vfile_read_with_retry() - Drop logic for .DEFAULT

acl: acl_backend_vfile_refresh() - Drop logic for .DEFAULT

acl: acl_backend_vfile_read_with_retry() - Drop parameter global always FALSE

acl: acl_backend_vfile_refresh() - Remove stray spaces

lib-compression: istream-decompress - Copy parent stream name

This is needed, because istream-decompress doesn't actually use the parent
stream as the istream parent.

fts-flatcurve: Enforce 1st character to be non-uppercase in terms added to Xapian

backported from github xapian issue #27
A Xapian term that begins with a capital letter, like "Jim", is actually
treated by Xapian as the term "im" with the "J" prefix.
Since Xapian interally stores data ignore case, we simply enforce the 1st
letter to non-uppercase to inhibit the prefix processing.

util: Remove maildirlock

stats: stats-service-openmetrics - Always clear the stack before next metric body

If the attempt of generating output for the previous metric doesn’t completely
consume the stack, the next metric will start with a non-empty stack, which breaks
the metrics output.

doveadm: Add dsync_features=no-header-hashes

When this setting is enabled and one dsync side doesn't support mail GUIDs,
there is no fallback to using header hashes. Instead, dsync assumes that
all mails with identical IMAP UIDs contains the same mail contents. This
can significantly improve dsync performance with some IMAP servers that
don't support caching Date/Message-ID headers.

doc: Don't trigger documentation rebuild in make dist tarball

util: dovecot-sysreport - Add timestamps before/after "ps" output

Sometimes running "ps" can take a while, so make it clear in the output if
this has happened.

util: dovecot-sysreport - Wait 3 seconds between "ps" outputs

This allows showing more clearly in ps_output file whether a process is
using 100% CPU.

configure.ac: Use correct path in flex and bison checks

auth: Fix assert-crash in iterating multiple userdbs

Broken by 501e17ba6b448ba3c88338596e0e8f99f0693f79

Fixes:
Panic: file userdb-blocking.c: line 125 (userdb_blocking_iter_next): assertion failed: (ctx->conn != NULL)

doc: Remove build artifacts when bundling the distribution package

doc: Fix typo for default target

Rename the intended default target to the proper name to ensure the html
documentation is built in the default make step. Previously it ran only
when running the install target which yielded files owned by the root
user (if ran with sudo), which in turn couldn't be removed by non-root
users.

submission: Don't log write() errors that fail due to ECONNRESET

These happen only if client initialization fails.

imap: Don't log write() errors that fail due to ECONNRESET

These happen only before the client is fully initialized.

lmtp: Allow reauth to return referral

Previously reauth was required to do proxying on success. Now it can also
return "nologin" and "host" to do a referral instead.

lmtp: Add auth_ prefix to lmtp_proxy_recipient.forward_fields

This makes it easier to differentiate from lmtp_recipient.forward_fields

doveadm: Split incoming and outgoing forward_fields into separate arrays

The outgoing forward_fields should be sent to outgoing proxy connections,
but not used for reauth lookups.

doveadm: Allow reauth to return referral

Previously reauth was required to do proxying on success. Now it can also
return "nologin" and "host" to do a referral instead.

doveadm: Log an error if referral is used with non-tcp client

lib-auth: auth_proxy_parse_redirect() - Return failure if host is empty

lib-doveadm: Fix hang if read() fails from cmd_input

lib-doveadm: Log cmd_input stream name if read() from it fails

lib-doveadm: doveadm_client_cmd() - Don't close cmd_input after reading it

The caller may want to use cmd_input for other purposes as well.

lib-doveadm, doveadm: Add and use enum doveadm_protocol_cmd_flag

doveadm: Fail cleanly if referral host is empty

doveadm: Allow proxy redirects with hostnames when doing reauth

login-common: Allow reauth to return referral

Previously reauth was required to do proxying on success. Now it can also
return "nologin" and "host" to do a referral instead.

login-common: Allow proxy redirects with hostnames when doing reauth

login-common: Add debug logging when doing reauth

login-common: Send forward_* fields to reauth requests

login-common: Don't require "pass" field for reauth proxy requests

The pass is already known, and even if it was returned the current code
wouldn't use it.

m4: Fix flex/bison check for out-of-tree builds

global: Remove wiki from packaging

doc/example-config: Replace wiki references with web documentation

m4: Use documentation project in build pipeline

To replace the wiki as the main documentation in the distribution
package the documentation project needs to be pulled into this project
in a build step. This commit checks the environment (e.g. python/venv)
and downloads/builds the documentation for packaging.

m4: Rename BUILD_DOCS to INSTALL_DOCS

This communicates more clearly that the docs are packaged and installed
and not merely built.

general: Replace online wiki references

For all wiki references that have a currently available documentation
page, replace the link to them appropriately.

lib-master: master-service-haproxy - Correctly align memory

Previous version caused type punning warnings.

lib-master: master-service-haproxy - Remove unnecessary initialization

lib-master: Use pointer for haproxy data

Data does not need to be copied.

Fixes static analyser complaint about writing to const fields.

doveadm-mail-mailbox-cache: Replace indentation spaces with tab

doveadm: Distinguish error status of doveadm_mail_iter_init()

Make doveadm_mail_iter_init() return different status codes depending on
whether the mailbox exists or not to fix a crash when calling `doveadm
mailbox cache remove` with an invalid/non-existing mailbox.

run-test-valgrind.supp: Suppress iconv open failure with EUC-JP.so

lib-dict-backend: Fix crash when accessing shared dict keys with NULL username

director: Use time_to_uint32() to cast time_t to unsigned int where needed

director: Use ioloop_time32 when ioloop_time is used as unsigned int

global: Use time_to_uint32_trunc() to cast time_t to uint32_t where needed

This addresses Coverity complaints.

global: Use time_to_uint32() to cast time_t to uint32_t where needed

This addresses Coverity complaints.

global: Use ioloop_time32 when ioloop_time is used as uint32_t

lib: Add time_to_uint32() and time_to_uint32_trunc()

time_to_uint32() asserts the time_t value fits in uint32_t and
time_to_uint32_trunc() truncates the value.

fts-flatcurve: Fix unused parameter warning

master: Assume service is successful if its process has existed for 10 seconds

master code has a special check to start throttling services that are
rapidly dying and have never had any successful exits. However, this exit
check is a bit bad for services whose processes are supposed to never
exit. So if a service has a process that exists for at least 10 seconds
and has initial status notification sent, handle it the same as if the
service had already once exited successfully.

lib-master: Wait for initial status notification write to finish before continuing

Otherwise if the process dies early, master might think the service is
broken and start throttling it. This would only happen if the initial
write() failed with EAGAIN, meaning master was very busy and didn't have
time to read all the notification pipes.

lib-master: If status write() fails permanently, remove IO_WRITE handler

This prevents rapidly looping to the function in case the IO is set.

master: Add comment about services flooding status notifications

lib-storage: mbox_read_from_line() - Check size given to mbox_from_parse() is positive

This addresses a Coverity complaint.