implement the report response for proc-filters too

Reported by renegm on GitHub:
<https://github.com/OpenSMTPD/OpenSMTPD/issues/1257>

ok gilles@

oops, fix previous. wrong patch committed. was lacking the final \n

include the filter protocol version in the handshake

This adds "config|protocol|0.7" during the filters handshake, like
was done for the tables too, so that the filters can know the version
of the p

include the filter protocol version in the handshake

This adds "config|protocol|0.7" during the filters handshake, like
was done for the tables too, so that the filters can know the version
of the protocol right from the start.

ok gilles@

show more ...

correct indentation; no functional change
ok tb@

cast tv_usec to long for *printf

mostly for -portable; on darwin suseconds_t is an int.

ok tb@

swap link-auth filter arguments

Fields which can contain a '|' character are kept last to avoid
ambiguities so move result before username; link-auth was likely forgot
in r1.61 of lka_filter.c when

swap link-auth filter arguments

Fields which can contain a '|' character are kept last to avoid
ambiguities so move result before username; link-auth was likely forgot
in r1.61 of lka_filter.c when the same treatment was applied to other
events.

Discovered after a report on -portable due to filter-rspamd crashing.

ok millert@

show more ...

clear errno before strtoull as it may already be set to ERANGE

noticed and ok millert@

avoid truncation of filtered data lines

Don't copy in a buffer the filter' output for parsing as we may truncate
filter-dataline (i.e. the mail body). Instead, parse the string by
advancing the poi

avoid truncation of filtered data lines

Don't copy in a buffer the filter' output for parsing as we may truncate
filter-dataline (i.e. the mail body). Instead, parse the string by
advancing the pointer without copying or modifications.

Issue reported by Joachim Schneider on the OpenSMTPD-portable
repository.

ok millert@

show more ...

cast to '(long long)' instead of '(long long int)'

while here adjust the spacing in some of the touched lines.
requested by deraadt@, ok tb@

cast time_t to long long for printf, reduces the diff with -portable

ok millert@

add required headers for smtpd.h and remove unnecessary ones in other files.

ok jung@

Remove unused variables found by clang. Additional unused var spotted by eric@.

OK mvs@, eric@

Rename the pony process to dispatcher and klondike to crypto.

From gilles@
OK millert@ giovanni@

smtpd's filter state machine can prematurely release resources
leading to a crash. From gilles@

Free filter_name in lka_filter_end

From gilles@
OK millert@ eric@

Add the admd keyword. This can be used by filters interested in the
Authentication-Results header.

OK giovanni@

strip trailing CRs at smtp level rather than io level

ok millert@

switch email and result fields in mail-from/rcpt-to event reports
and bump protocol version.

discussed with jung@, martijn@ and Gilles.

ok jung@

allow using the session username in builtin filters when available

enable builtin filtering for commit phase

emable builtin filtering for phase DATA, no idea why we didn't earlier as
the grammar allowed it and the code was already there.

if a filter was attached to a relay action in config, notify instance that
it can register smtp-out events

start bringing smtp-out reporting code, lacks some events still

do not pass rdns, fcrdns, ss_src and ss_dest with IMSG_FILTER_SMTP_BEGIN,
but gather the information from the link-connect reporting event instead.
this removes redundant code and makes it easier to

do not pass rdns, fcrdns, ss_src and ss_dest with IMSG_FILTER_SMTP_BEGIN,
but gather the information from the link-connect reporting event instead.
this removes redundant code and makes it easier to prepare for smtp-out.

show more ...

introduce a bypass keyword so that builtin filters can bypass processing of
a phase when a condition is met

suggested by several people including jung@, ok jung@