doveadm-server: http: Removed http_request field from struct client_connection_http.

Not useful to put it there.

doveadm-server: http: Renamed conn->http_client to conn->http_conn.

doveadm-server: Restructured client-connection code so that TCP and HTTP connection types are properly separated.

doveadm-server: http: Moved client_connection_destroy_http().

doveadm-server: tcp: Renamed client_connection_create() to client_connection_tcp_create().

doveadm-server: http: Renamed client_connection_create_http() to client_connection_http_create().

doveadm-server: http: Renamed conn->client to conn->conn.

doveadm-server: tcp: Moved client_connection_destroy().

doveadm-server: tcp: Cleaned up doveadm_server_log_handler().

doveadm-server: Moved common client code back from client-connection-tcp.c to client-connection.c.

doveadm-server: Renamed client-connection.c to client-connection-tcp.c.

Preparation for subsequent commits.

doveadm-server: http: Changed index variables from size_t to unsigned int where size_t makes no sense.

doveadm-server: http: Changed initialization of parameters in doveadm_http_handle_json_v1().

Use normal assignment, rather than memcpy().
This way, type checking is not skipped unnecessarily.

doveadm-server: http: Use `ret' rather than `rc' as name for the local variable used for return values.

doveadm-server: http: Cleaned up code formatting.

doveadm-server: http: Created local variable for conn->cmd_param->value.v_istream as an abbreviation.

This is a preparation for subsequent commits.

doveadm-server: http: Created local variable for conn->client.output as an abbreviation.

This is a preparation for subsequent commits.

doveadm-server: http: Created local variable for conn->http_request as an abbreviation.

This is a preparation for subsequent commits.

doveadm-server: http: Created local variable for conn->http_response as an abbreviation.

This also makes sure it is named consistently everywhere.
This is a preparation for subsequent commits.

doveadm-server: http: Created local variable for conn->http_server_request as an abbreviation.

This also makes sure it is named consistently everywhere.
This is a preparation for subsequent commits.

doveadm-server: http: Moved loop body in doveadm_http_server_read_request_v1() to separate function.

doveadm: mail: Removed redundant cur_username field from struct doveadm_mail_cmd_context.

doveadm: mail: Removed redundant cur_client_ip field from struct doveadm_mail_cmd_context.

doveadm: mail: Removed struct doveadm_cmd_context function parameters that are now useless.

It can be accessed from struct doveadm_mail_cmd_context.

doveadm: Changed command contexts to contain the input, output, and connection type values directly.

Before, it used a direct pointer to the connection.
It used also flags to indicate the connection type, which is now consolidated in the connection type enum.

lib-http: server: Implemented http_server_request_fail_text(), which allows creating failure responses with verbose payload.

lib-http: server: Allow determining at any time whether the connection will be closed after handling a request.

Previously, this was only possible during response submission, which potentially requires making a `close' parameter available for any API function that submits a (failure) response.
This would become very ugly and inflexible.

lib-http: server: Set `Connection:' header in response to `close' when request contains the same.

