lib: Fixed memory leak io_add_istream()+io_remove() when the stream didn't have fd.

lib: iostream-temp allows now calling o_stream_seek() on it.

doveadm: A bit nicer way to implement doveadm_cmd_param_array() returning C-string array

So replacement of 4afc67eb9, which I accidentally pushed before remembering
about this other way.

doveadm-kick: Rewrite to ver2 infra

doveadm-kick: Print kicked usernames via doveadm-print API

doveadm-who: Rewrite to ver2 infra

doveadm kick: Don't show who command's help on duplicate network/username masks

doveadm: Changed doveadm_cmd_param_array() to return C-string array instead of ARRAY()

We could support both with separate functions, but perhaps this one is enough.

doveadm-dict: Use ver2 structures

doveadm-dict: Remove i_fatal() usage

doveadm-fs: Use doveadm_print instead of printf

doveadm: Renamed long parameter names.

doveadm mailbox metadata list: Added missing mailbox parameter

lib-storage: Use getuid() instead of geteuid() for checking if we're running as root.

This fixes running multiple doveadm commands for different users. Running
mail_storage_service_init() + _deinit() multiple times left euid non-root,
which dropped the MAIL_STORAGE_SERVICE_FLAG_TEMP_PRIV_DROP flag.

doveadm-server: Drop privileges only temporarily when running mail commands.

This allows running mail commands for multiple users within the same doveadm
connection.

doveadm-http: Fixed crash on unknown commands.

Also if command had already failed, don't override the existing method_err with 403.

doveadm-mail-index: Use integer parameter type

lib-storage: When cached mime.parts is detected to be inconsistent, log it as hex-encoded.

lib-storage: If cached mime.parts is corrupted, log it hex-encoded.

lib-mail: message_search_msg() passes through message_parser_deinit_from_parts()'s error string

Use mail_set_cache_corrupted_reason() wherever possible.

lib-storage: mail_set_cache_corrupted*() now logs also the mailbox name that was corrupted.

lib-mail: Minor code cleanup to message_parser_deinit() calls.

lib-mail: Added message_parser_deinit_from_parts()

The returned error strings could still be improved to give more details,
especially give an approximate offset of where the problem is.

lib-storage: Added mail_set_cache_corrupted_reason()

*-login: If verbose_ssl=yes, make sure all SSL errors are logged.

They should already be logged in the client's disconnection line, but log
them also with debug level just to make sure no errors are lost or
overwritten.

lib-ssl-iostream: If verbose=TRUE, internally log all errors.

lib-ssl-iostream: Log all SSL info lines with debug-level.

They're not logged at all if verbose=FALSE, so they're really debug messages.

doveadm-fs: Use ver2 structures

doveadm-cmd: Support non-strings in compat wrapper

doveadm-mail: Rename ctx to mctx to separate from cctx

doveadm: Changed v2 command APIs to be easier to use.

doveadm: Minor code cleanup - use arg-> instead of argv[i].

doveadm: Code cleanup - Removed unnecessary pointer checks.

Originally for some reason added by e16cdc182.

doveadm: Fixed command parameter handling for doveadm-server.

doveadm: Add more consts to avoid a cast breaking strict-aliasing rules.

doveadm: Fixed parsing field/flag/file parameters.

doveadm-log: Fixed trimming empty prefixes.

Too eager (long) cast removal from the original code broke it. Changed to
a slightly simplified version now.

doveadm: Pass through mail_storage_service_input fields in doveadm_cmd_attributes

doveadm: Added struct doveadm_cmd_attributes, which is passed around instead of argc/argv

doveadm-penalty: Convert to ver2 infra

doveadm-flags: Support giving flags as an array.

doveadm-copymove: Use ver2 structures

doveadm: Code cleanup - coding style fixes for doveadm_cmd_param*() accessors

doveadm-cmd: Add CMD_PARAM_IP

doveadm-mailbox-status: Fixed declaring and processing args

doveadm: Fully populate mail_cmd in ver2 wrapper

doveadm-fts: Use ver2 structures

doveadm-acl: Use ver2 structures

doveadm-search: Use ver2 structures

doveadm-purge: Use ver2 structures

doveadm-force-resync: Use ver2 structures

doveadm-import: Use ver2 structures

doveadm-flags: Use ver2 structures

doveadm-expunge: Use ver2 structures

doveadm-deduplicate: Use ver2 structures

