fts-flatcurve: fts_flatcurve_xapian_db_populate() - Remove memory leak on returning error strings

Discovered on coverity

lib-lua/dlua-iostream: Remove unnecessary stream state assertion

lib-lua: test_io_lua() - Ensure a read error is handled correctly

lib-lua/dlua-iostream: dlua_i_read_common() - Optimize error handling

This also makes the error handling more explicit.

lib-lua/dlua-iostream: dlua_read_line() - Ignore result of i_stream_read()

Issued by coverity unchecked return value.

config: settings_add_include() - Close file descriptor if fstat() failed

Issued by coverity resource leak error.

submission-login: cmd_helo_reply() - Only check against SMTP UTF8 if feature is enabled

Issued by coverity deadcode error.

imapc: Do not forward ALERT response codes from sources with insecure connections

lib: hostpid - Use getaddrinfo instead of gethostbyname

gethostbyname is obsolote, so we should stop using it.

imap: imap-select: Disable SELECT UNSEEN response if IMAP4rev2 is enabled

lib-dns-client: test-dns-lookup - Add DLLIB for dlopen()

lib-http: Use explicit numbers for HTTP_CLIENT_REQUEST_ERROR_* codes

We're beginning on rely on these numbers not to change, so make the numbers
explicit to avoid accidental changes.

lib-dns src: Change dns_client_settings to config setting.

lib-http: Pass http_client_host to http_client_host_shared_lookup.

The following commit will make dns_lookup retrieving DNS settings from
event. http_client_host is need to pass http_client event to dns_lookup, the event
contains client-specific DNS settings.

lib-dns-client: Move struct dns_lookup initialization code into separate function.

Upcoming changes will add more usages of that.

lib-http: Set http_client_context.dns_lookup_timeout_msecs to default value unconditionally.

DNS lookup timeout will be made configurable in the following commit.

lib-smtp: Remove fallback to net_gethostbyname as not supported by upcoming changes.

lib-http: Remove fallback to net_gethostbyname as not supported by upcoming changes.

lib-dns-client: Move idle_timeout_msecs and cache_ttl_secs from dns_client_settings to dns_client_parameters.

As a preparation to make dns_client_settings configurable.

submission: Originate submission_client event from event having settings_root set.

lib-http: Fix bug with removal of http_client_request from delayed_requests.

Setting release_time to 0 in the beginning of
http_client_queue_submit_now() prevents removing of the request from
delayed_requests in case of failures.

lib-dns-client, lib-http: remove ioloop from struct dns_client_settings.

src: Remove event_parent from struct dns_client_settings.

src: Rename dns_lookup_settings to dns_client_settings.

config: Improve comment about self-referencing keys expansion.

config: Cleanup config_write_keyvariable() function.

config: Fix expanding $SET:key inside filters.

expand_parent parameter was added to config_get_value() in
92a9ff5e799adf9bffb8831576d84eb84b8ef09a, so its behavior was likely
useful only in the earlier hierarchical configuration.

pop3: Free var_expand program in parse_uidl_keymask()

Forgotten in 37c9d7802a6713443bcab3cfa3d01a57b154c4c1

lib-var-expand: Unload loaded crypt module at exit

Forgotten in 73ae2a9e45144f05a179d17d9b201226d93b81a0

imap: cmd_append_handle_args() - Fix calling i_stream_create_limit() twice

f12df6e0eed634aa4fe92c225b54ef5ee965b35c accidentally added a second
call to i_stream_create_limit() was which led to leaking events and
wrong iostream->refcount in other plugins

config: Fix config filter sorting order

It was somewhat luckily working in glibc, but broke with e.g. musl libc.
This caused a crash at startup.

Broken by 5acdd2b97ed6092d1f0204bdeb3e418180519370

imap: If IMAP4rev2 is enabled by the client also enable QRESYNC

lib-lua: Add base64 encode/decode support

lib-ssl-iostream: Fix OpenSSL 3.0 provider/engine preference logic

lib, lib-language: Use perl found by configure

This allows better errors if perl wasn't found.

lib-settings: Distribute settings-history-core.h

It does not need to be regenerated every time.

lib-settings: Use python3 found by configure

This allows better errors if python3 wasn't found.

config: Change config_dump_full_context.config const

config: Change struct config_parsed parameters const for most functions

lib-storage: Add explicit "set/" userdb prefix for settings

If the setting name is unknown, it's an error. At least for now this prefix
is optional.

lib-settings: Fail settings lookups if there are unknown setting overrides

This is now safe, because userdb no longer adds unknown settings to
overrides.

lib-storage: Don't add userdb fields to settings if they're unknown

Userdb extra fields can be used for other purposes than as being settings.
Now that we know all setting names, don't add them to settings overrides
if we know they're not known settings.

lib-settings: Add settings_key_exists()

lib-settings: Remove unused settings_override.filter_event

lib-settings, config: Remove unused override event filter

lib-settings, config: Fix and simplify override order handling

The order is now based on the number of named list filter elements in the
override path vs binary config's event filter.

