6.0.0-alpha12
6/12/25

[#13060] Wrong "Reject" recipe
Summary Wrong "Reject" recipe
Queue Ingo
Queue Version 3.1.4
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester admin (at) dei (dot) unipd (dot) it
Created 03/19/2014 (4103 days ago)
Due
Updated 03/20/2014 (4102 days ago)
Assigned 03/20/2014 (4102 days ago)
Resolved 03/20/2014 (4102 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
03/20/2014 11:03:21 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit ba854bc8ab887a9d46412a699eedb36175c700f7
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu Mar 20 17:01:28 2014 -0600

     [mms] Fix procmail reject recipe (Bug #13060; admin@dei.unipd.it).

     Conflicts:
             ingo/docs/CHANGES
             ingo/package.xml

  ingo/docs/CHANGES                   |    7 +++++++
  ingo/lib/Script/Procmail/Recipe.php |    1 -
  ingo/package.xml                    |   13 +++++++++++++
  3 files changed, 20 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/ba854bc8ab887a9d46412a699eedb36175c700f7
03/20/2014 11:02:15 PM Michael Slusarz Comment #4
State ⇒ Resolved
Reply to this comment
Ingo 3.1.5
03/20/2014 11:01:55 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_1):

commit bbb23c9a26cd5cd78d228f12417610b3bd0079da
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Thu Mar 20 17:01:28 2014 -0600

     [mms] Fix procmail reject recipe (Bug #13060; admin@dei.unipd.it).

  ingo/docs/CHANGES                   |    1 +
  ingo/lib/Script/Procmail/Recipe.php |    1 -
  ingo/package.xml                    |    4 ++--
  3 files changed, 3 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/bbb23c9a26cd5cd78d228f12417610b3bd0079da
03/20/2014 11:00:49 PM Michael Slusarz Comment #2
State ⇒ Assigned
Version ⇒ 3.1.4
Priority ⇒ 1. Low
Assigned to Michael Slusarz
Reply to this comment
Verified (and solution matches examples from procmailex man page).
03/19/2014 08:42:09 AM admin (at) dei (dot) unipd (dot) it Comment #1
Priority ⇒ 3. High
State ⇒ Unconfirmed
New Attachment: ingo_recipe.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Ingo
Summary ⇒ Wrong "Reject" recipe
Type ⇒ Bug
Reply to this comment
fixing bug http://bugs.horde.org/ticket/7520
introduced a bug on "Reject" procmail recipe:

That's wrong:
....
   :0 Wh
   * !^FROM_DAEMON
   * !^X-Loop: $SENDER
   :0 eh
   | (formail -rA"X-Loop: $SENDER" \
....
resulting in all matched messages delivered to ":0" mailbox :-{

should be:
...
   :0 Wh
   * !^FROM_DAEMON
   * !^X-Loop: $SENDER
   | (formail -rA"X-Loop: $SENDER" \
....

Please correct:
ingo/lib/Script/Procmail/Recipe.php

129d128
<             $this->_action[] = '  :0 eh';


Best regards

Valerio Pulese

Saved Queries