config: Rename config_module_parser.error to delayed_error

config: Replace global config_filter variable with local/static ones

It is still a static variable in some cases, but that's still more
manageable.

config: config_dump_full() - Add config_filter parameter

Use the parameter instead of global variable.

config: config_export_by_filter() - Add config_filter parameter

Use the parameter instead of global variable.

config: config_parse_file() - Return config_filter instead of setting it globally

config: Run settings check_func()s individually for each filter sections

The previous behavior was to apply parent filters as well before running the
check_func()s. This is currently causing too much complexity for the code to
be worth it though.

doveconf: Remove unnecessary "static" from variables

config: Remove wrongly used ATTR_UNUSED

config: Remove unused master_service_settings_output handling

config: Fix memory leak when exporting filters

config: Fix doveconf section output when char is unsigned

This happened at least with ARM CPU.

config: Add missing data stack frame when exporting filters

lib-settings: settings_*dup*() - Update pool field in destination settings

lib-settings: Add data stack frame when filling specific_services

doveconf -F: Don't write binary garbage to stdout

doveadm: Fix -O parameter handling broken by earlier changes

lib-login: Optimize session ID parsing

*-login: Send local_name to next hop

*-login: Accept local_name from remote host

imap-login: Fail ID with invalid parameters

imap-login: Move client_try_update_info() into cmd_id_handle_keyvalue()

Simplifies next commit.

imap-login: Use temporary structure for handling ID parameters

This allows discarding values if the ID request had problems.

lmtp: Send local_name with XCLIENT to next hop

lmtp: Accept local_name from XCLIENT

lmtp: Include local_name in auth request, so it ends up lib-storage

auth: Include local_name in userdb reply if it's available

auth: Fix use-after-free at deinit

login-common: Fail proxying if local_name is invalid

This is safeguard against sending malformed data that could
lead to unexpected override of fields in remote end.

lib-smtp: Send and accept DESTNAME parameter for XCLIENT

lib-storage: Allow filtering settings by local name {}

lib-smtp: Add accessor for TLS servername

lib-master: Reject invalid hostname in master_service_haproxy_parse_tlv()

lib-ssl-iostream: Reject bad servername extension

lib: connection - Add connection_is_valid_dns_name()

Checks that given name is valid DNS servername, including
characters _ and RFC5891 section 4.2.1 specified ones.

lib-master, global: master-service-settings.h - Don't automatically include settings.h

lib-master: Move settings_*() to lib-settings

lib-master: Add and use settings_has_mmap()

lib-master: master-service-settings.h - Remove unnecessary includes

lib-master: Split off settings_read()

lib-master: Settings reading - Fix failure handling when executed via doveconf

The DOVECOT_CONFIG_FD_ENV environment check didn't work, because it was
already unset.

Also this simplifies the code a bit. There's no need to re-parse the config
if it wasn't just mmaped, since nothing would have changed.

lib-master: Access settings_root via all created event roots

lib-master: settings_get*() - Require event to be non-NULL

global: Replace NULL event for settings_get*() with master_service.event

lib-lua: Get settings using dlua_script.event

lib-master: Add master_service_get_event()

lib-master: Move master_service.settings_pools tracking to settings_root

lib-master: Rename master_settings_pool to settings_mmap_pool

lib-master: Rename master_service_var_expand_init() to settings_var_expand_init()

lib-master: Rename master_service_settings_var_expand_t to settings_var_expand_t

lib-master, global: Rename MASTER_SERVICE_VAR_EXPAND_* macros to SETTINGS_VAR_EXPAND_*

lib-master, global: Rename master_service_settings_free() to settings_free()

lib-master, global: Rename master_service_settings_get*() to settings_get*()

lib-master, global: Rename master_service_settings_get_flags to settings_get_flags

lib-master, global: Rename master_service_set() to settings_override()

lib-master, global: Rename master_service_set_type to settings_override_type

lib-master: Change settings_{mmap|instance} to not depend on master_service

lib-master: Use master_service.config_overrides via settings_root

Copy them to settings_root the first time settings are read.
The following settings lookups use them via the settings_root.

lib-master: Move struct settings_override earlier

lib-master: Move master_service.set_protocol_name to settings_root

lib-master: Change settings_mmap_[un]ref() to be static

