lib-index: Simplify code to add missing header fields to cache

lib-index: Add only used fields to cache header, not everything

lib-index: Delay using file-specific field numbers in uncommited cache records

While adding data to cache, it's first kept in memory. Previously already
at this time the memory contained cache field numbers specified to the
current cache file. If the cache file was compressed, all the cached data
had to be discarded because the field numbers changed.

This commit changes the file-specific field indexes to internal index
numbers and converts them to file-specific numbers only just before writing
them to the cache file. This sigfinicantly simplifies the code. It also
allows following commits to avoid having to discard the data when cache is
compressed.

This change also required changing the cache lookup code to realize that
the uncommitted data contains internal field indexes.

lib-index: Keep a linked list of mail_cache_views

Assert-crash on cache deinit if any views are still open.

lib-ntlm: Replace strncpy() with memset()+memcpy()

There was nothing wrong with it, except gcc 9.2 logs a warning about it.

lib-imap: test-imap-url - Fix error handling

lib-http: Fix generating label for incomplete HTTP server request

lib-master: Fix error logging if reading "instances" file fails

The path was written as "(null)" in the error message (or it crashed in some
OSes).

lib-program-client: Do not derefence set when it is NULL.

Found by coverity. Broken in 23cfbf2451f7191dee1d9995011c898a7bf32bd4

log: LOG_OPTION is not supported here

Found by coverity, introduced in d176f84ce5ca2073f4dfbafb457b9c74f6bf0d76

lib-mail: message_header_decode_encoded - ensure num is QCOUNT

Satisifies static analyzers

auth: test-lua - Do not initialize NULL script

Satisfies code analyzers

lib-fts: Assert that data is non-NULL when size > 0

Satisfies code analyzers

lib-index: mail_cache_lookup_headers() - Use const for field_idxs[]

lib-index: Move MAIL_CACHE_MAX_WRITE_BUFFER to mail-cache-private.h

This will help unit tests.

lib-index: unit tests: Split off test_mail_index_close/delete()

lib-index: Remove mail_cache_field_private.adding field

This was used to make sure that cache compression didn't drop a field that
was just being added. After previous commit the cache is locked before this
code is even reached, so it's no longer possible.

lib-index: Use mail_cache_map_all() wherever possible

lib-index: Add mail_cache_map_all()

lib-index: mail_cache_transaction_update_index() - Return void since it can't fail

lib-index: mail_cache_sync_reset_id() - Move cache unlocking to caller

lib-index: Split off mail_cache_sync_reset_id()

lib-index: Split off mail_cache_lookup_rec_get_field()

lib-index: Split off mail_cache_transaction_forget_flushed()

lib-index: mail_cache_header_add_field_locked() - Move unlocking to caller

lib-index: Split off mail_cache_header_add_field_locked()

lib-index: mail_cache_header_fields_write() - Take cache parameter instead

lib-index: Split off mail_cache_trans_get_file_field()

lib-index: mail_cache_lock() - Clarify why index->mapping might be TRUE here

lib-index: mail_transaction_log_sync_lock() - Fix function's comment

lib-auth: Rename auth_client_request_continue event to ...continued

auth: auth_request_finished event - Rename translated_username to translated_user

All the other fields are also "user", not "username".

auth: auth_request_finished event - Rename original_username to orig_user

This way it matches %{orig_user} variable.

auth: Remove unnecessary NULL checks for adding event fields

The NULL values are ignored internally. No need to complicate the code.

auth: Remove duplicate "mechanism" field in auth request events

The "mech" field already existed, and it matches %{mech} variable.

lib-master: Replace auth-master-client-login category with auth-client

This is just a reimplementation of lib-auth/auth-master code. This commit
also changes lib-master to require lib-auth to get to the category. This
dependency shouldn't really be a problem, since eventually this
master-login-auth code should be replaced by lib-auth anyway.

lib-oauth2: Unit tests don't need to link libmaster

lib-auth: Replace auth-master-client category with auth-client

The auth-client and auth-master connections are using mostly the same auth
API. Also the auth-master event names were already auth_client_*. So
there's not much point in having separate categories for them - it's just
confusing.

push-notification: Rename event category from push_notification to push-notification

All other categories are using "-" instead of "_".

lib-dcrypt: Avoid using BN_bn2binpad

It's not available in older versions.

lib-dcrypt: Add EC_GROUP_order_bits if missing

Not there in older OpenSSL

lib: ostream.h - Fix indent in enum ostream_create_file_flags

lib-dcrypt: Require ECDSA signature size is even

Valid ECDSA signature has two parts of equal sizes

lib-dcrypt: Pad signature parts to match ECDSA key size

This is required for ECDSA signatures.

lib-compression: Add unit tests for gz header handling bugs

lib-compression: Fix length checks for zlib header

