https://bugs.exim.org/show_bug.cgi?id=2310
Jeremy Harris changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #19 from Jeremy Harris ---
Just not needed. We could create another value if we really cared, but
unless we need different processing anywhere in the ACL handling, or that
value gets exposed as a variable... we don't.
Thank for spotting
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #18 from Matthias Kurz ---
Thanks Jeremy!
I saw you didn't set acl_where? Was that intentional? Or just not needed?
Also there is a typo in the changelog:
> ...message is manually reoved...
Should probably be "...removed...".
--
You are
https://bugs.exim.org/show_bug.cgi?id=2310
Git Commit changed:
What|Removed |Added
CC||g...@exim.org
--- Comment #17 from Git Commit ---
https://bugs.exim.org/show_bug.cgi?id=2310
Jeremy Harris changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|ni...@exim.org
https://bugs.exim.org/show_bug.cgi?id=2310
Matthias Kurz changed:
What|Removed |Added
Attachment #1109|0 |1
is obsolete|
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #15 from Matthias Kurz ---
NOW it's really done ;)
The last patch file I added now also fires "msg:fail:internal" for
non-delivered recipients when removing a message via -Mrm.
Comment 9 still applies.
Let me know what you think, I am
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #14 from Matthias Kurz ---
Created attachment 1119
--> https://bugs.exim.org/attachment.cgi?id=1119=edit
Fire msg:fail:internal for each recipient for -Mrm
--
You are receiving this mail because:
You are on the CC list for the bug.
--
https://bugs.exim.org/show_bug.cgi?id=2310
Matthias Kurz changed:
What|Removed |Added
Attachment #1108|0 |1
is obsolete|
https://bugs.exim.org/show_bug.cgi?id=2310
Matthias Kurz changed:
What|Removed |Added
Attachment #1106|0 |1
is obsolete|
https://bugs.exim.org/show_bug.cgi?id=2310
Matthias Kurz changed:
What|Removed |Added
Attachment #1107|0 |1
is obsolete|
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #9 from Matthias Kurz ---
Alright, it's done - here is a working version that fires "msg:complete" when a
message gets removed via "-Mrm".
The problem I was facing a couple of days ago was that the ACL definitions were
not available in
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #7 from Matthias Kurz ---
Created attachment 1108
--> https://bugs.exim.org/attachment.cgi?id=1108=edit
Test: Fire event(s) when removing message from queue
--
You are receiving this mail because:
You are on the CC list for the bug.
--
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #6 from Matthias Kurz ---
Created attachment 1107
--> https://bugs.exim.org/attachment.cgi?id=1107=edit
Call event_raise when removing message from queue (-Mrm)
--
You are receiving this mail because:
You are on the CC list for the bug.
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #8 from Matthias Kurz ---
Created attachment 1109
--> https://bugs.exim.org/attachment.cgi?id=1109=edit
Set acl_where
--
You are receiving this mail because:
You are on the CC list for the bug.
--
## List details at
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #5 from Matthias Kurz ---
Created attachment 1106
--> https://bugs.exim.org/attachment.cgi?id=1106=edit
event_raise(...) can be used outside of transport
--
You are receiving this mail because:
You are on the CC list for the bug.
--
##
https://bugs.exim.org/show_bug.cgi?id=2310
Matthias Kurz changed:
What|Removed |Added
Attachment #1102|0 |1
is obsolete|
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #4 from Matthias Kurz ---
Created attachment 1102
--> https://bugs.exim.org/attachment.cgi?id=1102=edit
Unfinished patch with idea how to fix Bug 2310
Again, this is just an idea how to fix this - which should give an experienced
exim dev
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #3 from Jeremy Harris ---
We don't do github PRs as a matter of course; please attach your patch to
this bug.
--
You are receiving this mail because:
You are on the CC list for the bug.
--
## List details at
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #2 from Matthias Kurz ---
One last comment to my GitHub pull request/patch:
I am not entirely sure, but I think in `queue.c` something like
---
extern int acl_where;
---
with a value defined in macros.h needs to be set:
https://bugs.exim.org/show_bug.cgi?id=2310
--- Comment #1 from Matthias Kurz ---
Here is what I came up with to fix that bug:
https://github.com/Exim/exim/pull/65
It's not finished (of course) - however I think what I added in queue.c is a
good starting point for the work that needs to be done
21 matches
Mail list logo
