lib-auth: auth-client - Use the connection API.

lib-auth: auth-client - Free the request in a separate function.

lib-auth: auth-client - Rename auth_server_connection* to auth_client_connection*.

lib-auth: Move struct auth_client_request from auth-client-request.c to auth-client-private.h.

lib-auth: Move content of auth-client-request.h to auth-client-private.h.

lib-auth: Move remainder of auth-server-connection.h to auth-client-private.h.

lib-auth: Move struct auth_server_connection from auth-server-connection.h to auth-client-private.h.

lib-auth: Rename auth-server-connection.c to auth-client-connection.c.

doveadm: doveadm-auth-server - Remove useless include for auth-server-connection.h.

doveadm: doveadm-auth - Remove useless include for auth-server-connection.h.

lib: connection - Add vfunc called when the connection becomes ready after handshake.

master: Add unit test for auth-client.

lib-auth: auth-client - Add ability to configure the connect timeout.

This is currently only useful and required for unit tests (the default is just
too long).

lib-auth: auth-client: Do not connect implicitly in auth_client_init().

master: test-master-login-auth: Small coding style fix.

master: test-auth-master - Small coding style fix.

lib-master: master-login-auth - Emit named events at the start and end of auth requests.

lib-master: master-login-auth - Add support for event API and use it for logging.

lib-master: master-login-auth - Free the request in a separate function.

lib-master: master-login-auth - Provide more helpful error message for EACCESS connect error.

lib-master: master-login-auth - Detect connection to auth-client socket for better error message.

lib-master: master-login-auth - Use the connection API.

lib-master: master-login-auth - Handle response ID at a single location.

lib-master: master-login-auth - Handle response arguments at a single location.

master: Add unit test for master-login-auth.

master: Drop useless LDFLAGS for unit test.

lib-master: master-login-auth - Add ability to configure the lookup timeout.

This is currently only useful and required for unit tests (the default is just
too long).

lib-master: master-login-auth - Perform timeout manipulations in milliseconds.

lib-master: master-login-auth - Rename master_login_auth_set_timeout() to master_login_auth_update_timeout().

lib-auth: auth-master - Restore helpful error message for EACCESS connect error.

It got lost in earlier changes.

lib-auth: auth-master - Define event category.

lib-auth: auth-master - Fully reset connection state upon disconnect.

lib: test-connection - Add iteration count

Allows running some tests few times to make sure deinit/init works

lib: test-connection - Add no version sent test

lib: connection - Set connection list to NULL on deinit

Otherwise reusing the connection object causes panic on deinit

lib: connection - Reset handshake_received on connect

A connection object can be reused, and so handshake_received
need to be reset to FALSE on connection to avoid panic.

Fixes Panic: file connection.c: line 169 (connection_input_line_default): assertion failed: (conn->version_received)

lib-index: Add mail_index_append_finish_uids_full()

lib-index: mail_index_append_finish_uids() cleanup: remove unnecessary if check

It was pretty confusing, and it was always TRUE.

doveadm: Use doveadm_get_ssl_settings

doveadm: dsync - use mail_storage_settings_init_ssl_client_settings

lib-storage: mail-user - Use mail_storage_settings_init_ssl_client_settings

mail-storage: Add mail_storage_settings_init_ssl_client_settings

Simplifies following commits

doveadm: Use master service SSL settings

lib-smtp: client: Fix dead assignment in smtp_client_command_pipeline_is_open().

Found by scan-build.

lib-auth: auth-master - Add debug message for each user returned for auth_master_user_list_next().

lib-auth: auth-master - Emit named events at the start and end of lookups.

lib-auth: auth-master - Add support for event API and use it for logging.

lib-auth: auth-master - Pass struct auth_master_lookup_ctx to parse_reply() rather than several of its fields.

lib-auth: auth-master - Use the connection API.

lib: connection - Add connection_verify_version().

Only verifies the service name and major version number.

lib: connection - Rename connection_verify_version() to connection_handshake_args_default().

master: Add unit test for auth-master.

Cannot be added to lib-auth due to cyclic dependencies.

lib-auth: auth-master - Add ability to configure the lookup timeout.

This is currently only useful and required for unit tests (the default is just
too long).

lib-smtp: test-smtp-payload - Do not assume that a non-regular files is a directory.

The unit test failed if something else (like a socket) was found.

lib-http: test-http-payload - Do not assume that a non-regular files is a directory.

The unit test failed if something else (like a socket) was found.

lib-smtp: command: Move main loop from sub-function to smtp_client_command_send_more() itself.

This reduces code indent in smtp_client_command_do_send_more() (the
sub-function) and it turns the inifite for loop into a nice conditional
while loop.

lib-smtp: test-smtp-client-errors: Test early DATA command success and error replies.

Early success for the DATA command is treated as a protocol error, while early
failure is allowed. Both situations are tested in a new scenario for the
test-smtp-client-errors unit test.

lib-smtp: client: Fix handling of non-blocking command payload stream.

By inference, this fixes the handling of non-blocking transaction payload as
well. No io was created when the payload stream returned no data.

lib-smtp: test-smtp-client-errors: Add test scenario for premature success replies.

Such replies are sent immediately along with the reply of the previous command.
So, the reply is sent before the corresponding command is even submitted. This
tests whether the client handles this erroneous situation correctly.

lib-smtp: test-smtp-payload: Add option to test payload exchange with small socket buffers.

This is not part of the normal `make check` procedure, since using small socket
buffers makes the test extremely slow.

lib-smtp: client: Allow receiving replies before command data stream is sent completely.