doveadm-batch: Fixed it to work with ver2 commands.

doveadm-altmove: Use ver2 structures

doveadm: Remove return value from ver2 cmd() - doveadm_exit_code is enough

doveadm-cmd: Constify doveadm_cmd_param_*() usage

doveadm-log: Use ver2 structure

doveadm-log: Use doveadm_print() for output

doveadm-print: Add "formatted" print formatter

This allows using the header names as %{variables} in the format string.

*-login: Call client.auth_result() also when proxying

lib-http: client: Decreased request/connect timeout for test-http-client to 10s.

lib-http: client: Gave request, connect and dns timeouts defaults that make more sense.

lib-http: client: Fixed crash happening when connection hits an error while handling a new request.

At this point the connection dies within the peer request handler loop.
Before, the dying connection could free the peer when it ended up being
obsolete. But because it was still in the loop, a segfault occurred as the
loop continued. Fixed by deferring the the deallocation of the peer from
the connection_lost() function to the request handler loop itself if it is
active.

stats: Minor code cleaup - remove unnecessary includes and fix copyright year

lib: Added unit test to iostream-temp.

lib: iostream-temp makes the internal fd visible now to o_stream_get_fd()

Mainly for the following unit test.

lib-storage: Verify settings only after applying settings from userdb

This is especially important to verify() functions that parse the settings
and store data permanently based on them, e.g. pop3_delete_flag or
imap_client_workarounds, which effectively couldn't be overwritten by userdb.

lib: Added iostream_temp_create_sized() to specify the max in-memory size

login-proxy: Use io_add_istream() for adding client_input io.

This is required for non-fd istreams to work, e.g. ssl-iostream.

lib-http: Fix to earlier http_client_connection_unref() change 1dead6

Patch by Stephan Bosch

doveadm: Code cleanup - remove duplicate code

doveadm: Fixed usage printing for ver2 commands

doveadm: Define DOVEADM_CMD_MAIL_USAGE_PREFIX, which v2 commands use as their usage prefix.

This allows implementing the next changeset without kludging.

doveadm: Added doveadm_cmd_get_ver2() helper

doveadm-index: Use ver2 structures

doveadm-cmd: Add command flags

doveadm: Free istream parameters everywhere

Based on patch by Aki Tuomi

quota: Use doveadm ver2 structures

doveadm: Minor code cleanup - rename doveadm_cmd_find*() to doveadm_cmd_find_with_args*()

doveadm: Code cleanup - remove unnecessary NULL sets and checks

lib-http: Always set *_obj=NULL first in deinit functions (mainly for consistency)

This doesn't usually matter, but in case deinit calls some callback they may
be trying to access the partially destroyed object.

lib-http: http_server_connection_unref() now always sets *conn=NULL

This makes its behavior consistent with other APIs in Dovecot.

lib-http: Clarify http_server_response_*_payload() API and minor change to it

Similar to the change in c3a4c93. Nothing used this API yet.

lib-http: http_server_request_unref() now always sets *req=NULL

This makes its behavior consistent with other APIs in Dovecot.

lib-http: Clarify http_client_request_*_payload() API and minor change to it

The earlier behavior was pretty confusing, and potentially could have caused
double-freeing memory in some situations. Now it's clear that req is set to NULL
always when the request is finished, regardless of whether it has any references left.

Changed http_client_request_finish_payload() to return 0 on success instead of 1.
This could have been left alone, but it's unlikely that there is any code outside
Dovecot core that calls it and this way is cleaner.

lib-http: http_client_request_unref() now always sets *req=NULL

This makes its behavior consistent with other APIs in Dovecot.

Also http_client_request_finish() no longer sets req=NULL, because all of
its callers already keep a reference. Instead added an assert to make sure
the reference is there.

lib-http: http_client_connection_unref() now always sets *conn=NULL

This makes its behavior consistent with other APIs in Dovecot.

doveadm-http: Close client input correctly

doveadm-http: Return HTTP failure on unexpected JSON input

doveadm-http: Handle invalid input better for arrays.

doveadm-http: Handle istreams correctly

Based on patch by Aki Tuomi

doveadm-save: Use ver2 command structure

doveadm: Added support for "file" parameters

doveadm: Fixed memory leak at deinit

doveadm: Fixed double-free when using -F parameter

doveadm: Fixed memory leaks when ver2 command parsing fails with invalid parameter.