imap: cmd-list - Exit loop in cmd_list_continue() early upon connection failure

imap: cmd-list - Make the list iter flags a parameter for list_send_status()

Before, it was available only in the LIST command context. This is preparation
for adding infrastructure for dynamically adding support for new RETURN flags.

imap: cmd-list - Make the namespace a parameter for list_send_status()

Before, it was determined internally. This is preparation for adding
infrastructure for dynamically adding support for new RETURN flags.

imap: cmd-list - Collect parameters for list_send_status() in a struct

This is preparation for adding infrastructure for dynamically adding support for new RETURN flags.

imap: cmd-list - Add assert to list_send_status()

While listing subscriptions, but only child is subscribed, the appropriate
mailbox info flag must be also set.

imap-acl: Reformat imap-acl-plugin.c

m4: Update pkg.m4

lib-dcrypt: Fix ostream-encrypt error handling

Stream errno was not always set

config: Avoid duplicating version numbers when listing supported config versions

auth: Allow configuring passdb/userdb sql to use auth-workers

If passdb/userdb_use_worker=yes, this overrides the sql-driver specific
worker decision.

lib-storage: Clear storage errors if iteration failed due to no mailbox attribute dict

This doesn't normally matter, but if there are error handling bugs elsewhere
this error might wrongly show up in logs and cause confusion. It's better to
log "Unknown error" in that case.

lib-var-expand: Add backwards compability for older bison

Bison 3.5.1 does not add error symbol or handling, add them ourselves.

auth: lua - Remove dead code

fts-flatcurve: Switch from <stdio.h> to <cstdio>

fts-flatcurve: Fix potential crash when searching virtual mailboxes

Fixes:
Panic: file fts-search.c: line 87 (level_scores_add_vuids):
assertion failed: (array_count(&vuids_arr) == array_count(&br->scores))

fts-flatcurve: Support lock files in VOLATILEDIR

fts-flatcurve: Fix maybe queries

fts-flatcurve: Ignore non-selectable mailboxes when doing optimize/rescan

fts-flatcurve: Ignore alias mailboxes when doing optimize/rescan

fts: xml2text - Move lib_init()/deinit() to surround event_create() and unref()

Otherwise we crash at 23: .event = event_create(NULL)

config: Add explicit support for 3.1.0 version

Config version 3.0.x is no longer supported.

lib-var-expand: Use moduledir instead of pkglibdir for crypt

This is where the module is installed.

lib-dict: Add DISTCLEANFILES for dict-drivers-register.c

Makefile: Exclude conf files from dist uninstall check

auth: lua - Passdb/userdb lookups can no longer return a single string on success

The second return value must be either a table or nil. Returning a single
string was a bit unsafe since its values could accidentally become expanded
to spaces, causing unsafe behavior.

If the first return value indicates an error, the second can still be a
string.

auth: Remove unused passdb/userdb-template.[ch]

auth: lua - Remove auth_request#response_from_template()

This is a bit unsafe since it can't handle e.g. spaces in values.
It's also relying on otherwise obsolete template code.

auth: Move STATIC_PASS_SCHEME to passdb.h

auth: Expand debug message about auth failure delay handling

auth: Avoid auth failure delays entirely with auth_failure_delay=0

auth: Don't use auth failure delay for internal failures

Internal failures nowadays have their own delays.

auth: Split off auth_request_want_failure_delay()

lib-var-expand: Validate UTF-8 input

lib-var-expand: Fix lexer error passing

This way we can actually tell what the lexer did wrong.

lib-var-expand: fn_domain - Copy value before setting state

lib-var-expand: Reuse var_expand_state_set_transfer_data() in var_expand_state_set_transfer_binary()

It's same code

lib-var-expand: Ensure we don't try to set transfer from partial of it

imap: Fix crash with Maildir when trying to send INPROGRESS during mailbox syncing

Mailbox syncing could have been done outside command context, in which case
there is no global event.

global: Fix compiling without ldap

global: Replace PASSDB_LDAP/USERDB_LDAP with HAVE_LDAP macro

This macro is needed for other ldap-related code.

auth: Fix memory leak when building oauth2 failure JSON reply

Broken by 4bf8b11f5196752bb928baf59e48f9774bb0ce59

lib-smtp: Fix pipelining initial SASL response after AUTH

auth: Avoid delaying reply when shutting down

auth: Deinitialize connection already in auth_client_connection_destroy()

auth: Deinitialize passdb before freeing up settings

Some passdbs need the settings during deinitialization.

quota: Fix integer type in quota_warning_match()

Found by static analysis

lib-var-expand: Fix potential crash if referring to too large regexp capture group

Using sizeof() will not give number of elements here.

lib-var-expand: Ensure key is not NULL in fn_hash()

Satisfies static analysers

lib-var-expand: Fix memory leak if filter fails

