configure.ac: Update to v2.3.14.1

NEWS: Add news for 2.3.14.1

lib-dict-extra: dict-fs - Escape unsafe paths

Change any path components that are `.` or `..` to `...` and `....`.
Prevents path traversal attacks.

lib-oauth2: Do not escape '.'

This is not really needed and just makes things difficult.

lib-oauth2: Add missing test_begin/test_end to token escape

lib-smtp: smtp-server-connection - Fix STARTTLS command injection vulnerability.

The input handler kept reading more commands even though the input was locked by
the STARTTLS command, thereby causing it to read the command pipelined beyond
STARTTLS. This causes a STARTTLS command injection vulerability.

lib-imap: imap_bodystructure_write() - Return error on corruption instead of assert-crash

This could happen if broken message_parts came from cache and
message_part->data was newly read from the mail input.

lib-storage: Split off index_mail_parse_bodystructure_full()

lib-imap: fuzz-imap-bodystructure - Change failures to i_panic() instead of i_fatal()

imap: Fix using SETMETADATA NIL to unset value

Broken by 923362d27d9b2428e301614673cb0efba3bf928f

lib-oauth2: test-oauth2-jwt - Fix linkage for openssl

Without whole archive option ssl_iostream_unref isn't
included in linkage, which will prevent libdcrypt from
loading openssl backend.

lib-oauth2: Ensure azp is escaped too

lib-oauth2: Add test for token escape

lib-oauth2: Improve identifier escaping function

lib-oauth2: Move identifier escaping to own function

lib-compression: Get error with i_stream_get_error() so it's recursive

In some cases the iostream.error was NULL so the errno default message
was used. This fixes an regression introduced by 8b6142f9ba9.

NEWS: Mention XZ/LZMA removal.

NEWS: Update news for 2.3.14

lib-compression: Remove xz/lzma write support

For now, we keep the read support to allow for a more gradual migration from
lzma/xz.

The code has bugs.  Additionally, there is no benefit in keeping it around.
bzip2 or zstd (at maximum compression level) are likely better choices for
archival.  For non-archival usecases, a light compression algorithm like
zstd (at low to medium compression level) or lz4 are better choices.

doc/example: 10-metrics.conf - Add metrics related settings

doc/eaxmple: 10-logging.conf - Update syntax example to point to online documentation

lib-mail: message-parser - Always properly finish parsed input

If the caller hadn't called message_parser_parse_next_block() after
seeing EOF in the istream, the deinit could have crashed with:

Panic: file message-parser.c: line 802 (message_parser_deinit_from_parts): assertion failed: (ctx->nested_parts_count == 0 || i_stream_have_bytes_left(ctx->input))

stats: openmetrics: Do microsecond to seconds conversion only for duration values

Other metrics should be exported with their values unchanged.

stats: openmentrics: Change histogram bucket unit to seconds

In ae678116a79fff609cdf4fb1eb7eb3db2975bf1c the unit of durations was changed
from microseconds to seconds, but the `le` field in histograms is still in
microseconds.

stats: refactor duration field name into a macro

plugins: fts-solr: Use a private HTTP client.

This is a workaround for problems with the http_client_request_send_payload()
API that occur when several shared HTTP clients use this API at the same time
and payload sending gets interleaved.

plugins: fts: tika: Use a private HTTP client.

This is a workaround for problems with the http_client_request_send_payload()
API that occur when several shared HTTP clients use this API at the same time
and payload sending gets interleaved.

NEWS: Change 2.3.14 to release candidate

config: test-config-parser - Change filename to match .gitignore pattern

config: test-config-parser - Cleanup input file after test

NEWS: Add news for 2.3.14

lib-settings: test-settings-parser - Use ULL suffix

Some of the sizes do not fit to UL in 32 bit systems.

lib-mail: test-message-snippet - Add unit tests for multipart messages

lib-mail: test-message-snippet - Use test_assert_strcmp

Makes it easier to see what failed.

lib-mail: Reformat test-message-snippet.c

lib-mail: message-snippet - Stop building snippet on first non-empty part

When we have gotten some snippet out of first eligible part,
we can stop.

lib-mail: message-snippet - Do not stop on first non-text part

Some mail structures would break snippet generation because
the code would stop on multipart.

lib-mail: message-snippet - Free html parser before making new

If html parser has already been initialized, free it before restarting
the parser.

Fixes memory leak introduced by 7f7be2cbf68f8a202a688d5bc50f82483d461643

configure: Update version to 2.3.14

stats: stats-service-openmetrics - Use PRIdTIME_T for startup time

login: login-proxy - Fix type conflict

lib-mail: test-message-header-decode - Use ptrdiff_t for rep_char_len

The length is compared to a pointer difference, so using ptrdiff_t
squashes a warning.

lib-settings: test-setting-parser - Use correct type in test

SIZE uses uoff_t.

lib: backtrace - Use correct format modifier

Matches size_t correctly.

lib: test-time-util - Use correct types for test case

Fixes type mismatch on 32-bit systems.

master: Fix reopening log files on SIGUSR1

man: Clarify how doveadm log reopen works

dovecot.service.in: Update documentation URL

lib-master: test-event-stats - Fix unit test when compiling using build directory

When using build directory __FILE__ expands to
"../../../src/lib-master/test-event-stats.c", which doesn't match
"test-event-stats.c" what is normally used.

lib-master: test-event-stats - Cast test value to uint64_t

lib: Fix timeval_cmp_margin for 32-bit systems

The test suite compares times with seconds values of -INT_MAX and
INT_MAX. The result of this comparison does not fit in a value of
type int and so the test suite fails on 32-bit systems where time_t
is an int. To fix this, calculations on seconds values are done
using long long integers.

