doveadm: server-connection - Remove redundant ssl verification

Certificate validation is done by lib-ssl-iostream already,
don't do it here again.

Fixes ssl_client_require_valid_cert=no for doveadm sync

doveadm: Make sure SSL settings are always available

lib-ssl-iostream: Add comment about using ssl_iostream_check_cert_validity

lib-ssl-iostream: Add comment about verifying certs in callbacks

lib-ssl-iostream: Call ssl_iostream_check_cert_validity as default

Unless callback is specified, call ssl_iostream_check_cert_validity
instead of ssl_iostream_cert_match_name to make sure we perform
same checks consistently.

auth: db-lua - Add method for creating request event

This is needed to make sure that correct event is used when
prefix logging is wanted.

auth: mech-gssapi - Fix compile problem.

auth-policy: Move result logging to auth policy code

Removes duplicate and also unnecessary logging of result

auth: Use request event for logging in generic logging

auth: Use event logging for subsys mech

Automated change

sed -i -e 's/auth_request_log_\(.*\)(\(.*\), AUTH_SUBSYS_MECH,/e_\1(\2->mech_event,/' *.c

Whitespace fixing done with custom script

auth: Use event logging for subsys db

Done with automated change

sed -i -e 's/auth_request_log_\(.*\)(\(.*\), AUTH_SUBSYS_DB,/e_\1(\2->authdb_event,/' *.c

Followed by coccinelle

@@
expression E;
@@

-E->authdb_event
+authdb_event(E)

and some manual cleanup

Whitespace fixing done with custom script

auth: Add events for passdb/userdb lookups

auth: userdb - add userdb_result_to_string

auth: passdb - Add passdb_result_to_string

auth: auth-worker - Add event

auth-policy: Use events

auth-policy: Use dedicated pool

Otherwise memory gets released too early.

auth-policy: Move callback to helper function

auth: Emit event when auth request is finished

auth: Add mechanism event

auth: Split creation of log identifier from get_log_prefix()

auth: Update dummy_set

Add auth_verbose, was forgotten in
b484ab4b55b0d5341f2f4dd98a655a75f0bf1275

lib-http: http-client-connection - Create function for obtaining client settings reliably.

This fixes and prevents segfaults occurring when settings are needed while the
connection is detached. In that case, the shared client context settings need
to be used rather than the client settings. Before, the attachment of a peer was
assumed for obtaining the settings when it actually could be absent in rare
cases, causing a NULL dereference.

lib-http: http-client-connection - Make sure the request timeout is stopped when no more request are pending.

lib-http: http-client-connection - Descend event from client context rather than client.

The connection is potentially switched between clients often causing the
client event prefix to make no sense.

lib-smtp: smtp-client-transaction - Add event field for the amount of data sent.

lib-smtp: smtp-client - Perform the LF -> CRLF conversion for the message data in the transaction.

Before, it was performed in the DATA/BDAT command code, which makes it very
difficult to extract post-conversion size statistics.

lib-smtp: smtp-client-command.h - Reformat the comments for the standard commands.

lib-smtp: smtp-client-transaction - Emit named recipient events.

lib-smtp: smtp-client-transaction - Emit named events.

lib-smtp: smtp-client-transaction - Pass last reply to smtp_client_transaction_finish().

lib-smtp: smtp-client-transaction - Record the first failure reply of the DATA command.

This is needed when the DATA command yields more than a single reply (LMTP).

lib-smtp: smtp-client-transaction - Record recipient statistics in the transaction.

lib-smtp: smtp-client-transaction - Drop smtp_client_transaction_rcpt_drop_pending() as a separate function.

It is now merged into smtp_client_transaction_rcpt_approved() to make the code
less confusing.

lib-smtp: smtp-client-transaction - Drop the rcpt->failed flag.

It is no longer needed.

lib-smtp: smtp-client-transaction - Explicitly keep track of whether a recipient is already finished.

lib-smtp: smtp-client-transaction - Free the rcpt object in smtp_client_transaction_rcpt_fail_reply().

lib-smtp: smtp-client-transaction - Handle recipient failure callback in a separate _rcpt function.

lib-smtp: smtp-client-transaction - Handle recipient DATA callback in a separate _rcpt function.

lib-smtp: smtp-client-transaction - Handle RCPT reply in a separate _rcpt function.

lib-smtp: smtp-client-transaction - Free the mail object in smtp_client_transaction_mail_fail_reply().

lib-smtp: smtp-client-transaction - Handle MAIL failure callback in a separate _mail function.

lib-smtp: smtp-client-transaction - Handle MAIL reply in a separate _mail function.

lib-smtp: smtp-params - Add support for adding RCPT parameters to an event.

lib-smtp: smtp-params - Add support for adding MAIL parameters to an event.

