This one took a little digging. I upgraded to 0.31 (from 0.29) in SME Server 7.0beta4, and we've had bugs raised about ClamAV and SpamAssassin not working. They are, just not for small mails...

If messages are smaller than 10k, the scanners don't get to see them:

https://sourceforge.net/tracker/index.php?func=detail&aid=1298343&group_id=96750&atid=615772

You can set a smaller size as noted in the WORKAROUND in the bug, but this appears to be the culprit:

Transaction.pm:
[...]
# TODO - should we create the file here if we're storing as an array?
sub body_filename {
  my $self = shift;
  return unless $self->{_body_file};
  return $self->{_filename};
}
[...]

The options appear to be:

- Change all of the scanners to deal with array or filename
- Auto-create a file when this method is called (as the TODO says)

AFAICT, only the scanners call this method and the command-line scanners need a filename to play with, so the first option is out for most scanners.

So, I guess the TODO needs to be done. And if we're not doing this for 0.31, we should mention (a form of) the workaround in the release notes.

Thoughts?

Gordon

Reply via email to