lib-ssl-iostream: Remove input_handler flag

It is no longer needed after 87da941c

lib-ssl-iostream: Add get_compression accessor

This is needed by login-common

lib-ssl-iostream: Close ostream before istream

If i_stream is autoclose, then o_stream
close will fail if done after i_stream
close.

lib-ssl-iostream: Allow skipping CRL check

lib-ssl-iostream: Add alternate certificate support

global: Splice cert into separate struct from iostream_ssl_settings

lib-fs: test-fs-posix - Fix checking whether unlink_directory() failed

man: Update manpage for mailbox delete command

lib: Don't use NEEDS_LOCAL_CREDS undefined

global: Rename i_stream_is_eof() to i_stream_read_eof()

This describes its behavior a bit better.

global: Avoid i_stream_is_eof() where it's not necessary

lib: Improve i_stream_is_eof() comment

lib: Update struct istream comments

lib-http: If istream read fails, preserve the istream's full error message

lib-http: test-http-client-errors - fix typo in error message

lib-http: Detect istream errors correctly.

i_stream_is_eof() returns TRUE also when streams have an error, so all
errors were logged the same as a regular disconnection.

lib,lib-index: Use #ifdef with WORDS_BIGENDIAN

Some were missing from the last change.

global: Remove explicit o_stream_flush() calls from ostream.close() methods

It's now being done automatically.

lib: Automatically flush ostream when it's closed

Also verify that it unexpectedly didn't get fully flushed.

mail-filter: ostream-ext-filter - Require o_stream_finish() to be called

Similar to ostream-zlib.

lib-mail: ostream-dot - Require o_stream_finish() to write the trailer

Similar to ostream-zlib.

lib: ostream-multiplex - Don't close parent streams at destroy

The close() method is already called as needed.

lib-dcrypt: ostream-dcrypt - Require o_stream_finish() to write the trailer

Similar to ostream-zlib.

lib-compression: ostream-zlib - Require o_stream_finish() to write the trailer

It's now possible to call o_stream_flush() without the trailer getting
written.

It's also now required to call o_stream_finish() to write the trailer before
closing the ostream. This guarantees that the caller checks that the
trailer was successfully written. This isn't needed if the ostream already
failed before writing the trailer.

lib: Remove o_stream_nfinish()

global: Replace o_stream_nfinish() with o_stream_flush()

global: Replace o_stream_nfinish() with o_stream_finish() whenever possible

lib: Add o_stream_finish()

This marks the ostream as fully finished, with no further writes coming to
it. Otherwise it behaves like o_stream_flush().

The ostream.flush() methods are supposed to write their trailer (if the
ostream has one) when they see that ostream.finished is set. These will be
implemented by the following commits.

lib: o_stream_flush() - check for send overflows and reset error handling

This begins the removal of o_stream_nfinish().

lib: Remove o_stream_nflush()

None of its previous callers were calling it correctly, so it's better to
remove it entirely to avoid more of such wrong calls.

imap: Make sure long COPY commands send "* OK Hang in there" packets

o_stream_nflush() already flushed the writes to kernel, and Linux should
have sent them after 200ms delay, so this probably doesn't fix anything.
Using uncork+cork is a bit clearer way to indicate that the data really
is wanted to be sent now.

lib-imap: When sending "+ OK" to client, uncork ostream immediately

This makes sure that it's sent to the client as soon as possible.

Also removed o_stream_nflush(), since uncorking does that already.
If there is no corking, the "+ OK" was already attempted to be sent
by the o_stream_nsend() call and a flush is unlikely to help.

global: Remove unnecessary o_stream_nflush() calls

The following o_stream_nfinish() or o_stream_uncork() calls it anyway.

lib: o_stream_uncork() - call o_stream_nflush() instead of o_stream_flush()

Otherwise flush errors may be unintentionally ignored. This change requires
that all the o_stream_uncork() callers must call o_stream_nfinish()
afterwards, unless the ostream is set to ignore errors.

lib: test-ostream-* - Call o_stream_nfinish() after uncorking

configure: Add support for EXTRA_LDFLAGS

imapc: Add imapc_features=fetch-empty-is-expunged

When FETCH returns an empty mail, the mail is assumed to be expunged.

global: Use #ifdef instead of #if with WORDS_BIGENDIAN

This is to not rely on undefined WORDS_BIGENDIAN being evaluated as 0.

lib/compat.h: Undefine WORDS_BIGENDIAN when it's defined as 0

m4: Use AC_C_BIGENDIAN instead of custom m4

lib-storage: Replace DBOX_GUID_BIN_LEN with GUID_128_SIZE

DBOX_GUID_BIN_LEN define was removed in 0c909e, replace the use with
GUID_128_SIZE.

director: Don't use undefined DIRECTOR_CONNECTION_PING_TIMEOUT_MSECS

auth: Don't use undefined SASL_VERSION_MAJOR

global: Rename client_connection_type to doveadm_connection_type

doveadm: Rename client_connection_type to doveadm_client_connection_type

Move it to doveadm.h to allow plugins use doveadm-cmd.h

doveadm: Reorder includes

This makes next commit possible

quota: Improve error messages

quota-fs: Default to quota API v2 if _LINUX_QUOTA_VERSION is undefined.

glibc v2.25 drops v1 support in sys/quota.h and doesn't define
_LINUX_QUOTA_VERSION.

doveadm-server: http: Properly implemented error handling for requests.

doveadm-server: http: Mark the HTTP request for connection closure right away.

This means that the connection is closed once the request finishes.
Before, each response did that individually.
Once connection reuse is implemented/needed this can now be turned off at one place.

doveadm-server: http: Restructured JSON parsing to improve code clarity.

Made state machine more explicit and easier to understand.
Also added more comments.

doveadm-server: http: Wrapped long code lines.

doveadm-server: http: Joined JSON parsing and handling in one function.

doveadm-server: http: Split JSON handling into several sub-functions.

doveadm-server: http: Split JSON parsing into several sub-functions.

doveadm-server: http: Changed request parse error into an assertion.

The condition can never occur when json-parser and the request parsing state machine are behaving correctly.
At that point only an object key can be returned; the client cannot cause it to return anything else.

doveadm-server: http: Moved doveadm_http_server_command_execute().

doveadm-server: http: Renamed parse state enum.

doveadm-server: http: Split off request handling in separate struct, so that the connection can be reused.

doveadm-server: http: Added structural comments.

doveadm-server: http: Moved doveadm_http_server_request_destroy().

doveadm-server: http: Moved doveadm_http_server_send_response().

doveadm-server: http: Moved connection-related code to the bottom of the file.

doveadm-server: http: Make http_server_set a local variable of doveadm_http_server_init().

doveadm-server: http: Create the http_response once needed, not preemptively.

doveadm-server: http: Split authentication into multiple functions.

doveadm-server: http: Renamed conn->http_server_request to conn->http_request.

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.