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.

global: Use setting_parser_info.parent only for structs that are used as arrays

This will be needed by the following commits, although the whole
parent-child relationship will go away in later commits.

doveadm: Remove explicitly set master_service_settings_input.protocol

It is now unnecessary.

lib-master: If master_service_settings_input.protocol isn't set, default to master_service.name

lib-master, global: Rename master_service_settings_input.service to protocol

It is actually protocol, although the only place where they differ is login
processes (service=imap-login, protocol=imap).

lib-master: Log a warning about leaked settings structs at deinit

lib-master: Delay setting master_service to NULL

This will be required by the following changes.

auth: Stop reading settings multiple times

lib-storage, lda: Remove unused MAIL_STORAGE_SERVICE_FLAG_USE_SYSEXITS

lib-storage: Remove unused mail_storage_service_read_settings()

lib-storage: mail_storage_service_lookup*() - Stop reading settings

The caller is now expected to have read them already.

global: Stop assuming mail_storage_service_lookup*() is going to read settings

lib-storage: mail_storage_service_init_settings() - Stop reading settings

The settings should already have been read. Just access them.

lmtp: Replace mail_storage_service_read_settings() with master_service_settings_instance_new()

There's no need to actually read the settings, since they're already read.
LMTP client just needs a settings instance.

quota-status: Replace mail_storage_service_read_settings() with master_service_settings_read_simple()

lib-master: Use settings_parse_keyvalue_nodup()

lib-settings: Add settings_parse_keyvalue_nodup()

lib-settings: Remove internal code for multiple settings roots

lib-settings, global: Replace settings_parser_get_root_set() with settings_parser_get_set()

There is only one root now, so the root parameter isn't necessary.

lib-settings: Remove ability to use multiple roots in setting_parser_context

lib-storage: Unwrap set_keyval() wrapper function

It had become a very small wrapper, and the code is clearer without it.

lib-master, global: master_service_set() - Remove error handling

This function can no longer fail.

lib-master, lib-storage: Remove unused fields from master_service_settings_input

The username, local_ip and remote_ip are now looked up with the
master_service_settings_get() calls rather than at this early stage.

mail-crypt: Stop reading settings

The settings can now be accessed without reading them again.

doveadm: Stop reading per-connection settings

This is no longer necessary. The filtering is done by
master_service_settings_get()

doveadm: Add local_ip and remote_ip fields to connection event

These will be required to preserve support for local {) and remote {}
config filters after the following change.

login-common: Stop reading per-connection settings

This is no longer necessary. The filtering is done by
master_service_settings_get().

This commit also adds "local_name" field to client's event when TLS SNI is
used.

lib-storage: Remove mail_storage_vfuncs.get_setting_parser_info()

This is no longer necessary.

lib-storage, global: Remove "settings roots"

These were no longer used for anything.

lib-master, global: Remove "settings roots"

It's no longer necessary to know what settings the process will use.

lib-master: Delay settings parsing until master_service_settings_*get()

lib-master: Change master_service_settings_find() to be static

lib-master, lib-storage: Move key+=append and plugin/ handling to lib-master

This is necessary because all settings roots won't be known after following
changes. The plugin/ handling should eventually go away once all plugin
settings have been converted.

lib-master, lib-storage: Move -o parameter checks inside master_service_set()

lib-master: Split master_settings_mmap parsing from applying the settings

lib-master: Store filters in struct master_settings_mmap

lib-master: Remove unused config_cache_size setting

config: old-set-parser - Move imap_id_log handling to removed_settings[]

config: old-set-parser - Move login_access_sockets handling to removed_settings[]

config: old-set-parser - Merge "key is no longer needed" settings into "key has been removed" settings

There's not much of a difference between these.

config: old-set-parser - Move removed settings into an removed_settings[] array

lib-master, global: Prepare master_service_set() for type parameter

lib-master, global: Introduce master_service_settings_instance

This commit mainly consists of renaming "settings parser" to "settings
instance" in various variables and function names. The following commits
make it more useful.

lib-storage: Remove unused mail_storage_service_user_set_setting()

global: Replace mail_storage_service_user_set_setting() calls with master_service_set()

lib-storage: Don't keep mail_user.set_parser referenced

It's always just a direct pointer to service_userr->set_parser, so the
reference isn't needed. Also the following changes will change this to
a non-reference counted "settings instance", which makes this change
necessary.

lib-storage, global: mail_storage_service_read_settings() - Return duplicated parser

This will simplify following changes.

doveadm, lmtp: Use mail_storage_service_user_set_setting()

lib-storage: Replace direct settings-parser API calls with lib-master API

lib-master: Add master_service_settings_find()

lib-master: Change master_service_set() API

It wasn't currently used by core, and this API makes it close to what we
want it to be eventually.

lib-master: Remove unused master_service_get_settings_parser()

lib-settings: Remove unused settings_parse_is_valid_key()

lib-storage: Avoid separate settings_parse_is_valid_key() call

The same information can be found with settings_parse_get_value().
This will simplify the following changes.

lib-storage: Use mail_storage_service_user_set_setting() where possible

lib-storage: Fail if userdb returns key+=value for non-string

Previously an error was logged, but the user initialization didn't fail.
However, it's safer to fail since it's a bad configuration.

lib-storage: Cleanup - Change set_keyvalue() to return error_r

lib-storage: Cleanup - parse userdb settings with settings_parse_keyvalue()

lib-settings: Don't allocate settings error from parser pool

This avoids wasting memory if caller wants to ignore errors and continue
parsing.

lib-storage: Rename unexpanded_set_parsers to set_parsers

The settings are no longer expanded inside set_parsers, so they're always
unexpanded.

lib-storage: Don't duplicate mail_user.set_parser

It's always a pointer to mail_storage_service_user.set_parser, which is
already user-specific.

lmtp: Rename unexpanded_lda_set to lda_set

Also expand them, although there are currently no STR_VARS settings to be
expanded.

lib-storage: Remove unused mail_user_var_expand()

lib-storage, global: Move settings expansion out of mail_user_init()

master_service_settings_parser_get() calls can now decide whether to expand
settings or not.