m4: quota.m4 - Modernize m4 syntax

m4: q_quotactl.m4 - Modernize m4 syntax

m4: pr_set_dumpable.m4 - Modernize m4 syntax

m4: notify.m4 - Modernize m4 syntax

m4: mntctl.m4 - Modernize m4 syntax

m4: mmap_write.m4 - Modernize m4 syntax

m4: linux_mremap.m4 - Modernize m4 syntax

m4: libcap.m4 - Modernize m4 syntax

m4: ipv6.m4 - Modernize m4 syntax

m4: iovec.m4 - Modernize m4 syntax

m4: ioloop.m4 - Modernize m4 syntax

m4: gmtime_tm_gmtoff.m4 - Modernize m4 syntax

m4: gmtime_max.m4 - Modernize m4 syntax

m4: glibc.m4 - Improve glibc-check

The previous check was based on a compilation error and negating the
value. This updated check is more clear as it runs the preprocessor with
the appropriate header file and checks if the output matches, which in
turn allows for a more forward readable definition of the
i_cv_have_glibc value.

m4: glibc.m4 - Modernize m4 syntax

m4: Delete flexible_array_member.m4

Autoconf is recent enough.

m4: fd_passing.m4 - Modernize m4 syntax

m4: dovecot.m4 - Add missing ;; to case

m4: dovecot.m4 - Update compiler checks

Require at least C99 compiler and try to use gnu11, gnu99, c11, c99.
This avoids using too modern syntax.

m4: dovecot.m4 - Modernize m4 syntax

m4: dbqlk.m4 - Modernize m4 syntax

m4: crypt_xpg6.m4 - Modernize m4 syntax

m4: blockdev.m4 - Modernize m4 syntax

m4: arc4random.m4 - Modernize m4 syntax

m4: ac_typeof.m4 - Modernize m4 syntax

configure.ac: Modernize m4 syntax

lib-storage: Make NO-NOSELECT the default

Use KEEP-NOSELECT to revert the old behaviour.

lib-storage: struct mailbox_list_settings - Use bit fields for flags

plugins: Drop mailbox-alias

lib-storage: mbox_sync_handle_eof_updates() - Fix for crash on implicit conversion

implicit conversion from type ‘unsigned long’ of value
0xFFFFFFFFFFFFF3FD (64-bit, unsigned) to type ‘off_t’ (aka ‘long’)
changed the value to -3075 (64-bit, signed)

lib-index: Fix updating minor_version for old index files

If the index was originally created between Dovecot v1.1 and v2.1, the
minor_version wasn't updated. This caused log2_rotate_time and
last_temp_file_scan fields to become reset to 0 constantly, which
prevented optimizations using those fields.

lib-login: Move login_request/reply to login-interface.h

These are used by both the login-client and login-server.

lib-login: Add comments to .h file headers explaining what they are doing

lib-login: Rename "master login auth" structs/defines/functions to "login server auth"

lib-login: Rename master-login-auth.[ch] to login-server-auth.[ch]

lib-login: Rename "master login" structs/defines/functions to "login server"

 * master_login -> login_server
 * master_login_client -> login_server_request
 * other master_login_* -> login_server_*

lib-login: Use i_close_fd() where possible

lib-login: Rename master-login.[ch] to login-server.[ch]

login-common: Replace master-login.h include with login-client.h

lib-login: Rename "master auth" structs/defines/functions to "login"

 * master_auth -> login_client_list
 * master_auth_connection -> login_connection
 * master_auth_request -> login_request
 * master_auth_reply -> login_reply
 * master_auth_request_params -> login_client_request_params
 * MASTER_AUTH_* mostly to LOGIN_REQUEST_*, but also LOGIN_REPLY_* and LOGIN_CLIENT_*

lib-login: Rename master-auth.[ch] to login-client.[ch]

This better describes what it's doing.

doveadm: doveadm-auth - Remove unused master_auth_request variables

auth: Fix assert-crash if auth-master client sent too long cookie in REQUEST

Only root can connect to auth-master socket by default, so this shouldn't
have caused any real problems.

global: Use AUTH_CLIENT_PROTOCOL_MAJOR/MINOR_VERSION also for auth-master socket

Both auth-client and auth-master are using the same auth protocol. There's
no need to have separate version numbers for auth-master socket, it's just
confusing things. The auth server's auth-client version checking code was
already wrongly using AUTH_MASTER_PROTOCOL_MAJOR_VERSION.

lib-login, login-common: Rename master_auth_request_full() to master_auth_request()

lib-auth: Merge test-auth-master and test-auth-master2

master: Move test-auth-* unit tests to lib-auth

Now that it no longer depends on lib-master, there's no cyclic dependency
problem.

Because test-auth-master already exists, this commit temporarily creates a
second test-auth-master2.

master: test-auth-* - Remove lib-master dependency

master: Move test-master-login-auth unit test to lib-login

lib-master: Move master-auth/login code to lib-login

This makes it clearer that this code is only used for "login services"
while lib-master is used by all services.

lib-master: master_service - Remove dependency for master_login.

global: Move setting push_notification_backend to old-set-parser

config: old_settings_handle_path() - Handle case where path/<x>/ has x other than 0

lib-imap: fuzz-imap-bodystructure - Fix parsing to be used correctly

imap_bodystructure_parse_full() should be used to generate the message_parts
correctly based on the input string. This fixes wrongly triggered panics.

global: Rename auth_default_realm into auth_default_domain

auth: Removing stray whitespaces

