On Tue, Feb 28, 2006, Oleg Lapshin <[EMAIL PROTECTED]> said:
> Aaron Stone wrote:
>> On Tue, Feb 28, 2006, Oleg Lapshin <[EMAIL PROTECTED]> said:
>> > When I changed line 673 of timsieve.c from:
>> > "{%u+}\r\n",
>> > to
>> > "{%u}\r\n",
>> > kmail can get script to change it!!!
>>
>> They need to read the draft RFC:
>> http://www.ietf.org/internet-drafts/draft-martin-managesieve-06.txt
>>
>> 2.9. GETSCRIPT Command
>
> Yes, I read this.
> Nevertheless, both kmail and smartsieve don't like {size+} syntax
> They like {size} one... :)
There's a basic rule of these protocols that says you should speak the RFC
as closely as possible, but understand as widely as possible. Reading the
SmartSieve code, they use this regex to match the response:
elseif (preg_match("/^\{(\d+)\+?\}\r\n$/", $line, $m)){
Appears to me that the + is optional and understood if there or not... I'm
looking to see if that was added in a recent CVS update. SmartSieve hasn't
had a formal release in a while, but there has been lots of CVS activity.
Try a recent snapshot maybe?
> And 1 more thing:
> when I try to upload _new_ script from kmail,
> small script is uploaded fine
> but large - don't
>
> dbmysql.c,db_query: executing query [BEGIN]
> dbmysql.c,db_query: executing query [SELECT COUNT(*) FROM dbmail_sievescripts
> WHERE owner_idnr = 2 AND name = '@!temp-script!@']
> dbmysql.c,db_query: executing query [INSERT into dbmail_sievescripts
> (owner_idnr, name, script, active) values (2, '@!temp-script!@', '
>
> <skip large script>
>
> ]
> dbmysql.c,db_query: executing query [COMMIT]
> sortsieve.c, sort_getscript: Getting default script named [EMAIL PROTECTED]@]
> dbmysql.c,db_query: executing query [SELECT script from dbmail_sievescripts
> where owner_idnr = '2' and name = '@!temp-script!@']
> Error 12 when calling sieve2_validate: Sieve Error: script was not retrieved
>
> timsieve.c, tims: Error inserting script 2
>
> I don't know, but script don't saved to dbmail_sievescripts table.
How big a script do you need to cause a failure? I'll try to reproduce it!
Aaron
