mail-lua: Fix argument number checking for mailbox#sync

push-notification-driver-lua: Perform garbage collection after transaction

Ensure Lua releases resources here.

push-notification-driver-ox: Convert to event based logging

Also fixes the logging to be standard

push-notification-driver-lua: Add events

push-notification: Emit event on transaction end

push-notification: Add event to push notification transaction

push-notification: Add event category

lib-lua: Move __gc metamethod for event to userdata

Compability for lua 5.1

mail-lua: Check number of arguments

lib-lua: Add DLUA_REQUIRE_ARGS(s,x) and DLUA_REQUIRE_ARGS_IN(s,x,y)

First form requires that x arguments are provided for dlua script s.

Second form requires that at least x and at most y arguments are provided
for dlua script s

Automatically returns luaL_error.

mail-lua: Make flags optional for mailbox#sync

mail-lua: Make flags optional for user#mailbox

mail-lua: Change mailbox#status API to varargs

It's much easier to use this way

lib-lua: Check number of arguments

lib-lua: Add flag manipulation functions

lib-lua: Add test for event framework

lib-lua: Add event interface

lib-lua: Add events to scripts

lib-master: Fix test-event-stats

New stats uses gettimeofday that generated unpredictable values
for this test.

lib: event - Stop exporting ioloop time

It is not needed by stats process.

stats: Use duration field from events

lib: event - Record real duration

The old code only measured difference between successive
ioloop_time updates. The new code uses gettimeofday to
get current real time, making measurements accurate.

lib: event - Duplicate source_filename

Prevents crash on event leak when plugin that caused the leak
is unloaded.

lib: event - Rename tv_created as tv_created_ioloop

push-notification: Resolve confusion about msg->seq

It's not the same seq what the mail has, but a sequence that
is used to find out UID for saved mails. So rename it to
save_idx and treat it accordingly. Note that save_idx is 0
based, and mail sequence is 1 based.

push-notification-driver-lua: Include date from MessageAppend

push-notification-event-messageappend: Include date information parsed from headers

Consistency with MessageNew

push-notification: Make sure txn messages are unique

save_count is not reliable as it's not incremented for events like
flag changes.

push-notification-driver-lua: Always use DLUA_LOG_LABEL

push-notification-driver-lua: Properly register all events

mail-lua: Fix gc code to push key first then value

Otherwise it uses nil as key, which makes no sense and leads to eventual crash.

push-notification-driver-lua: tx begin pushed mail user not username

push-notification-driver-lua: Do not push user again

User is already pushed in tx begin, and can be stored there into the context.

push-notification-driver-lua: Move event push earlier

Simplifies code

mail-lua: Add deinit_pre function

push-notification-driver-lua: Reference mail user only if initialization is successful

Will cause assert-crash otherwise if initialization fails

push-notification-driver-lua: Fix symbol names

mail-lua: Deinit script after others

Some other plugin might need Lua for deinit

push-notification-driver-lua: Use Lua references for context

mail-lua: Implement mail_lua_plugin_get_script

It was forgotten in 950c7e06dbf7f1cd696653af95b8c0053a9e86da

auth: db-lua - Remove unnecessary pushnil usage

lua_pushstring is NULL-safe

push-notification: Add lua driver

mail-lua: Add Lua support plugin for mail storage

lib-lua: Add lua_pushboolean helper to push real booleans

Makes boolean handling in dovecot code slightly easier

push-notification: Fix mailbox unsubscribe event

Subscribe should be FALSE here.

push-notification: Fix initializing push notification mailbox triggers

None of the mailbox triggers were working, since the transaction was
not being correctly initialized.

dns: Remove duplicate event_unref() in dns_client_input_args()

The event is unreffed at the end of the function also.

dns: Return the lookup error string to the querier

lib-dns: Set error_r in dns_client_connect() on failure

lib-dns: Remove DNS from the log messages now that it's in the prefix

lib-dns: Use dns(host): prefix instead of just host: in event