doc/example-config: Removing stray whitespaces

lib-storage: Disable all events for raw user/storage

The events can be enabled for debug logging, but not for statistics
and such. These were just causing confusion.

lib: Add event_disable_callbacks()

lib-smtp: Clarify how timeouts actually work

connect_timeout_msecs can't ever default to command_timeout_msecs, because
a zero timeout gets converted to the default 30s timeout.

mail-crypt: mail_crypt_user_set_private_key() - Move mailbox alloc later to avoid a panic

This fixes:

  Panic: file mail-user.c: line 252 (mail_user_deinit): assertion failed: ((*user)->refcount == 1)

When mail_crypt_require_encrypted_user_key is enabled and the user does
not have key or password.

mail-crypt: Remove unused mail_crypt_box_get_or_gen_public_key()

lib: unichar - Don't return 0 from uni_utf8_get_char_n() when NUL byte is encountered.

This caused problems for code that uses this function to read from a
size-limited buffer that can contain NUL bytes; i.e. lib-smtp. In that case,
hitting a NUL byte yields the same result as hitting the end of the buffered
data, which is unacceptable. No other code relies on the function returning 0,
so this can be changed safely to returning -1 instead.

lib-http: http-client-request - Fix handling of request aborted before submission.

Modifying an aborted request was not always allowed, which could yield assertion
errors when the request was aborted before submission. This would for example
happen when a payload was assigned to a new request created from an invalid URL.

dsync: Add more debug logging

dsync: Fix dsync_features=no-header-hashes to match UIDs

Mails were previously assumed to be the same, even if their UIDs didn't
match. This caused the sync to misbehave in some situations.

lib-lua: Add DLUA_TABLE_STRING_SELF()

lib-dict: dict-lua - Export dovecot.dict.ITERATE_FLAG_*

lib-lua: Add DLUA_TABLE_ENUM_NOPREFIX()

lib-dict-backend: dict-sql - Implement dict_expire_scan()

dict: Add dict-expire service

If enabled with service dict-expire { process_min_avail=1 }, this service
will go through all configured dicts and delete all expired fields every
30 seconds.

lib-dict: Add dict_expire_scan()

dict-client: Fix potential crash if iteration fails to send the command

lib-lua: Fix atpanic error message with Lua threads

The function used the main thread's state for finding the error
instead of the thread-specific state.

lmtp: proxy - Add "dest_host" and "dest_ip" to lmtp recipient event

lib-dict: dict-lua - Support expire_secs parameter in dict_transaction_begin()

dict-sql: Add support for dict_op_settings.expire_secs

Add a new map { expire_field } to specify the expiration timestamp field in
sql table. This field is automatically updated when
dict_op_settings.expire_secs is used. Also when lookup/iteration sees an
expired row, it's skipped over.

dict-redis: Add support for dict_op_settings.expire_secs

dict-redis: Use %zu printf format for strlen()

dict-client, dict: Add support for dict_op_settings.expire_secs

doveadm dict set: Add -e <expire-secs> parameter

lib-dict: Add dict_op_settings.expire_secs

This may be implemented by dict drivers to provide built-in expiration.

lib-dict: Support early rollback if dict_transaction_context.error is set

lib-dict: Split off dict_transaction_rollback_run()

lib-dict*: Use named initializer for dict_vfuncs

lib-master: Use ssl_require_crl setting only for server-side SSL settings

We don't currently properly support checking CRLs when acting as SSL client.
The CRL would have to be stored as part of the CAs, which isn't commonly
done. This bug has been in the code ever since it was added in
30c5c1fc3608ae575f11960281d3e338b6bf7bc8, but it became more noticeable
with recent changes that started using lib-master for getting all SSL
client settings, e.g. 1e5324b5805bf7299cd8196f7b659fe935f027bd

config: old-set-parser - Handle CONFIG_LINE_TYPE_KEYFILE as well

This covers settings with value =< /file.

Forgotten in ff6c0588196da567197da323865821cd51af8f90

global: Remove pre OpenSSL v1.0.2 workarounds

These are also present in libressl 2.4.2

lib-dcrypt: Remove gnutls

It never worked

global: Make OpenSSL v1.0.2 mandatory requirement

m4: ssl.m4 - Modernize m4 syntax

lib-auth: Remove auth_client_cache_flush_started|finished event

This was being sent by just doveadm which isn't very useful.

doveadm deduplicate: Allocate memory properly for keys in hash table

This caused wrong mails to be deleted somewhat randomly.
Broken by 320844f50cd669b602d30210e2e5216f65d2050f

indexer: indexer_queue_append_request() - Reorganize the code for better readability

Isolate the section handling duplicated entries from the part creating creating new ones.

indexer: indexer_queue_deinit() - Assert queue->users last

In case of panic, this allows to discriminate if the mismatch is in the count
of requests between queue and queue->requests count or between queue->requests
and queue->users

indexer: indexer_queue_append_request() - Fix repeated insertions in queue->users indexer_queue(s)

The repeated insertion happens when invoking for an existing entry and
append=FALSE. At that point, the section dealing with queue->users is
erroneously executed. This will cause later a panic in indexer_queue_deinit()
due to the extra entries remaining in the table

indexer: Remove stray whitespaces

lib-fs: Remove assert for !file->writing_stream from fs_delete.

The assert is not particularly useful and disturbs obox uncertain
write objects remove attempts.

lib-storage: Add "service" field to user event

lib-storage: Override service->name with input->service when available