Perry E. Metzger wrote:
Stephan Bosch <stephan@rename-it.nl> writes:
It is related to that change, yes. I fixed the problem, but I am still not entirely happy with the error handling of the store action.
It would be best, I think, if the system failed safe (i.e. if it did not lose the user's mail in case of error, but rather stored it in the main mailbox or in some sort of last ditch error mailbox.)
That's exactly what it does now and if the Sieve engine is to blame for losing e-mail, it is always a bug.
Anyway, that kind of error handling is not what I meant. It pertains to the problem that Timo described and the passing of error codes and error strings between the Dovecot API and the Sieve engine in general. That is all a bit ugly right now.
It should also report a reasonably sane error in syslog...
This type of error is mainly useful for developers, much like a Segfault. Occurences of this kind are always a bug and are fixed as soon as possible. Considering the fact that I've added a test for this case in the test suite, it should not surface again in the future.
Regards,
Stephan.