Summary | Compound filters with procmail |
Queue | Ingo |
Queue Version | 1.2 |
Type | Bug |
State | Resolved |
Priority | 2. Medium |
Owners | chuck (at) horde (dot) org |
Requester | tim (at) sig (dot) msstate (dot) edu |
Created | 07/28/2008 (6185 days ago) |
Due | |
Updated | 12/28/2009 (5667 days ago) |
Assigned | 07/28/2008 (6185 days ago) |
Resolved | 11/07/2008 (6083 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
http://bugs.horde.org/ticket/8804
I noticed a problem at procmail script generation.
'To,Cc,Bcc,Resent-to'
is translated to
* \^To,Cc,Bcc,Resent-to: ...
should be
* ^To,Cc,Bcc,Resent-to: ...
I solved this as follows:
--- ingo/lib/Script/procmail.php_orig 2009-11-11 08:55:31.000000000 +0100
+++ ingo/lib/Script/procmail.php 2009-11-11 08:59:06.000000000 +0100
@@ -640,7 +640,7 @@
if (strpos($condition['field'], ',') == false) {
$string = '^' . $condition['field'] . ':';
} else {
- $string .= '/^(' . str_replace(',', '|',
$condition['field']) . '):';
+ $string .= '^(' . str_replace(',', '|',
$condition['field']) . '):';
}
$prefix = ' ';
}
State ⇒ Resolved
Taken from ben
Taken from Matt Selsky
Assigned to Chuck Hagenbuch
#7611.http://cvs.horde.org/diff.php/ingo/docs/CHANGES?r1=1.206&r2=1.207&ty=u
http://cvs.horde.org/diff.php/ingo/lib/Script/maildrop.php?r1=1.23&r2=1.24&ty=u
http://cvs.horde.org/diff.php/ingo/lib/Script/procmail.php?r1=1.99&r2=1.100&ty=u
Assigned to ben
State ⇒ Assigned
Priority ⇒ 2. Medium
Type ⇒ Bug
Summary ⇒ Compound filters with procmail
Queue ⇒ Ingo
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
'To,Cc,Bcc,Resent-to' => array(
'label' => _("Destination (To,Cc,Bcc,etc)"),
'type' => INGO_STORAGE_TYPE_HEADER
doesn't get translated into the proper procmail recipe. The above
translates into:
* ^To,Cc,Bcc,Resent-to: ...
which actually wouldn't match any headers. I am guessing the best
place to fix this is in the procmail driver but I haven't had time to
look too much into it yet. Right now I just comment out the compound
headers (the special case Desitnation is all I need for procmail
anyway), but a quick test shows that changing to
'^(To|Cc|Bcc|Resent-to)' => array( ... in fields.php is a temporary
fix for procmail (but probably breaks other drivers).
Thanks,
Tim