Serge Chernyavsky wrote:
> Paul,
> 
> could you comment on my issue with double slashes?

I did already, didn't I?

> It might be gone since some newer dbmail version but it's pretty 
> difficult to find the exact Sieve changes comments in your changelog (or 
> maybe I just look in a wrong place?)

If double slashes are giving you trouble, it's not a dbmail issue.
dbmail uses and relies upon libsieve. There may be a regexp bug in
libsieve. Maybe there is. The original regexp does look valid enough,
but I couldn't get it to match the timezone part.

However, since the regexp doesn't *need* to match the timezone, just the
day, I simply simplified the regexp and be done. kiss principle. Problem
solved.


fyi, original patch attached...


> 
> Regards,
> Serge
> 
> Paul J Stevens wrote:
>> Serge,
>>
>> I didn't have an issue with double slashes, but with the regexp not
>> matching my received headers' date format. Looking at the diff involved,
>>  the scripts didn't match the timezone after the timestamp. Since those
>> weren't needed to begin with I stripped those parts of the regexp.
>>
>>
>>
>> Serge Chernyavsky wrote:
>>> Hi!
>>>
>>> Paul, which part of it do you find crappy?
>>>
>>> I had the same issue under dbmail 2.2.9 which went away after I changed 
>>> double backslashed regexes to a single backslahed in Ingo.
>>> I thought it's an Ingo bug and wrote their mailing list but was sent to 
>>> http://tools.ietf.org/id/draft-murchison-sieve-regex-08.txt which says 
>>> that "Unlike [POSIX.2], a double-backslash is required as per section 
>>> 2.4.2 of [SIEVE]."


-- 
  ________________________________________________________________
  Paul Stevens                                      paul at nfg.nl
  NET FACILITIES GROUP                     GPG/PGP: 1024D/11F8CD31
  The Netherlands________________________________http://www.nfg.nl
>From 6ffe00162a7bb906678232d44847a006a7f473c2 Mon Sep 17 00:00:00 2001
From: root <[email protected]>
Date: Fri, 18 Jul 2008 14:23:33 +0000
Subject: fix ingo sieve generation

---
 ingo/lib/Script/sieve.php |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ingo/lib/Script/sieve.php b/ingo/lib/Script/sieve.php
index 5ab5264..1c7cb98 100644
--- a/ingo/lib/Script/sieve.php
+++ b/ingo/lib/Script/sieve.php
@@ -2727,7 +2727,7 @@ class Sieve_Action_Vacation extends Sieve_Action {
             $code .= '|' . $i;
         }
         return $code
-            . ') (\\\\(.*\\\\) )?..:..:.. (\\\\(.*\\\\) )?(\\\\+|\\\\-)....( \\\\(.*\\\\))?$" {'
+            . ') (\\\\(.*\\\\) )?..:..:.. " {'
             . "\n    ";
     }
 
@@ -2742,7 +2742,7 @@ class Sieve_Action_Vacation extends Sieve_Action {
             $code .= '|' . $months[$i - 1];
         }
         return $code
-            . ') (\\\\(.*\\\\) )?.... (\\\\(.*\\\\) )?..:..:.. (\\\\(.*\\\\) )?(\\\\+|\\\\-)....( \\\\(.*\\\\))?$" {'
+            . ') (\\\\(.*\\\\) )?.... (\\\\(.*\\\\) )?..:..:.. " {'
             . "\n    ";
     }
 
@@ -2755,7 +2755,7 @@ class Sieve_Action_Vacation extends Sieve_Action {
             $code .= '|' . str_pad($i, 2, '0', STR_PAD_LEFT);
         }
         return $code
-            . ') (\\\\(.*\\\\) )?... (\\\\(.*\\\\) )?.... (\\\\(.*\\\\) )?..:..:.. (\\\\(.*\\\\) )?(\\\\+|\\\\-)....( \\\\(.*\\\\))?$" {'
+            . ') (\\\\(.*\\\\) )?... (\\\\(.*\\\\) )?.... (\\\\(.*\\\\) )?..:..:.. " {'
             . "\n    ";
     }
 
-- 
1.5.6.5

_______________________________________________
Dbmail-dev mailing list
[email protected]
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail-dev

Reply via email to