Before, it would set it to `Connection: keep-alive', which makes no sense.

lib-http: server: Make sure broken connection input always causes the connection to be closed after response.

maildir: Fix memory leak when mailbox opening fails

lib-fts: Fix test-fts-filter to use valid unicode

test-json-parser: Use idx to better identify failing case

test-json-parser: Update tests

Now that we actually check for valid unicode,
the tests need to have valid unicode and invalid unicode
in correct places.

test-unichar: Add test for surrogate handling

test-unichar: Update test to conform RFC3629

unichar: Do not accept invalid ranges

As per RFC3629

json-parser: Fix unicode parsing

Check for surrogates and validate ucs4 codepoint.

unichar: Use surrogate macros in ucs4 validity check

unichar: Add surrogate handling

unichar: Add uni_is_valid_ucs4

lib-master: Change instances.lock settings to make it less likely to fail

With use_io_notify it's spending less time usleep()ing. Use also a slightly
longer timeout just in case.

If a lot of instances were started at the same time, some of them could have
failed with:
master: Error: file_dotlock_open(.../instances) failed: Resource temporarily unavailable

lib-storage: Fix assert-crash when searching header and MIMEPART

For now this is just a bit kludgy workaround. The proper fix requires
larger changes, which aren't worth the effort right now.

For example:
doveadm fetch -u testuser uid MAILBOX inbox FROM foo MIMEPART FILENAME CONTAINS bar BODY baz

Crashes with:
Panic: file index-mail-headers.c: line 294 (index_mail_parse_header): assertion failed: (part != NULL)

quota-maildir: Don't deref NULL pointer on list deinit

lmtp: Fixed passing SMTP submit settings to plugins.

This caused Sieve to crash upon SMTP message submission; e.g., during redirect.

lib: printf_format_fix*() - Support '*' in precision

We're commonly using "%.*s"

lib: printf_format_fix*() - Move minimum field width check to its own function

lib: printf_format_fix*() - Use the same 4 digits as maximum precision length

I forgot the precision can also be used to truncate strings, not just
specify the precision for floating point numbers. So it makes more sense
that the limit is the same as for minimum field width.

quota-maildir: Improve error_r strings

quota-fs: Improve error_r strings

quota-dirsize: Improve error_r strings

quota-dict: Improve error_r strings

quota-count: Improve error_r strings

quota-imapc: Improve error_r strings

quota: Add error_r to update() of quota_backend_vfuncs

quota: Add error_r to get_resource() of quota_backend_vfuncs

quota: Add error_r parameter to init_limits() of quota_backend_vfuncs

quota: Add error_r to quota_get_resource()

quota: Add error_r to quota_transaction_set_limits()

quota: Add error string to quota_set_storage_error()

quota: Add error_r to quota_settings->test_alloc()

quota-maildir: Remove T_BEGIN..T_END blocks

This is to enable using t_strdup_printf() with error_r later on.

quota-fs: Remove T_BEGIN..T_END from fs_quota_get_resource()

This is to enable using t_strdup_printf() with error_r later on.

quota-dirsize: Remove T_BEGIN..T_END from dirsize_quota_get_resource()

This is to enable using t_strdup_printf() with error_r later on.

quota-dict: Remove T_BEGIN..T_END blocks

This is to enable using t_strdup_printf() with error_r later on.

quota: Initialize quota_backends with designated initializers

auth: Fix %{ldap_dn} not to leak memory

acl: Don't deref a potentially uninitialized pointer in acl_global_file_read

lib: printf_format_fix*() - Be over-strict in what format strings are allowed

The checks could have been bypassed by some invalid format strings that were
handled differently by the printf_format_fix*() code and libc. For example
"%**%n" was passed through as ok, but glibc handled the %n in it.

Found by cPanel Security Team.

libpassword: Add unit tests

auth: Fix compiling crypt-blowfish with i386

We're not going to support any assembly code, so remove BF_ASM entirely.

doveadm-server: http: Fixed lingering connections after the request is sent.

Turns out conn->http_client is already NULL while doveadm_http_server_connection_destroy() is called.
This is because http_server_connection_unref() sets it to NULL;
Fixed by removing useless HTTP connection reference.

cassandra: Support "timestamp" type fields properly

Fixes setting them with prepared statements. Reading them never worked
earlier.

cassandra: Include "prepared" when logging about prepared statement queries

Mainly useful for debugging/testing.

cassandra: Fix using bigint types with unprepared statements

This reverts the code to not using the statements with binding at all.
Alternative fix would be to start using explicit int32 or int64 parameter
types, but that breaks backwards compatibility a bit.

libpassword: Deinitialize hash iterator in password_scheme_detect

Otherwise it will leak memory

ipc: Improve error logging when IPC server is disconnected

lib-dict-backend: Fix linking test-dict-sql

dict_drivers_register_builtin() wasn't found.

libpassword: Add BLF-CRYPT support independent of crypt

auth: Fix typo in blowfish name

lib-fs: Fix fs_wrapper_write_stream_finish() to work with async parent fs

This only became a problem with the previous fs-compress change.

login-common: Use HAproxy provided proxy.ssl information

If the connection is proxied via system that can terminate
ssl for us, such as HAproxy, use that information only.

lib-master: Support HAproxy TLVs

This allows passing in information about SSL and client
certificate(s) and other information from haproxy.

lib-master: Use pooled memory for haproxy

lib-master: Fix haproxy reading code to read data incrementally

libpassword: Add support for ARGON2 scheme

configure: Detect libsodium presence

acl plugin: avoid "'struct stat' declared inside parameter list" warning

fs-compress: Allow compress level 0 to skip compression

This can be useful when combined with the "maybe-" prefix, so Dovecot will
support reading compressed files without creating new ones.

Added "ULL" to the hex literals that needed it.

On 32-bit platforms with older compilers (e.g. gcc 4.2 on MacOS 10.6
running on a 1st-gen Core Duo) a 'long' is 4 bytes and the compiler does
not automatically use a 'long long' when needed, but instead generates
an error. e.g.:

libtool: compile:  /usr/bin/g++-4.2 -DHAVE_CONFIG_H -I. -I../../.. -I../../../src/lib -I../../../src/lib-mail -I../../../src/lib-index -I../../../src/lib-storage -I../../../src/plugins/fts -I../../../src/doveadm -I/opt/local/include/openssl -I/opt/local/include -I/opt/local/include/CLucene/ext -pipe -Os -arch i386 -D__STDC_LIMIT_MACROS -MT lucene-wrapper.lo -MD -MP -MF .deps/lucene-wrapper.Tpo -c lucene-wrapper.cc  -fno-common -DPIC -o .libs/lucene-wrapper.o
In file included from ../../../src/lib/lib.h:33,
                 from lucene-wrapper.cc:4:
../../../src/lib/byteorder.h:94: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:95: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:96: error: integer constant is too large for ‘long’ type
../../../src/lib/byteorder.h:97: error: integer constant is too large for ‘long’ type
make[4]: *** [lucene-wrapper.lo] Error 1

Adding the 'ULL' to the end of the 16-digit hex literals that are used
to test the structure of 64-bit integers fixes this and avoids any
problem which could arise from the compiler using a 32-bit type for
those literals that could fit in 32 bites.

doveadm proxy: Don't crash if remote doesn't support log proxying

lib-storage: Add settings to configure lib-index optimization parameters

The defaults are expected to be pretty good, but these settings make it
easier to test whether other values might be more optimal.

lib-index: Add mail_index_cache_optimization_settings

lib-index: Add mail_index_base_optimization_settings

lib-index: Replace mail_index_set_log_rotation() with mail_index_set_optimization_settings()

This allows more easily adding optimization-related settings.

lib-dict: Remove test_dict_sql

It got accidentically added in 149299c7

lib-master: Allow a second log initialization after settings have been read.

This fixes logging with services that bypass the log service (lda or -L
parameter).

fs-compress: Support reading uncompressed input by prefixing compression format with "maybe-"

lib: Add istream-try

This can be used to automatically detect the underlying istream format from
a given list of choices.

lib: io_stream_set_error() - Allow one of the parameters to be the old error

global: Use check-local in Makefile.am instead of overriding check directly

This helps with dependency problems, like running "make check" in
lib-storage without "make" first would try to compile the test programs
too early and fail.