indexer: indexer-worker - Use event logging

indexer: indexer-worker: Use connection.c functions

indexer: indexer-worker: Use connection structure

indexer: worker-connection - Use connection.c functions

indexer: worker-connection - Prepare for using connection.c

indexer: worker-pool - Use connection_list

indexer: worker-pool - Remove unused last_use

indexer: worker-connection - Use connection.h structures

indexer: worker-connection - Move constructor further down

Simplifies next commit

Reformat worker-connection.c

lib: lib-event - Copy ru_last when making passthrough event

Otherwise user_cpu_usecs does not get added to the sent events.

master: Fix idle-kill for processes with a single short-lived client

lib-master sends unimportant service status updates to master only once per
second when client_limit>1, because within that 1 second the available client
count may have already changed many times. Normally this is fine, but there's a
problem with the initial client:

 * The client connection launches a new process
 * The process sends the initial status notification with available_count=MAX
 * The client connection is handled, but because the status notification was
   just sent, the update won't be sent until 1 second later
 * The client disconnects within the 1 second
 * The available_count is again MAX, so no status notification is sent

In this situation the master process never created the idle-kill timeout.
Usually the following clients will cause a status notification to be sent, but
if there was only the one client then this process won't be idle-killed. Fix the
situation so that the idle-kill timeout is created already by the initial status
notification.

lib: Add array_foreach_reverse[_modifiable]()

This is especially useful when deleting multiple elements inside the loop.

lib: Don't use special code for i_unreached() with STATIC_CHECKER

This was done originally to help scan-build, but this is no longer
necessary. Also actually running code where __builtin_unreachable() was
reached produced (very weird) undefined behavior.

Reverts e2e9ea6da9f3db5fd7fe467db79232d20d03832f

Makefile.am: Update configure parameters for scan-build

Especially add --enable-static-checker.

lib-imap: imap-parser - Parse literal size using as same algorithm as str_parse*().

This prevents wrapping the integer value and fixes an ubsan complaint.

Based on Stephan's similar patch in managesieve-parser.

lib-imap: imap-parser - Fix 8 bit atom check to use unsigned char.

lib-http: Add http-client category to client events

acl: Ignore acls in acl_mailbox_right_lookup if ignore_acls is set

In case an namespace has been configured to ignore ACLs make sure that
also happens for acl_mailbox_right_lookup.

acl: If acl_ignore_namespace is set acl_backend can be NULL

Allow an uninitialized acl backend when ignore_acls is set.

lib-mail: Add MESSAGE_PART_FLAG_OVERFLOW to MIME parts that reached parsing limits

This allows callers to find out which MIME parts were properly parsed.

imap: copy: Abort if client disconnects during COPY

Since the client didn't receive the COPY tagged reply, it doesn't know
whether the COPY succeeded or not. This likely causes it to try the COPY
again and duplicate the mails.

imap: imap-sync - Fix VANISHED response with UID=1

Fixes a bug introduced in 4eb3f6f27, where a UID EXPUNGE 1 command
resulted in a VANSIHED 0:1 response, which is invalid IMAP syntax
(sequence sets must have nonzero values).

fts: Do not consider arbitrary headers with 8-bit data as language-specific

If we do stemming etc processing for arbitrary headers, they might
become impossible to find in subsequent searches due to stemming.

This practically breaks e.g. From/To header searches with non-ascii
names.

lib: DEBUG: Fix potential crash in handling "Growing data stack" debug message

This could have only happened when data_stack_grow event was enabled and
when --enable-devel-checks was used.

lib: test-data-stack: Run each test with the same data stack state

lib-lua: Delay running Lua script until dlua_script_init()

This allows the caller to register globals before running the script.

lib-lua: Move finalizing event and setting linked list to dlua_create_script()

There's no need to delay them until dlua_script_create_finish().

lib-lua: Update dlua_script_create_*() comment

Reusing existing scripts wasn't actually implemented.

lib-lua: dlua_script_init() - Always return -1 if script_init() fails

The previous behavior was to forward the script_init() return value to
the caller. This was somewhat unexpected behavior.

lib-storage: dlua_register_mail_storage() - Fix popping dlua_dovecot in Lua stack

lib-storage: Fix corrupted cache when adding attachment keywords

If IMAP BODYSTRUCTURE parsing fails, it means the BODYSTRUCTURE doesn't
match the MIME parts. This likely means that one or both of them are
corrupted. Handle this by deleting the email from cache, so this error
won't keep repeating.

