| 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 | 7/28/08 (6465 days ago) |
| Due | |
| Updated | 12/28/09 (5947 days ago) |
| Assigned | 7/28/08 (6465 days ago) |
| Resolved | 11/7/08 (6363 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