Re: [courier-users] Courier::Filter 0.16

2005-01-27 Thread Gordon Messmer
Sam Varshavchik wrote: I think that ctlfile should be opened for append mode. ...capital idea. Am I wrong to assume that all of the calls to open() in the openctl() function should have the same flags? --- courier/submit2.C.orig Wed Jan 26 18:38:43 2005 +++ courier/submit2.C Wed Jan 26

Re: [courier-users] Courier::Filter 0.16

2005-01-26 Thread Gordon Messmer
Sam Varshavchik wrote: It's always safe to append more stuff to the control file. Whether it's safe to rewrite it from scratch is something I need to think about. At the time submit calls run_filter, that's not true. ctlfile is open and the write pointer will indicate the end of the file. If

Re: [courier-users] Courier::Filter 0.16

2005-01-26 Thread Gordon Messmer
Gordon Messmer wrote: ctlfile doesn't seem to support a seekp() method that can jump to the end of the file. Attached is a patch which adds and uses such a method. I can now modify the control file from a courierfilter without corrupting the file. diff -ru courier-0.48.2.orig/afx/afx.h

Re: [courier-users] Courier::Filter 0.16

2005-01-26 Thread Sam Varshavchik
Gordon Messmer writes: Sam Varshavchik wrote: It's always safe to append more stuff to the control file. Whether it's safe to rewrite it from scratch is something I need to think about. At the time submit calls run_filter, that's not true. ctlfile is open and the write pointer will indicate

RE: [courier-users] Courier::Filter 0.16

2005-01-24 Thread Julian Mehnle
Mark Bucciarelli wrote: [taking off list, as I think I have borked my setup.] No, unfortunately this is a general issue with IO::InnerFile. See below. Julian Mehnle wrote: Try 0.11 as soon as it appears on CPAN (which should be within the next few hours). That got me futher, but I'm

RE: [courier-users] Courier::Filter 0.16

2005-01-21 Thread Julian Mehnle
Mark Bucciarelli [EMAIL PROTECTED] wrote: On Tuesday 18 January 2005 15:03, Julian Mehnle wrote: * Added the ClamAVd filter module for malware filtering using a ClamAV `clamd` dameon. Looks like this modules required ClamAV::Client? I was not able to retrieve it using CPAN. Below

Re: [courier-users] Courier::Filter 0.16

2005-01-20 Thread Alexander Lazic
On Mit 19.01.2005 21:17, Sam Varshavchik wrote: It's always safe to append more stuff to the control file. Whether it's safe to rewrite it from scratch is something I need to think about. Cool, thanx for your great work on courier and i'am looking, very positive, forward to have these feature in

Re: [courier-users] Courier::Filter 0.16

2005-01-20 Thread Alessandro Vesely
Gordon Messmer wrote: Sam Varshavchik wrote: It's always safe to append more stuff to the control file. Whether it's safe to rewrite it from scratch is something I need to think about. I thought it was, too. However, when I started actually testing that theory, I found that it's

RE: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Julian Mehnle
Scott [EMAIL PROTECTED] wrote: Julian Mehnle wrote: Courier::Filter 0.16 has been released on CPAN[1]. The most important changes are: * Added the ClamAVd filter module for malware filtering using a ClamAV `clamd` dameon. Before I dive into this new release, does that mean

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Alexander Lazic
Hi, On Mit 19.01.2005 12:59, Julian Mehnle wrote: The point of using Courier::Filter is that it is very flexible and supports arbitrary filter modules and complex filter module configurations, not just ClamAV scanning. But at the moment the quequefile can't still be modified by any filter, isn't

