From: Jared Johnson
Update Qpsmtpd::Connection::notes() and Qpsmtpd::Transaction::notes()
with clearer, more brief syntax. A previous patch used seemingly better
syntax, but did not evaluate the size of @_ in order to distinguish
between a call to notes('arg1',undef) and notes('arg1'). This cor
This makes hook_reset_transaction a bit more useful by running it on
disconnections. It also ensures that hook_disconnect runs when the
client disconnects without QUITing.
---
lib/Qpsmtpd/SMTP.pm |1 +
lib/Qpsmtpd/TcpServer.pm |4
lib/Qpsmtpd/TcpServer/Prefork.pm
Our internal fork of QP talks to the postfix cleanup service on a remote
host, falling back to the local postfix cleanup socket. Continuig my
quest to reduce the diff between our fork and abh/master, this adds
support for a 'cleanup_sockets' config directive containing a list of
socket paths and r
The current postfix-queue plugin allows the administrator to set a
single path to a local postfix cleanup socket file from the plugin
'command line'. This adds a 'cleanup_sockets' configuration directive
that can contain a list of paths as well as host/port combinations
pointing to postfix cleanup
lib/Qpsmtpd/Postfix.pm now has a 4-space indent instead of a two-space
indent.
---
lib/Qpsmtpd/Postfix.pm | 279
1 files changed, 140 insertions(+), 139 deletions(-)
diff --git a/lib/Qpsmtpd/Postfix.pm b/lib/Qpsmtpd/Postfix.pm
index 5b2c4f0..3df36
Add a sample cleanup_sockets config file containing the standard postfix
cleanup socket path. Postfix socket redundancy is now tested to work!
---
config.sample/cleanup_sockets |4
1 files changed, 4 insertions(+), 0 deletions(-)
create mode 100644 config.sample/cleanup_sockets
diff --
This patch makes sure that hook_disconnect and hook_reset_transaction
plugins are run at the end of all transactions, including those where
the remote end hung up unexpectedly or we timed out the connection.
My previous changes were a bit more complex than they had to be, had no
effect on apache o