Broken by 16ab5542

dsync: Use a better default vname escape character

A control character was a bit unreadable in strings.

dsync: Replace escape characters in outgoing mailbox names with alt_char

Especially when dsync is used for migration, we don't want to end up having
invalid mUTF7 mailbox names locally. Also, remote might not even be
configured to use the same escape character.

dsync: Send alt_char also to slave brain

lib-storage: mailbox_list_escape_name_params() - Assert that escape_char isn't NUL

imapc: Add comment explaining various mailbox names

imapc: fs mailbox_list - Remove vname_escape_char

This isn't necessary.

imapc: Add IMAPC_LIST_FS_NAME_ESCAPE_CHAR

This clarifies how the escape characters are used.

imapc: Add a default vname_escape_char

Having vname_escape_char guarantees that all the remote mailbox names are
accessible by escaping them as necessary.

imapc: Fix local filesystem directory name escaping

Use the remote mailbox name as the base. Convert the hierarchy separator
and add escaping as necessary.

The old method was based on storage_name, and it's a bit unclear if it
worked correctly in all situations.

imapc: Add imapc_list_remote_to_storage_name()

imapc: Fix remote/storage mailbox name mismatches

imapc: Use more explicit mailbox name variable names

Clearly specify which type of a mailbox name it is to avoid
misunderstandings.

imapc: Rename imapc_list_get_fs_name() to imapc_list_storage_to_fs_name()

imapc: Rename imapc_list_to_vname() to imapc_list_remote_to_vname()

imapc: Rename imapc_list_to_remote() to imapc_list_storage_to_remote_name()

lib-storage: mailbox_list_default_get_storage_name() - Escape ns_prefix/INBOX if possible

If storage_name_escape_char is set, escape the "I" letter in the
storage_name if the namespace also has inbox=yes. This way it's possible to
distinguish between the actual INBOX and ns_prefix/INBOX.

lib-storage: Update mailbox_list_settings.{vname|storage_name}_escape_char comments

Also add the examples in the comments to unit tests to make sure they really
work.

lib-storage: Fix mailbox name escape/unescape ordering

This fixes some cases like not properly escaping '/' in storage_name if it
was also the namespace separator. This also allows using the same character
for both vname_escape_char and storage_name_escape_char.

lib-storage: mailbox_list_default_get_vname() - Fix escaping namespace separator in storage_name

For example if { storage_name="foo/bar", storage_sep=".", ns_sep="/" }
the "/" in the name shouldn't be treated as a namespace separator.
If vname_escape_char is set, it should be returned escaped instead.

lib-storage: Use imap_escaped_utf8_to_utf7()

lib-storage: Use imap_escaped_utf8_hex_to_char()

lib-storage: Use mailbox_list_settings.vname_escape_char also with utf8 storage names

mailbox_list_default_get_storage_name() was already unescaping with utf8,
so mailbox_list_default_get_vname() should also be escaping with utf8.

lib-storage: mailbox_list_default_get_vname() - Use imap_utf7_to_utf8_escaped()

This changes the output a bit for broken mUTF7 names. Previously if any
part of the string had broken mUTF7 input, none of it was converted.
Now it's instead trying to convert as much as it can.

lib-storage: mailbox_list_default_get_vname() - Escape each hierarchical name separately

Same reasoning as for the similar change for
mailbox_list_default_get_storage_name().

lib-storage: Split off mailbox_list_storage_name_prepare()

lib-storage: mailbox_list_default_get_storage_name() - Escape each hierarchical name separately

This makes it possible in the following commits to allow unescaping to
write namespace hierarchy separator character without converting it into
the mailbox_list separator.

lib-storage: mailbox_list_default_get_storage_name() - Cleanup shared namespace root handling

lib-storage: Split off mailbox_list_vname_prepare()

This also slightly changes the code logic. mailbox_list_escape_name() is
now called also for INBOX, but that doesn't actually matter.

lib-storage: Remove mailbox_list_unescape_name() by moving its code inline

There was only a single caller for this function. It doesn't need to exist.

lib-storage: Remove mailbox_list_escape_name() by moving its code inline

There was only a single caller for this function. It doesn't need to exist.

lib-storage: Add unit tests for mailbox_list_default_get_{storage_name|vname}()

lib-storage: Rename mailbox_list_settings.broken_char to vname_escape_char

This clarifies that it's only used in vnames.

lib-storage: Rename mailbox_list_settings.escape_char to storage_name_escape_char

This clarifies that it's only used in storage_names.

lib-imap: Add imap_utf7_to_utf8_escaped() and imap_escaped_utf8_to_utf7()

These can be used to convert invalid mUTF-7 to escaped UTF-8 and back to the
original invalid input.

lib-imap: test-imap-utf7 - Simplify test_imap_utf7_non_utf16()

lib-imap: Move unnecessary shift check to mbase64_decode_to_utf8()

lib-imap: test-imap-utf7 - Improve/clarify testing '&'

& is never base64-encoded.

config: Add unit test for config parser

config: Extract sources to libconfig.la

Makes unit testing possible

lib-settings: test-settings - Add tests for settings file

test-settings-parser: Add test for size

lib-settings: test-settings-parser - Add test for settings parser

lib-settings: Add support for $ENV:name

Expands into given environment variable

config: Add support for $ENV:name

Expands into given environment variable

config: Expand multiple variables per line

Needed to properly support environment variables.

config: config-parser - Add config_write_keyvariable

Move code handling key variable expansion to separate function.
Simplifies next commit.

lib-storage: Make lua methods and functions use the passed in lua_State * directly

lib-storage: Make dlua_push_mail() take lua_State * directly

lib-storage: Make dlua_push_mail_user() take lua_State * directly