This also happens to fix at least some issues where @group include was
wrongly overriding userdb fields.

lib-settings: settings_mmap_apply_filter() - Refactor to use settings_mmap_event_filter parameter

lib-settings: Introduce struct settings_mmap_event_filter

lib-settings: Optimize skipping overrides that aren't in the looked up setting_parser_info

lib-settings: Remove unnecessary settings_override.path_element_count

It has not been used since 09e551901eeb21bd8fc7b07eb67d80fd43fbb71e

lib-settings: Build override settings filters using "all settings" hash table in binary config

If there is no binary config (= unit tests), require that all the settings
infos have been explicitly registered.

This simplifies the override handling code and allows further optimizations.

lib-settings: Move settings override "*" handling out of the loop

This simplifies the code a bit.

lib-master, global: Split MASTER_SERVICE_FLAG_NO_CONFIG_SETTINGS into separate flags

Use MASTER_SERVICE_FLAG_CONFIG_BUILTIN for unit tests and
MASTER_SERVICE_FLAG_CONFIG_DEFAULTS for getting defaults via doveconf.

global: Explicitly register some settings

These will be needed by the following changes

config: Add CONFIG_MODULES environment to override the default module directory

config: doveconf -dF - Add dovecot_storage_version

config, lib-settings: Write all setting keys into hash table in binary config

config: Reimplement config_parsed_key_lookup() with all_keys hash table

config: Add config_parsed_get_all_keys()

config: Remove unused dump_context.filter_idx

config: Move all structs and enums to beginning of file

config, lib-settings: Access filter index/offset arrays directly via mmap

Add 64bit alignment to the start offset to make this safe.

lib-master: test-master-service-settings - Fix earlier tests

lib-settings, config: Use native CPU endianess in binary config file

The binary config file is never shared between different servers, so for
performance it's better to just use native endianess.

lib-settings: Add and use SETTINGS_APPEND_KEY_SUFFIX macro

lib-storage: Don't use \Recent flags if IMAP4rev2 is enabled

IMAP4rev2 sessions still update the "highest \Recent message UID", but it
doesn't return any messages as having \Recent flag.

imap: imap-status: Handle RECENT as invalid status item for IMAP4rev2

imap: Disable RECENT replies when IMAP4REV2 is enabled

imap: Do not use deprecated IMAP4rev1 search arguments if IMAP4rev2 is enabled

lib-storage: Deprecate IMAP4rev1 mail_search_register_args for IMAP4rev2

Also introduce mail_search_register_get_imap4rev2() and
mail_search_register_get_imap4rev1().

lib-storage: Add enum mail_search_register_arg_flags to mail_search_register_arg

 lib-storage: index_storage_mailbox_enable() - Add MAILBOX_FEATURE_IMAP4REV2

imap: Fix registering IMAP4rev2 as imap feature

global: Drop support for old iconv()

iconv() is part of POSIX.1-2001.

lib-lua: Add input & output stream wrappers

lib-mail: Use explicit boolean expressions to fix compile errors

lib-mail: Use test_assert_strcmp_idx instead of custom debug code

fts: Explicit initializer names

lib-mail: Tests for attachment detection

lib-mail: Refactor and improve attachment detection

This also adds support for detecting Content-Type->name when Content-Disposition->filename
is not available. This only works in the case of Content-Disposition: attachment.

lib-mail: Fix clang-tidy lints

global: clang-tidy: disable recursion warnings

imap: Implement UTF8=ACCEPT for APPEND command

imap: Split off cmd_append_handle_parse_error()

imap: Refactor append handling

Simplifies next commit.

lib-imap: Add some tests for literal parsing

imapc: Propagate response codes

lib-storage: mail_storage_callbacks - Add notify_bad()

lib-imap: Add defines for all the Response Codes currently in IANA

auth: remove proxy_always extra field

settings: Use marker when Pro version is not specified

settings: Improvements to settings-history conversion script

- Some cleanup and refactoring
- Improve version handling
- Improve error messages and reporting

settings: Fix build issue with unsorted settings-history file

anvil: Change anvil socket's default group to default_internal_group

imap/test-imap-client-hibernate: Append new unhibernation statistics to expected fields

Forgotten in 3d415cf2dd393e1b4ad6e2be1a06bb221406509f

imap/test-imap-client-hibernate: Propagate failures in imap_hibernate_server()

The subprocess needs to propagate the error to the diligent parent process.

config: Use python3 instead of python

config: Change the generated settings-history-core from .h to .c

This changes the output file to a header file instead of a C file, this helps meson deal
better with dependencies on custom_targets.

It is technically an #included header file so it makes sense.

The ideal solution would be to keep it as a C file, and add forward declarations of the
arrays defined in it instead of verbatim #include'ing it. However, array forward decls are
not possible without a specified size which cannot be provided in advance and declaring
them as pointers is perhaps not the best idea.

config: Replace settings-history Perl script with Python rewrite

global: Add pyproject.toml for Python scripts