On Fri, 2007-07-06 at 20:25 +0300, Uldis Pakuls wrote:
> and most important things:
> 1. notify now always reports SIEVE_OK as notify failure is not delivery
> failure. all notify failures are reported in log. Also "SIEVE_ERROR"
> aborts script execution, but there is no reason to abort on notify error.

These should maybe be changed elsewhere too..

> 2. sieve plugin returns "-1" if script failed load or on any error. i
> think it is wrong; if no scripts are executed or it is aborted for some
> reason - it is not fatal delivery error;  this just means "nothing is
> done" by script. so i changed plugin always return "0" on errors.

How about: http://hg.dovecot.org/dovecot-1.0/rev/87d0ae7fbe12

I committed the notify patch too with small changes.

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to