This avoids unnecessary problems with servers that reply somewhat early to DATA
and BDAT commands. For one, early failure replies are now handled properly.
Also, race conditions at the client between sending the closing CRLF.CRLF and
receiving the DATA reply are no longer causing problems.

lib-smtp: client: Put moving a command from the queue to the wait list in a separate function.

lib-smtp: client: Put determining the status of the command pipeline in a separate function.

lib-smtp: client: Put sending the command line in a separate function.

lib-smtp: client: Don't put connection in TRANSACTION state unless there's a transaction.

Fixes hang occurring when connection is ready before transaction is started.

quota: Rename quota_set_resource() error_r to client_error_r

Don't pass on the error from dict_init() to client, though.

imap-acl: Rename error_r sent to client as client_error_r

lib-imap-urlauth: Rename error_r sent to clients as client_error_r

lib-imap-storage: Rename error_r sent to clients as client_error_r

imap,lib-storage: Rename error_r sent to clients as client_error_r

LAYOUT=index: Fix rebuilding mailbox list index on force-resync

The if-check was a bit confusing because have_backend==TRUE means that there
the mailbox list index is only an index, while have_backend==FALSE means
that the mailbox list index is the only source for the list of mailboxes
(= list index is the backend).

submission-login: proxy: Fix omission of spaces between XCLIENT command parameters.

It would send for example:

> XCLIENT ADDR=10.0.1.2PORT=39074

The other end will obviously not accept this.

--
Commit message modified by Stephan Bosch

lib-smtp: client: BDAT command: Make sure the message contains CRLF consistently.

Before, only messages sent using DATA command guaranteed consistent CRLF,
because these are sent using o_stream_dot(), which does that internally. For the
BDAT command, the message was sent as-is, which is not always accepted by MTAs.

lib-smtp: client: BDAT command: Ensure data stream max buffer size is appropriate when stream size is unknown.

When stream size is unknown, the chunk size are determined by what can be
buffered in the data input stream immediately. If it's maximum buffer size is
(much) smaller than the maximum chunk size, the maximum chunk size is never
achieved.

lib-smtp: server: Fix forwarding a multi-line reply.

A multi-line reply had the '-' on the first line stripped upon sending, which
makes clients see two separate responses rather than just one. This was caused
by the fact that forwarded replies had the last_line field not set properly,
in which case the '-' was substituted on the first line, rather than the last.
The fix makes a forwarded reply indistinguishable from a normally created reply
by also allowing for amending the reply with additional lines using
smtp_server_reply_add_text().

lib-smtp: test-smtp-reply: Test whether parsed valid replies can be written back to the original input.

In some cases, the parser mangles the input a little to substitute invalid
characters. In those cases, the expected output is stated explicitly.

lib-smtp: Rename test-smtp-reply-parser.c to test-smtp-reply.c.

Encoding tests will also be included, making the old name confusing.

global: hash_table_iterate_deinit(NULL) is a no-op

@@
expression E;
@@

- if (E != NULL) {
-  hash_table_iterate_deinit(&E);
- }
+ hash_table_iterate_deinit(&E);

lib: hash_table_iterate_deinit(NULL) should be a no-op

global: hash_table_destroy(NULL) is a no-op

@@
expression E;
@@

- if (hash_table_is_created(E))
-  hash_table_destroy(&E);
+ hash_table_destroy(&E);

imap: client: Add init() client vfunc.

This is called when client creation is finished completely, including namespace
initialization. The normal create hook is executed before namespace
initialization, which is a problem when a plugin needs to access the mail
storage immediately.

imap: client: Rearrange and document the client vfuncs.

submssion: DATA command: Seek data stream to the beginning before executing backends.

submission: DATA command: Provide a default name for the data stream.

lmtp: local: Make the local_deliver client vfunc responsible for submitting the reply.

This gives plugins more power to override delivery behavior.

lib-mail: ostream-dot - Add unit test for parent stream buffer being almost full.

lib-mail: ostream-dot - Add asserts to make sure max_bytes doesn't underflow

lib-mail: ostream-dot - Fix potential assert-crash when parent stream buffer gets full

If max_bytes=1, the (max_bytes-2) calculation brings it to (size_t)-1. This
causes too much data to be sent to the parent stream, which then returns a
partial write and causes an assert-crash.

The final chunk calculation doesn't need the -2 check, because additional
bytes aren't inserted at that point.

Fixes:
Panic: file ostream-dot.c: line 208 (o_stream_dot_sendv): assertion failed: ((size_t)ret == sent + added)

global: Rename array_first and array_last to array_front and back

This is more consistent with other languages

global: Use array_push_front

Re-run of 70540d8b3e0799113fa4c47e3b778efb4679f24f due to
bad spatch usage

global: Use array_pop_front

Re-run of 9b726934956ed9ca02077a82f7b2a1d74b7c9146 due to
bad spatch usage.

global: Use array_pop_back

Re-run of 48c70592eb36b1263ad63e3c1dca8a0b5aff09ba due to
wrong spatch usage.

global: Use array_push_back instead of array_append

lib: hash_table_destroy(NULL) should be a no-op

m4: Fix moduledir behaviour

This is mainly a fix for plugins that depend on dovecot.m4

push-notification-driver-lua: Request flags and keywords

push-notification-driver-lua: Fix comment typo

push-notification-driver-lua: Add keywords and flags to MessageAppend event

push-notification-driver-lua: Add keywords and flags to MessageNew event

push-notification-driver-lua: Rename keywords_set and keywors_clear to keywords

This is more concince since these are the keywords being operated on.