https://bugs.exim.org/show_bug.cgi?id=2314

--- Comment #12 from Matthias Kurz <m.k...@irregular.at> ---
> Could you supply a comment (for the first patch) with the rationale for
> raising the event in the new place and not the old?

If I remember correctly not all (failed) messages reach the failure_log(...)
function (Which itself is called by post_process_one(...)). Some failed
messages "stop" sooner and don't even execute post_process_one(...).

That's why I had to raise the event (for ignored errors) earlier. And to avoid
duplicate events I had to add the new "if (!addr->prop.ignore_error)" condition
in the original place within the failure_log(...) function (because obviously
the event was fired already).

Does that answer your question?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

Reply via email to