lib-storage: Don't log error if attachment flags couldn't be set due to missing cache

Ever since 194dcaa65cdd247393633f2daa4b40fd12985440 missing attachment
keywords have been attempted to be automatically added if all the necessary
fields are in dovecot.index.cache. However, if mime.parts wasn't in the
cache an error was unnecessarily logged.

Fixes:
Error: Failed to add attachment keywords: mail_get_parts() failed: Mail field not cached

imap-acl: In case mailbox name is "" set it to INBOX for shared namespaces

To prevent proxying ACL commands with mailbox "" replace it with "INBOX"
when used with shared namespaces.

imapc: Extract imapc_mailbox_name_equals from imapc_untagged_status

imap-acl: Replace mailbox_open_as_admin with mailbox_open_allocated_as_admin

imap-acl: Allow LISTRIGHT replies without opening as admin

As the reply is hardcoded there is no need to proxy the ACL
command to a remote backend in case imapc_feature "acl" is
enabled.

imap-acl: Implement proxying commands to an imapc location for DELETEACL

Before calling imap_acl_cmd_deleteacl in cmd_deleteacl the new code
checks if the command should be proxied and if so, does the proxying.

imap-acl: Implement proxying commands to an imapc location for SETACL

Before calling imap_acl_cmd_setacl in cmd_setacl the new code checks if
the command should be proxied and if so, does the proxying.

imap-acl: Implement proxying commands to an imapc location for GETACL

Before calling imap_acl_cmd_getacl in cmd_getacl the new code checks if the
command should be proxied and if so, does the proxying.

imap-acl: Extract imap_acl_cmd_deleteacl from cmd_deletacl

This change makes acl_mailbox_open_allocated_as_admin called later which
is necessary for the following proxying IMAP ACL changes.

imap-acl: Extract imap_acl_cmd_setacl from cmd_setacl

This change makes acl_mailbox_open_allocated_as_admin called later which
is necessary for the following proxying IMAP ACL changes.

imap-acl: Extract imap_acl_cmd_getacl from cmd_getacl

This change makes acl_mailbox_open_allocated_as_admin called later which
is necessary for the following proxying IMAP ACL changes.

imap-acl: Implement proxying commands to an imapc location for MYRIGHTS

Introduce the needed changes to proxy imap-acl commands to a remote
backend using imap-client.

Before calling imap_acl_cmd_myrights in cmd_myrights the new code checks if
the command should be proxied and if so, does the proxying.

imap-acl: Add imapc_acl_get_mailbox_error

Allows to retrieve full mailbox error.

imap-acl: Split off imap_acl_cmd_myrights from cmd_myrights

imap-acl: Split acl_mailbox_open_as_admin

This allows easier implementation of the imap-acl proxying.

imap-acl: Add imap_acl_storage with module context

imapc: Add imapc_mail_error_to_resp_text_code to access error msg by enum

imapc: Change function signature prefix to imapc_resp_text_code_parse

imapc: Add imapc_storage_client_unregister_untagged function

imapc: Add imapc_features=acl

In order to allow proxying IMAP ACL commands to a remote backend

lib: test-event-filter-*: Hide details of tests unless they fail

This removes ~14k lines of output from test-lib run.

imap: Send tagged login reply before finalizing user initialization

Broken by 5fc66f182ff6941639d30372b414c1b39ae1e67e

m4, dovecot-config: Add LIBDOVECOT_LUA_INCLUDE

This allows building plugins that use lib-lua against uninstalled Dovecot
source code tree.

lib: event filter - Return invalid chars in the lexer

This way, we leave it up to the parser to figure out whether or not they
make sense.

lib: event filter - Allow '?' in filter language tokens

Without this, it is not possible to use '?' wildcards without quoting the
value.

dict-sql: Support SQL_DB_FLAG_ON_CONFLICT_DO

lib-sql: Add SQL_DB_FLAG_ON_CONFICT_DO for new enough sqlite and pgsql

lib-sql: sqlpool: Implement get_flags() as a wrapper to the real db

lib: test-array - Add more unit tests for array_foreach*()

Make sure that the elem value is as expected after the loop completes.

lib-sql: Add sql_db_vfuncs.get_flags()

dict-sql: Use SQL_DB_FLAG_ON_DUPLICATE_KEY

Instead of hardcoding mysql check.

lib-sql: Add SQL_DB_FLAG_ON_DUPLICATE_KEY flag for MySQL

auth: mech-oauth2 - Iterate fields in safer manner

Static analyser wasn't happy about the previous way.