Broken in a9d9eb4297a2cccec0cf698a45efb4f1987bdc63

lib-settings: Fix sorting named list filter overrides

It should check that both set1 and set2 have non-zero
filter_element_count.

Broken in e2a3a0a6915668a44750aa30b70fa39caf22be4a

fts: Use FTS_USER_CONTEXT_REQUIRE in fts_mail_user_init()

User is required to be present.

Broken in 6634e45500e81cfa36932203ee69c81745efd3e6

fts-solr: Require user context

Broken in 324adafd921edddc83606af9ea97d9c4587ea920

fts-flatcurve: Require user context

Broken in b4ca8f7e58a0a7561a314fe733c65914966d9046

auth: Remove useless variable in db_oauth2_var_expand_func_oauth2()

NEWS: Add news for v2.4.0

NEWS: Fix formatting to match editorconfig

editorconfig: Use tabs with NEWS file

login-common: Add proxy_session_reconnecting event and log it as warning

Even if the login succeeds on a retry, it can still indicate e.g. network
packet loss, which can be useful to see in log files as warning.

lib-settings: Allow spaces in "key +=value"

This is useful in configuring userdb_fields { key += value }

config: Use isspace() for checking whitespace

global: Update documentation links in source code

INSTALL: Update documentation links for 2.4

README: Master RFC list is now maintained in documentation

global: Set application protocol on new context only

lib-ssl-iostream: Change ssl_iostream_context_cache_get() to return 1 on new context

This allows differentiating when a new context was actually created.

lib-ssl-iostream: Set context application protocols only once

lib-ssl-iostream: Fix compiling with libressl

libressl has no SSL_OP_NO_RENEGOTIATION. Fix this by disallowing channel
binding with libressl when using TLSv1.2 or older.

auth: mech-oauth2 - Use json ostream instead of generator

configure.ac: Remove unused want_devel_checks

lib-settings: Rename DEVEL_CHECKS ifdefs to DEBUG

imap: Move ENABLE's mail_utf8_extensions check to UTF8=ACCEPT callback

imap: Allow feature callbacks to not enable the feature

imap: Fix ENABLE UTF8=ACCEPT to not return it to client if mail_utf8_extensions=no

lda: Remove check to fail if binary is setuid-root and world-executable

It's not safe to use as setuid-root even when not world-executable.
Remove the check so people might not get wrong ideas about its safety.

lib-lua: Add event.[un]set_forced_debug()

fts-flatcurve: Fix deadlock in flatcurve indexing

The deadlock was caused by not releasing soon enough the writable database,
causing the search process and the indexer process to conflict

lib-var-expand: test-var-expand - Use test_assert_idx

lib-var-expand: test-var-expand - Fix expected message to match on older bison

Bison 3.0.4 produces different error

auth: Remove duplicate prefix from providers

lib-var-expand: Remove duplicate prefix from providers

lib-var-expand: Prefix provider errors with provider prefix

auth: Set error when field not found in auth_request_fields_var_expand_lookup()

lib-var-expand: Ensure provider returns either error or value

master: Add suggestion to use "unlimited" for some service error messages

auth: Change hardcoded 3 to AUTH_CLIENT_MINOR_VERSION_CHANNEL_BINDING

lib-var-expand: Drop -Wcounterexamples from bison

It's not needed. Forgotten in 2b8d353de6ee61793a411cc1729cf9cff857ce34

auth: Add "auth-legacy" listener type

This is needed at least by Exim authentication until it gets updated.

auth: auth_client_connection_create() - Change boolean parameters to enum

auth: userdb sql - Fix memory leak when user iteration fails with invalid settings

imap-login: Fix accessing freed memory when using IMAP COMPRESS

Broken by "imap-login: Fix leaking settings instance memory" commit.

config: Fix error handling for illegal filter ordering

lib-settings, config: Use named (non-list) filter hierarchy length also for sorting overrides

lib-settings: Fix override ordering when mixing multiple override types

lib-settings: Skip unmatched settings overrides earlier

global: Remove SETTINGS_EVENT_MAILBOX_NAME_WITH[OUT]_PREFIX macros and special handling

The mailbox settings are now looked up with settings_filter_names, so
there's no need to do this kind of special translations.

lib-settings: Fix seen_filter check to work properly with multiple settings_name_filters

lib: Add event_filter_get_root_node()

global: Remove event fields no longer necessary for settings lookups

settings_filter_names are now used for settings lookups.

lib-settings: Change overrides to use only settings_filter_names

config: Require named list filter lookups to use settings_filter_name

The config client callers are now responsible for doing lookups with e.g.
settings_filter_name=namespace/inbox in the event, rather than relying on
just namespace=inbox being in the event.

This allows separating the filter name from the setting values. So for
example userdb can change the mailbox name (mailbox/spam/name=Junk).