lib-smtp: smtp-client-command - Emit named events.

lib-smtp: smtp-client-command - Create local variable for "was_sent" command status.

The condition is amended with (state < SMTP_CLIENT_COMMAND_STATE_FINISHED),
because that part is now checked after the intiialization of this variable
(causing the function to exit). It is important that the content of this
variable makes sense immediately (for future changes), rather than only after
the if statement that checks the added condition.

lib-smtp: smtp-client-command - Rename "waslocked" to "was_locked" in smtp_client_command_abort().

lib-smtp: smtp-reply - Add smtp_reply_add_to_event().

This sets standard reply fields in the provided pass-through event.

lib-smtp: smtp-client-connection - Add peer fields to the connection events.

lib-smtp: smtp-client - Replace smtp_*_debug() with e_debug().

lib-smtp: smtp-client - Replace smtp_*_warning() with e_warning().

lib-smtp: smtp-client - Replace smtp_*_error() with e_error().

lib-smtp: smtp-client - Replace i_error() with e_error().

lib-smtp: smtp-client - Remove redundant debug logging checks.

lib-smtp: smtp-client - Add support for event API.

lib-smtp: smtp-client-command - Split command name extraction from label construction.

lib-smtp: smtp-reply - Add smtp_reply_get_message().

Returns only the message of the reply without statuses as a single line.

doveadm-server: Load SSL settings

Otherwise operations that might use ssl can lead
into invalid memory access.

lib: test-istream-seekable - Add asserts to make sure blocking state changes at EOF

lib-storage: mail_get_*stream*() - Assert that returned streams are blocking

Several callers already rely on them being blocking. Making these asserts
explicit makes sure that any bugs are caught early.

lib: istream-seekable - Change stream to be blocking=TRUE after reaching EOF

After EOF is reached, the stream is now fully read into file or memory.
read()s can no longer return 0, so blocking=TRUE can be used.

Some callers were reusing the seekable stream in places that required
blocking=TRUE.

Fixes at least with imapsieve vnd.dovecot report extension:
Panic: file ostream.c: line 427 (o_stream_nsend_istream): assertion failed: (instream->blocking)

lib-ssl-iostream: iostream-ssl-test - Update the test certificate to have very long expiry period.

The old certificate expired after one year, which causes all kinds of unit test
failures.

auth: Add caching to passdb-oauth.

auth: Add support for oauth2 password grant

lib-oauth2: Implement password grant authentication.

lib-oauth2: Take http_client_request_url_str in use for oauth2 requests.

lib-http: Expose origin_url of the struct http_client_request

master: test-master-login-auth - Add test for parallel requests.

lib-master: master-login-auth - Fix hang when auth server returns multiple replies at once.

input_args() should return 1 to continue reading input, while 0 stops it.

master: test-auth-client - Add test for parallel requests.

master: test-auth-client - Add progress timeout.

lib-master: master-login-auth - Name auth finished event same way as the others

So change auth_master_client_login_finished to
auth_master_client_login_finished.

submission-login: Only overwrite XCLIENT fields in the client when a value is assigned.

Before, omitted fields in the XCLIENT command would cause the values in the
client to be reset to the equivalent of zero.

lib-auth: Fix hang when auth server returns multiple replies at once

input_line() should return 1 to continue reading input, while 0 stops it.

Broken by a91862deade9dd12275f939f2716b5f56633f89e

lib-smtp: Mark default as unreached in smtp_client_command_abort

Satisfies static analyzers

lib-smtp: smtp-client-command - Avoid calling the callback for the DATA command during submission.

This causes a race condition in which the command object is already freed when
it is returned from smtp_client_command_data_submit(). This scenario occurs when
reading data for the first BDAT command fails.

lib-smtp: smtp-client - Send correct size value for BDAT command when the data lacks CR.

The addition of CR was ignored in the size specification because the original
unaltered stream was used for the size calculations.

master: test-auth-client - Drop useless local variable.

This addresses a scan-build report.

lib-smtp: smtp-client - Fix sending of duplicate QUIT command.

The command name comparison function was broken after the command was sent. Made
the function robust against addition/removal of CRLF to the command line.

lib: istreams - Don't lose IO pending state when unsetting and setting io

This makes sure that the pending state stays even after io_remove() when
io_add_istream() is again called.

lib: Fix i_stream_set_input_pending() when IO is set later

After i_stream_set_io() is called to set the IO, it should already be marked
as pending.

This fixes at least running imaptest with SSL.

lib: Add io_is_pending()

auth: Do not import empty certificate username

auth: Fail authentication if certificate username was unexpectedly missing

login-common: Ensure we get username from certificate

lib-auth: auth-client - Emit named events.

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

lib-auth: auth-client - Add optional reason argument to auth_client_request_abort().

It is not actually used yet in this commit.

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.