lib-mail: test-message-address - Add unit test for the one byte overread

Note that this is visible only with valgrind.

lib-mail: message_address_parse_path() - Fix one byte overread

lib-master: If stats_writer_socket_path is empty, don't try to send stats

lib-test: test_exit() - Use regular exit() instead of _exit()

Otherwise code coverage output isn't written for the forked process.

submission: submission-backend-relay - Abort client command when server command is destroyed.

This fixes segmentation faults occurring when server commands are aborted
(connection is closed).

submission: submission-backend-relay - Clear client command in each client command callback.

submission: submission-backend-relay - Add assertions for command contexts in callbacks.

submission: submission-backend-relay - Rename "helo" parameter to "helo_cmd".

Makes it more consistent with implementation of other non-transaction commands.

submission: Reformat submission-backend-relay.c.

lib: Remove deprecated str_append_n

This causes error when compiling within C++, so easiest to remove as it
is already not being used anywhere

lib: var_expand*() - Ignore modifiers for unknown variables

So for example %M{asdf} will return UNSUPPORTED_VARIABLE_asdf instead of MD5
hash of it, which might not be so obvious that it's not working correctly.

lib: var_expand*() - Return UNKNOWN_VARIABLE_x if %x is unknown

This was already done for %{long_keys}, just not for single letter keys.

dict: Remove "dict client: " log prefix

Almost all log messages from dict process would end up having this prefix.

lib-storage: index: imapc: Implement support for SAVEDATE attribute.

lib-imap-client: imapc-client - Recognize the new SAVEDATE capability.

imap: Implement the standard SAVEDATE capability.

lib-storage: mail-search - Add support for the SAVEDATESUPPORTED search item.

lib-storage: mail-storage - Change mail_get_save_date() to return 1 when result is reliable.

dict-sql: Pass dict event to lib-sql

dict: Add events

dict-client: Use event based logging

dict-redis: Use event based logging

dict-memcached: Use event based logging

dict-file: Use event based logging

lib-dict: Add named events

lib-dict: Add event support

dict-file: Move error logging higher up

Makes next commit easier.

lib-oauth2: Fix possible timing attack in oauth2_validate_hmac()

Use mem_equals_timing_safe() instead of memcmp() to test the equality of the digest.

lib-dict: Add refcounting

This way we do not crash if lookup callback happens after caller
has already called dict_deinit. This is now done because
dict-sql does not support aborting SQL queries. This is hopefully
fixed in future.

lib-dict: Add async lookup/commit helpers

Simplifies following commits

lib-dict-backend: Use ioloop pointers in parent struct

lib-dict: Use ioloop pointers in parent struct

lib-dns: Remove superfluous client event and avoid duplicate client connection event unreferencing.

lib-smtp: smtp-server-command - Change log label for invalid/unknown command to "[unknown]".

lib-smtp: smtp-command-parser - Don't upper-case the parsed command name.

lib-smtp: smtp-server-command - Set cmd_name event field only for valid commands.

Set cmd_input_name for the user input.

lib-smtp: smtp-server-command - Assign cmd->reg immediately.

lib: ostream-file - Fix using sendfile() when input stream is ends unexpectedly early

Handle this the same as if writing was done without sendfile(), i.e. ignore
the problem and just return EOF in input stream early.

lib: safe_sendfile() - clarify return value 0 handling

global: Remove dead assignments

Found by latest clang scan-build.

stats, fts-solr: Tweak unit tests to make static analyzers happier

imap: SETMETADATA - Reorder if checks to remove unnecessary indenting

Nothing changes in the code logic.

imap: SETMETADATA - Cleanup istream return value checking

The old code seems to have worked correctly already, but now it's a bit
clearer how it was intended to work.

lib-oauth2: Add assert to make sure json_parse_next() doesn't think istream is nonblocking

imap: Add assert to make sure fd was successfully sent to imap-hibernate

lib-index: Don't rewrite dovecot.index if another process has already done it

This mainly happened when there were multiple sessions accessing the index
at the same time.

lib-index: Log a debug message with reason whenever recreating dovecot.index

lib-index: Change mail_index.need_recreate into a string

The string specifies the reason why the recreation is wanted.

lib-index: Change mail_transaction_log_file.need_rotate into a string

The string specifies the reason why the rotation is wanted.

auth: Add session_id to default policy_request_attributes

imap: Fix SEARCH PARTIAL handling with other options

Especially SAVE and RELEVANCY should use only the partial results, not all
results.

imap: Fix SEARCH (MAX RELEVANCY) to return the relevancy

imap: Simplify SEARCH MIN and MAX result handling

imap: Fix using SEARCH RELEVANCY option with MIN/MAX

It should return only the one MIN/MAX mail's relevancy, not all of them.
Similarly the SEARCHRES $ should be updated only to the MIN/MAX mail, not
all mails.