Re: Alternative Postfix-Queue Plugin 'postfix-maildrop'

2012-11-28 Thread Matt Simerson

It doesn't look like it would be very difficult to update your plugin so that 
it works using the drop-in method, and the old method as well.  Perhaps with a 
config setting that lets people swap between the efficient -vs- paranoid method?

I don't have a preference on this matter, but others that use this plugin may. 
Having a backwards compatible option, and a paragraph of POD explaining the 
difference (pros  cons) would go a long ways towards smoothing over any 
potential objections. 

Matt

On Nov 28, 2012, at 7:16 PM, Michael Holzt k...@fqdn.org wrote:

 Hello everyone,
 
 considering what I wrote in my last mails, I have now written my own
 experimental queue plugin for postfix which will drop the mail into
 the postdrop-Spool and then notify the pickup daemon.
 
 With this plugin the queuing of a 10 MB mail will now only take
 three seconds. The processing afterwards (by the pickup daemon)
 will still take two minutes, but at least the SMTP client is no
 longer stalled and might run into an error.
 
 This plugin currently duplicated some of the functions out of
 Qpsmtpd::Postfix. It might make sense to add this as an alternative
 inject method to that class.
 
 My plugin is attached. I don't make any promised but it works for
 me. Testing by others and feedback would be very much appreciated.
 
 
 
 Regards
 Michael
 
 -- 
 It's an insane world, but i'm proud to be a part of it. -- Bill Hicks
 postfix-maildrop.txt



Re: Alternative Postfix-Queue Plugin 'postfix-maildrop'

2012-11-28 Thread Michael Holzt
 It doesn't look like it would be very difficult to update your 
 plugin so that it works using the drop-in method, and the old 
 method as well.  Perhaps with a config setting that lets people 
 swap between the efficient -vs- paranoid method?

I mentioned in my other mail that it probably makes sense to have
both methods. However the functionality of the current plugin is
mostly not done in the plugin itself but in Qpsmtpd::Postfix. I'm
not sure why this is so, and I don't think it is good to have that
code in Qpstpd-Core, but that is how it is.

So to integrate the alternative method, Qpsmtpd::Postfix needs to
be modified and then the existing Postfix-queue plugin could be
enhanced to support both methods.

I will provide a patch to do so after some more testing and when
I'm confident that everything works fine. I sent the plugin as of
now mainly in the hope that others might test it as well.


Regards
Michael

-- 
It's an insane world, but i'm proud to be a part of it. -- Bill Hicks