RE: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Julian Mehnle
Alexander Lazic [EMAIL PROTECTED] wrote: On Mit 19.01.2005 12:59, Julian Mehnle wrote: The point of using Courier::Filter is that it is very flexible and supports arbitrary filter modules and complex filter module configurations, not just ClamAV scanning. But at the moment the quequefile

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Alexander Lazic
On Mit 19.01.2005 16:49, Julian Mehnle wrote: True. But that means ClamCour and Amavis can't do it either. Courierfilters can do SMTP-time message rejection only. Yeah :-( I like courier but these is a really bad limitation, imho :-( al ;-) ---

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Jay Lee
Alexander Lazic said: On Mit 19.01.2005 16:49, Julian Mehnle wrote: True. But that means ClamCour and Amavis can't do it either. Courierfilters can do SMTP-time message rejection only. Yeah :-( I like courier but these is a really bad limitation, imho :-( To be fair, very few mail servers

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Gordon Messmer
Jay Lee wrote: To be fair, very few mail servers I know of or have used have this feature. At least Postfix and Sendmail have it. It's not that exotic. It should be fairly simple to add to Courier. About all that needs to be done, AFAIK, is add a seek() to line 947 of submit2.C, or close the

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Sam Varshavchik
Gordon Messmer writes: Jay Lee wrote: To be fair, very few mail servers I know of or have used have this feature. At least Postfix and Sendmail have it. It's not that exotic. It should be fairly simple to add to Courier. About all that needs to be done, AFAIK, is add a seek() to line 947 of

RE: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Julian Mehnle
Sam Varshavchik [EMAIL PROTECTED] wrote: No -- the problem that prevents this from working is that by that time the message's MIME structure is already parsed. If, suddenly the message's contents change, the pre-parsed offsets of various MIME entities in the message will result in a corrupted

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Gordon Messmer
Sam Varshavchik wrote: Gordon Messmer writes: About all that needs to be done, AFAIK, is add a seek() to line 947 of submit2.C, or close the control files before run_filter and reopen them afterward. It doesn't look like the afx lib in use has a suitable seekp() function, so the latter may be

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Sam Varshavchik
Gordon Messmer writes: Sam Varshavchik wrote: Gordon Messmer writes: About all that needs to be done, AFAIK, is add a seek() to line 947 of submit2.C, or close the control files before run_filter and reopen them afterward. It doesn't look like the afx lib in use has a suitable seekp()

RE: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Mitch (WebCob)
Can't Courier just determine whether the message file was modified, and if so, re-read and re-parse the message? That should avoid any performance problems for backward compatible behavior (i.e. for non-modifying filters). [Mitch says:] That's been kicked around before - it would seem to be a

RE: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Mitch (WebCob)
Sam Varshavchik [EMAIL PROTECTED] wrote: No -- the problem that prevents this from working is that by that time the message's MIME structure is already parsed. If, suddenly the message's contents change, the pre-parsed offsets of various MIME entities in the message will result in a

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Sam Varshavchik
Mitch (WebCob) writes: Can't Courier just determine whether the message file was modified, and if so, re-read and re-parse the message? That should avoid any performance problems for backward compatible behavior (i.e. for non-modifying filters). [Mitch says:] That's been kicked around before - it

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Gordon Messmer
Sam Varshavchik wrote: Gordon Messmer writes: Sam Varshavchik wrote: No -- the problem that prevents this from working is that by that time the message's MIME structure is already parsed. If, suddenly the message's contents change, the pre-parsed offsets of various MIME entities in the message

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Sam Varshavchik
Gordon Messmer writes: Sam Varshavchik wrote: Gordon Messmer writes: Sam Varshavchik wrote: No -- the problem that prevents this from working is that by that time the message's MIME structure is already parsed. If, suddenly the message's contents change, the pre-parsed offsets of various MIME

Re: [courier-users] Courier::Filter 0.16

2005-01-19 Thread Gordon Messmer
Sam Varshavchik wrote: It's always safe to append more stuff to the control file. Whether it's safe to rewrite it from scratch is something I need to think about. I thought it was, too. However, when I started actually testing that theory, I found that it's not the case at the time that a

Re: [courier-users] Courier::Filter 0.16

2005-01-18 Thread Scott
Julian Mehnle wrote: Courier::Filter 0.16 has been released on CPAN[1]. The most important changes are: * Added the ClamAVd filter module for malware filtering using a ClamAV `clamd` dameon. Before I dive into this new release, does that mean that I can quit using amavis and instead use

Re: [courier-users] Courier::Filter 0.16

2005-01-18 Thread Frederik Dannemare
On Wednesday 19 January 2005 03:56, Scott wrote: Julian Mehnle wrote: Courier::Filter 0.16 has been released on CPAN[1]. The most important changes are: * Added the ClamAVd filter module for malware filtering using a ClamAV `clamd` dameon. Before I dive into this new release, does