https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7786

Henrik Krohns <apa...@hege.li> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|4.0.0                       |SVN Trunk (Latest Devel
                   |                            |Version)
                 CC|                            |apa...@hege.li
           Priority|P2                          |P3
                 OS|OpenBSD                     |All
           Hardware|PC                          |All
           Severity|normal                      |enhancement

--- Comment #1 from Henrik Krohns <apa...@hege.li> ---
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.

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to