lib: Use basename of unix socket path as event log prefix

This is to make the prefix consistent with other unix socket loggings in
Dovecot.

lib: connection - Clarify logging of incoming connections

Don't log "Client connected" for both incoming and outgoing connections.

lib: connection - Remove dead code

connection_client_connected() can only be called for clients. There was
already an assert for it.

global: Don't try to send stats from unit tests

driver-sqlite: Fix crash and event leak in driver_sqlite_query_s()

driver-sqlite: Assign db in driver_sqlite_exec()

This fixes a crash.

lib-sql: Fix wrong sql_query_finished_event() success parameter use

driver-sqlite: Don't give false out of memory errors

Only log error from sqlite3_errmsg() when connected to the db, since
otherwise dovecot logs false "out of memory" errors.

driver-sqlite: Add error to sql_query_finished event on error

driver-pgsql: Add error to sql_query_finished if it fails

driver-cassandra: Add error to sql_query_finished event on error

lib-sql: Do not create event for old drivers

They can't use it anyways

driver-sqlpool: Only strdup error on error

Satisfies static analyzers

driver-sqlite: Document why rollback is called when commit fails

driver-cassandra: Add events

driver-pgsql: Add events

driver-sqlite: Add events

driver-mysql: Add events

driver-sqlpool: Add events

lib-sql: Add events

auth: Use sql_init_full

global: Replace sql_db_cache_new with sql_db_cache_new2

dict-sql: Use sql_db_cache_new2

lib-sql: Add sql_db_cache_new2 that uses the new sql_init_full initializer

driver-cassandra: Add init_full

driver-cassandra: Return error from connection string parser

driver-cassandra: Add driver_cassandra_free

driver-pgsql: Add init_full

driver-pgsql: Add driver_pgsql_free

driver-sqlite: Add init_full

driver-mysql: Add init_full

driver-mysql: Allocate mysql connection ourselves

This allows reusing the same mysql object on reconnect,
and simplifies cleanup in case initialization fails.

driver-mysql: Return error in connection string parser

driver-sqlpool: Add driver_sqlpool_init_full

Needed by SQL API change

driver-sqlpool: Use init_full when available

lib-sql: Add init_full vfunction

driver-sqlite: Whitespace fix

auth: Add db_ prefix to sql in db-sql

Prevents clash with lib-sql

util: Fix compile with tcpwrap

stats: Wait for clients to disconnect at shutdown

global: Replace MASTER_SERVICE_FLAG_SEND_STATS with _DONT_SEND_STATS

Most services want to send statistics, so it's better to reverse the flag
behavior. Mark the few services that don't want stats with
MASTER_SERVICE_FLAG_DONT_SEND_STATS.

master: Start service processes only after adding all service listeners

Otherwise the processes could try to connect to services that aren't being
listened on, causing errors. Especially stats-writer socket needs to be
available early on.

lib: Test event category parent matching

lib: Fix matching parent categories in event filter

So far this hasn't caused problems, because parent events had been using
parent categories. However, if the parent category wasn't used by an event
or its parents, an event filter wouldn't have matched it.

lib-master, master: Initialize stats-client via environment variables

That way it's initialized before settings are read, and it can catch events
earlier. It also doesn't require all services to read settings if they
otherwise don't need them.

dns: Fix duration field in dns_worker_request_finished event

lib-dns - struct member cleanup in dns_lookup

The index was never used anyway. Now we're using an alloconly pool,
there's no need for the dns_lookup to keep a freeable pointer to the
fields also int result.

Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib-http: test-http-client-errors - Skip VERSION line in DNS reply

Otherwise we send extraneous results causing SIGPIPE crash.

Broken in ddb9ead03f4cc7a2c61094e50803e07dfd590cb5

lib-dns: Always use dns_lookup_free

Ensures that all resources get free'd in single place,
Fixes memory leak and event double-unref.

Broken in 9ee82e3f660e6e072532283cad8668a578ac137d