lib-master: Introduce struct settings_root

It contains the latest created settings_mmap.

lib-master: Rename struct master_service_set to struct settings_override

Also rename settings_instance.settings to overrides.

lib-master, global: Rename MASTER_SERVICE_SETTINGS_INSTANCE to SETTINGS_EVENT_INSTANCE

lib-master, global: Rename struct master_service_settings_instance to settings_instance

lib-master: Rename struct master_service_mmap_* to settings_mmap_*

lib-master: Rename master_service_settings_mmap_*() to settings_mmap_*()

lib-master: Rename struct master_settings_mmap to struct settings_mmap

lib-master: Rename config_mmap variables to mmap

master_service.config_mmap is kept for now.

lib-master: Change master_service_settings_instance_get() to be static

lib-storage: Remove unused mail_user.set_instance

global: Replace master_service_settings_instance_get() calls with master_service_settings_get()

This is now possible because the lookup event has
MASTER_SERVICE_SETTINGS_INSTANCE set.

lib-storage, lmtp: Set MASTER_SERVICE_SETTINGS_INSTANCE for events

lib-master: master_service_settings_get() - Use instance from event if it exists

Added MASTER_SERVICE_SETTINGS_INSTANCE that can be used to set the instance
to events.

global: Use more specific events for settings lookups

lib-settings, global: Rename settings.[ch] to settings-legacy.[ch]

It will eventually be removed, so don't bother changing the function names.

lib-master: Add test-master-service-settings unit test

Initially it tests all the error code paths in binary config file reading.

global: unit tests - Use test_create_temp_fd() instead of reimplementing it

lib-test: Add test_create_temp_fd()

lib-settings: Remove unused settings_parse_set_key[s]_expanded()

If these are ever needed to be added back, they should be fixed to handle
SET_ALIAS correctly.

doveconf: Add -w parameter to hide warnings about obsolete settings

This is used by managesieve's dump-capability to avoid logging them twice.

doveconf: Remove -e parameter from getopt string

Forgotten in 649ab77ece61bee900e9c39b4db24f09fbc22590

master: Don't delay settings errors at startup

Recent change added delayed errors, which are useful for doveadm/etc, but
they are confusing/harmful for the dovecot startup.

config: Remove now unused disable-check-settings request parameter

doveconf: Remove now unused -E parameter to disable settings checks

lib-master, global: Remove master_service_settings_input.disable_check_settings

This is no longer necessary.

config: Remove kludge to skip SSL server settings with doveadm

The previous changes make this unnecessary. The SSL server setting read
failures are delayed until they are attempted to be used, but doveadm
won't actually try to use them.

Reverts 310f9fac3abe5e9889262bd679af67343c8b4af0

config: Delay handling settings check errors when doing "full" config lookups

config: Delay handling errors in settings values when doing "full" config lookups

The config client is responsible for giving error. However, if the config
client never accesses the setting struct, no error is reported.

lib-master: Check that settings blobs in config binary format end with NUL

This allows using strlen() safely later on.

config, lib-master: Optimize binary config format - split settings blocks by their structs

This allows looking up settings only for the requested settings struct
without reading all the other settings as well.

Additionally this new format allows delaying error handling. For example
if "ssl_key = <path" handling can't open the path because it doesn't have
permissions, the error is delayed until SSL server settings are actually
used by the config client. This is beneficial when the client never even
attempts to use the SSL server settings, in which case the lack of
permission is irrelevant and no error is ever seen. This example
behavior doesn't actually yet work in this commit though.

config: Unlink temporary binary config file on failure

lib-master: Avoid duplicates in master_service_settings_output.specific_services

config: Split off config_export_parser()

Also add config_export_get_parser_count() and
config_export_parser_get_info(), which are related to it.

config: Move config_dump_full_write_filter() earlier

config: Rename config_export_parsers() to config_export_set_parsers()

config: Handle config_export_by_filter() failures earlier

config: Make config_export_free() public and use it for aborting export

config: Rename config_export_finish() to config_export_all_parsers()

lib-settings, global: Convert setting_parser_info.module_name to a unique name

This unique name will be used by the binary config file format in the
following commits.

config: get_setting() - panic if setting isn't found

This function is used only for settings that are expected to be found.
Panic makes it easier to notice if there are typos.