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.

imap: Fix using $ in a SEARCH that also uses SAVE, or fails with BAD

SEARCH RETURN (SAVE) $ cleared the $ instead of being a no-op.

Also using a SEARCH that returns BAD shouldn't have cleared $ (but NO
reply especially for BADCHARSET should clear it).

imap: Clear SEARCHRES $ after closing mailbox

$ shouldn't have been preserved after a mailbox was SELECTed.

lib-storage: mail_search_arg_init() - Remove support for changing uidset

This is done earlier now - there's no need to do it twice.

lib-storage: Fix using $ (IMAP SEARCHRES) in search seqset/uidset

auth: userdb: Ensure changed username in auth worker gets updated in cache

If a username gets changed and auth cache is in use, auth-workers for
blocking userdb did not update the username in cache. This is now
fixed by setting user_changed_by_lookup in case the username changed.

configure.ac: Don't look for PRIuSIZE_T

We are using %zu instead.

global: use %zu directly instead of PRIuSIZE_T

lib-storage: Remove stub mail support

Nothing used them in the 3 years since they were introduced.

imap: Unknown SEARCH charset should return NO, not BAD

imap: Use "unknown" for cmd_name when the command name is not known

stats: Move group-by value and label getting code into helper functions

This avoids some excessive indentation.

stats: Add support for exponential stats group-by fxn

One can specify the quantization parameters in the config file as:

    <field>:exponential:<min mag>:<max mag>:<base>

Currently, only base 2 and base 10 are supported.

For example:

    group_by = bytes_out:exponential:1:4:10

Which will quantize the bytes_out values into the buckets: (-inf, 10],
(10,100], (100,1000], (1000, 10000], (10000, +inf).

stats: Add support for linear stats group-by fxn

One can specify the quantization parameters in the config file as:

<field>:linear:<min val>:<max val>:<step>

For example:

group_by = bytes_out:linear:0:1000:100

Which will quantize the bytes_out values into the buckets: (-inf, 0],
(0,100], (100,200], ... (900, 1000], (1000, +inf).

stats: Process 'duration' like any other metric value

This will allow the soon-to-be-added group-by quantization functions to work
on the duration value.

stats: Move event field stats distribution update into a helper function

stats: Use a switch instead of if/else-if

The compiler will then warn about unchecked enum values.

stats: Add support for explicit discrete group-by function

One can specify the function as:

    <field>:discrete

This has the same exact behavior as the bare field name syntax before this
change.

For example, the following two lines have identical semantics:

    group_by = cmd_name:discrete
    group_by = cmd_name

stats: Rewrite group-by discrete test to be more thorough

This will also help with expanding the test in later commits.

stats: Move group by setting parsing earlier

Instead of doing the split and parsing in the metric code, we now do it in
the settings code.  This makes the following commits much simpler.

lib-compression: test-compression - Fix test buffer size in partial write test

The buffer size does not match the amount of data filled.

Found by valgrind.

dict-ldap: Do not attempt to query if LDAP query building fails

plugins/fts: script parser - Properly determine the attachment extension

plugins/fts: Lower-case content types received from script parser

Later on during the execution, the script parser code does a strcmp() to
match the content type of the attachment with that returned by the script.
The content type from the mail is already lowercased.