SA Bugzilla – Bug 7786
Clamav plugin
Last modified: 2022-04-22 07:01:58 UTC
If anyone would like to play with it, here is the clamav plugin I recently wrote, working but still under heavy development. https://github.com/bigio/spamassassin-clamav
Not sure what the purpose of keeping this bug open is.. of course you are free to commit it for SA project to take over maintaining. But please not "Clamav", as again proper designation for it is "ClamAV".. details matter. But here's some observations if that was the point of this bug: - clamd_sock seems to assume that clamd is always running on localhost and not some other server - it's useless to have separate hardcoded "check_clamav_sanesecurity" function, everything should go through check_clamav, otherwise you need to update plugin and if can(foo) constantly for every signature. So check_clamav() should just accept a custom regex - add result caching if called multiple times (with different regex as above for different scoring etc) - it's loading two modules needlessly, try loading other only if other was not available - why is it using $self->{main}->{registryboundaries}->{conf} - of course custom async call would be preferable, instead of waiting for clamd response. The protocol is so simple it's probably pointless to even use external modules for this. So I'd rather see it committed only after much polishing.
Thanks, I will commit it when/if I will find time/interest to improve it.