lib-smtp: smtp-server-recipient - Remove smtp_server_recipient_initialize().

It is no longer needed.

lib-smtp: smtp-server-recipient - Create transaction event early if there is no transaction yet.

This allows having a parent for the recipient event once the recipient is
created, which can precede the creation of the transaction.

lib: Free unused data stack memory once per second while running ioloop

This way if the data stack has grown excessively large temporarily, it
won't permanently waste memory. And if the data stack grows back to the
same large size, re-allocating it once per second doesn't cause performance
problems.

lib: data-stack - Keep the largest unused block in memory after all

This reverts 2da21080dbbe50b4924ac0135c84babfb404dcce, but that alone
doesn't produce working code so there are also some further fixups.

lib-smtp: test-smtp-payload - Double the client progress timeout.

Otherwise, SSL tests can fail on systems with low entropy.

lib-smtp: smtp-client-transaction - Allow destroying connection while transaction still holds reference.

The lifetime of the transaction is controlled by the application and not so much
by the connection. The transaction holds a reference to the connection, so if
the connection is forcibly destroyed it needs, to notify the transaction of this
event.

This fixes:

Panic: file connection.c: line 938 (connection_list_deinit): assertion failed: (conn != list->connections)

lib-smtp: smtp-client-transaction - Properly handle pending failure upon smtp_client_transaction_send_reset().

lib-smtp: smtp-client-connection - Add smtp_client_connection_transactions_fail() and use it.

Use it to always fail the whole transaction first before the individual commands
if possible.

lib-smtp: smtp-client-connection - Split off smtp_client_connection_transactions_fail_reply().

lib-smtp: smtp-client-connection - Always abort all commands upon smtp_client_connection_close().

Before, these would get a disconnect failure, which makes no sense when the
client is causing the disconnect.

lib-smtp: smtp-client-connection - Abort all transactions upon smtp_client_connection_close().

Before, these would get a disconnect failure, which makes no sense when the
client is causing the disconnect.

lib-smtp: test-smtp-payload - Finish client transaction only when it finishes completely.

lib-smtp: Reformat smtp-client-transaction.c.

lib-smtp: Reformat smtp-client-connection.c.

auth: oauth2 - Support passing openid-configuration URL to client on failure

auth: mech-oauth2 - Pass error fields as array

Makes it easier to add fields to it later on.

imap: copy/move: Fix memory leak when no messages were found

doveadm: Delay reading settings until doveadm plugins are loaded

doveadm: Add doveadm_setting_roots_*() API for plugins to register settings

doveadm: Move settings reading code into doveadm-settings.c

This code is now shared between doveadm and doveadm-server.

lib-master: Add master_service_settings_input.extra_modules

This allows specifying more than one module.

lib-master: master_service_exec_config() - Build argv using dynamic array

auth: auth-worker - Respect service_count

If service_count is set, then auth-worker will stop after
given number of requests.

imap: Improve sending internal error to client if initialization fails

The "Internal server error" should be sent using the same ostream that
was created. Also if service_user initialization failed there was no
OK reply to sent to login.

imap: Send initial tagged reply directly using ostream

This simplifies the following commits.

imap: Send initial tagged reply and capability after rawlogs are initialized

This way the initial tagged reply and capability is written to the rawlog.
Broken by 16328738d35370685387ec51d20657265855e770

imap: Split client_add_input_capability() into two functions

imap: Rename struct client_input to imap_login_request

This better indicates what it is about, and causes less confusion with
all the other client_input variables.

pop3: Handle pipelined input via istream-concat

This fixes rawlog_dir to work properly. It didn't log the pipelined input
to rawlog and it may have also caused the connection to hang.

pop3: Move finishing istream to client_create_finish()

imap: Handle pipelined input from imap-hibernate via istream-concat

Similarly to the previous commit, this fixes issues with rawlog.

imap: Handle pipelined input via istream-concat

This fixes rawlog_dir to work properly. It didn't log the pipelined input
to rawlog and it may have also caused the connection to hang.

imap: Move iostream-rawlog creation to client_create_finish()

This allows the caller to modify the istream before rawlog.

lib: Add i_stream_copy_fd()

acl: Fix broken LIST for shared namespaces

Due to the recent changes in the usage of the acl_ignore_namespace
setting shared namespaces where trying to use fast listing too. This
resulted in wrong LIST IMAP command outputs when using acl plugin.

Broken by dc8ecd38a7e54b8bb80ae97712a0d8ad4edcbed3