search

thias / puppet-postfix

Puppet Postfix module
Other
17 stars 117 forks source link

clamav support #29

Closed tjnicholas closed 10 years ago

tjnicholas commented 10 years ago

Is there a specific module that is expected to be installed for 'clamav => true' to work?

I can't find a module in the forge that has a clamav::smtp class...

Cheers, Tim

tjnicholas commented 10 years ago

Actually, I'm not sure configuring an smtp virus scanner should really be in a different module. Perhaps I'll just add a postfix::clamav subclass and send you a pull request.

thias commented 10 years ago

You can find the module here : https://github.com/thias/puppet-modules/tree/master/modules-wip It's old and not in a very good shape, but I think I'm still running it in production somewhere :-)

The "problem" with ClamAV is that it's more than just an SMTP antivirus. It's a command-line tool with signature databases, update scripts... definitely worth its own module IMHO.

But feel free to put in into shape if you have the need for it, I would be fine with reviewing major changes to it and properly releasing it to the forge.