Re: autocreate.sieve doesn't work correct. temporary files

2015-10-20 Thread Artyom Aleksandrov
Sorry for delay. Yes! Now it works! On Fri, Oct 16, 2015 at 1:57 AM, ellie timoney wrote: > Aha! > > Okay, it was calling cyrus_mkdir() with the wrong path: cyrus_mkdir() > expects to get a path to a *file*, and it ensures all the directory > components exist (but ignores the file part). But

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-15 Thread ellie timoney
Aha! Okay, it was calling cyrus_mkdir() with the wrong path:  cyrus_mkdir() expects to get a path to a *file*, and it ensures all the directory components exist (but ignores the file part).  But we were passing it only the directory component (/var/spool/sieve/r/root).  It was ignoring "root", ass

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-14 Thread Artyom Aleksandrov
I'm glad to help =) After applying all patches I got follow Oct 14 23:11:18 imapsync cyrus/imap[28525]: User root, Inbox subfolders, created 4, subscribed 4 Oct 14 23:11:18 imapsync cyrus/imap[28525]: autocreate_sieve: Unable to create /var/spool/sieve/r/root/default.script.bc.NEW: No such file o

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-08 Thread ellie timoney
Hi again, A couple more patches: >> First I guest we need NOT here (?) > > Doh! acsieve-sievedir.patch corrects this (though it sounds like you've already made this change yourself -- included for completion though) >> Oct  7 16:54:10 imapsync cyrus/imap[4062]: autocreate_sieve: >> Unable to cr

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-08 Thread ellie timoney
> First I guest we need NOT here (?) Doh! > And second creating are failing with follow logs (yes I use 'root' > user for testing) > > Oct  7 16:54:10 imapsync cyrus/imap[4062]: User root, Inbox > subfolders, created 4, subscribed 4 Oct  7 16:54:10 imapsync > cyrus/imap[4062]: autocreate_sieve: U

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-08 Thread Bron Gondwana
Ok, thanks for the info.  I'll get a test case into Cassandane based on your info so we can be sure it gets fixed. On Fri, Oct 9, 2015, at 01:30, Artyom Aleksandrov wrote: > Definitely no. It's just normal user > > grep '_admins:' imapd.conf #lmtp_admins: postman #mupdate_admins: > mupdateman #im

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-08 Thread Artyom Aleksandrov
Definitely no. It's just normal user grep '_admins:' imapd.conf #lmtp_admins: postman #mupdate_admins: mupdateman #imap_admins: cyrus #sieve_admins: cyrus On Thu, Oct 8, 2015 at 12:55 AM, Bron Gondwana wrote: > Is user 'root' an admin user? Lots of things behave differently for > admins. > >

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-07 Thread Bron Gondwana
Is user 'root' an admin user?  Lots of things behave differently for admins. On Thu, Oct 8, 2015, at 01:04, Artyom Aleksandrov wrote: > Hi, thank you for patch. But it does not work. (( First I guest we > need NOT here (?) > > @@ -139,7 +138,7 @@ static int autocreate_sieve(const char *userid, >

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-07 Thread Artyom Aleksandrov
Hi, thank you for patch. But it does not work. (( First I guest we need NOT here (?) @@ -139,7 +138,7 @@ static int autocreate_sieve(const char *userid, const char *source_script) } /* Check if sievedir is defined in imapd.conf */ -if(!(sieve_dir = config_getstring(IMAPOPT_SIEVEDIR

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-06 Thread Artyom Aleksandrov
Hi, You should to configure Cyrus that it put default sieve to user _sieve_path during autocreating user mailbox. Please follow this manual https://docs.cyrus.foundation/imap/features/automatic-creation-of-mailboxes.html and notice that this feature is available only in 2.5 branch. Roundcube cre

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-05 Thread signaldeveloper
Where are the default sieve rules at? If I go into round cube there is a sieve group called round cube. Where is that at? - Paul > On Oct 5, 2015, at 8:51 PM, ellie timoney wrote: > > Hi Artyom, > > Yeah wow, this is really gross. > > I'm pretty sure the gibberish in those "??Default.s

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-05 Thread ellie timoney
Hi Artyom, Yeah wow, this is really gross. I'm pretty sure the gibberish in those "??Default.script.bc" filenames is just whatever junk was in the (uninitialised) sieve_script_dir variable. I've had a rummage around, and there's a user_sieve_path() function in imap/user.c that does the heavy

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-05 Thread Artyom Aleksandrov
Guys I don't understand hot it can work. I added additional logging and found that sieve_script_dir is not defined. After adding this definition the problem gone. 147 /* Check if autocreate_sieve_compiledscript is defined in imapd.conf */ 148 if(!(compiled_source_script = config_getstri

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-03 Thread Bron Gondwana
Guess I chose first ;) Patches (untested in production, but I do have Cassandane tests for them) https://github.com/brong/cyrus-imapd/commit/6f3e97023c16b1af19926b6ef67b6dc2ddcc844b https://github.com/brong/cyrus-imapd/commit/bda966274a8f53497bcf510a43b1bfc81baf4f7f They'll probably be rebased a

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Bron Gondwana
It's not impossible yet, though maybe it should be. I'll pick either the first or last depending on implementation ;) Bron. On Sat, Oct 3, 2015, at 09:37, Stephen Ulmer wrote: > Now THAT is a good idea! > > What are you going to do for delivery if there’s more than one folder > with the flag (or

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Stephen Ulmer
Now THAT is a good idea! What are you going to do for delivery if there’s more than one folder with the flag (or is that already impossible some other way)? -- Stephen > On Oct 2, 2015, at 5:43 PM, Bron Gondwana wrote: > > I'm planning to create an option to reject delete or move into a su

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Bron Gondwana
I'm planning to create an option to reject delete or move into a sub folder for any folder with a special-use flag. We would allow rename. Of course with this would be a sieve extension to do delivery to a special- use flag instead of folder name... Bron. On Sat, Oct 3, 2015, at 05:41, Stephen Ul

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Stephen Ulmer
A user deleting her spam folder won’t be prevented by a sieve script. I hope that most of your spam identification rules are run in your MTA (not in sieve) and then sieve is used to sort messages based on a a few simple flags. There may be a perfectly good use case for confusing users by having

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Mai Ling
Even if you create once & forget, there are still cases where site-enforced rules are useful: On the previous setup I've managed I've encountered lots of cases where people accidentally deleted their spam folders or their spam rules; Having an user rule automagically imported/enforced from a site

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Stephen Ulmer
I would not imagine that you’d need (or want) to change all of the users’ rules to make an update for spam filtering. Just filter on an X- header that’s added by your MTA. Change the way the header gets calculated, but not what it means to the sieve script. This is presuming that you make the m

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Mai Ling
Oh awesome! I wish they add support for editable site-default sieve rules that override user sieve rules. This way you won't have to update all users rules when you want to make modification to everyones' rules (usecase: spam filtering) On Sâm, sept. 26, 2015 at 1:30 a.m., Artyom Aleksandrov wrot

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-02 Thread Artyom Aleksandrov
Yes patch from uoa.gr was finally applied at version 2.5 https://cyrusimap.org/mediawiki/index.php/RoadMap On Thu, Oct 1, 2015 at 10:23 PM, Mai Ling wrote: > Whoa, wait! has cyrus finally integrated uoa.gr autocreate patches?! Or > is it something different with same name?! > > On Joi, oct. 1,

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-01 Thread Mai Ling
Whoa, wait! has cyrus finally integrated uoa.gr autocreate patches?! Or is it something different with same name?! On Joi, oct. 1, 2015 at 7:18 p.m., Artyom Aleksandrov wrote: Does anybody use autocreate_sieve? On Sat, Sep 26, 2015 at 1:30 AM, Artyom Aleksandrov < mailing.l...@tem4uk.ru [mai

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-01 Thread Artyom Aleksandrov
Is it works? Which version do you use? Could you guest the reason of the problem? How I can troubleshoot it? :/var/lib/cyrus# ls -la total 2176 -rw--- 1 cyrus mail 124 Sep 25 16:04 ??Default.script.bc -rw--- 1 cyrus mail 231 Sep 25 16:04 ??Default.script.script lrwxrwxrw

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-01 Thread Alvin Starr
I use autocreate. So there is at least one. On 10/01/2015 12:18 PM, Artyom Aleksandrov wrote: Does anybody use autocreate_sieve? On Sat, Sep 26, 2015 at 1:30 AM, Artyom Aleksandrov mailto:mailing.l...@tem4uk.ru>> wrote: Hello,I want to create default sieve scipt for all my users but I

Re: autocreate.sieve doesn't work correct. temporary files

2015-10-01 Thread Artyom Aleksandrov
Does anybody use autocreate_sieve? On Sat, Sep 26, 2015 at 1:30 AM, Artyom Aleksandrov wrote: > Hello,I want to create default sieve scipt for all my users but I stuck > with strange problem that looks like the bug. Unfortunately I've never > wrote on C so it's difficult for me to find it. > Whe