Mailto Email Attribute appears to not get used in clamav-scan.sh

[gyoza]

Am I incorrect to assume that is where the specific attribute was intended to be used? Thanks!

[phendren]

After looking at the code to understand this better for myself - it seems that the mailto email attribute is setting the email "mailto" for the scan cron. However, there is also line 32 in the clamav-scan.sh script /usr/bin/mailx -s "Virus warning for $HOST at $NOW" root which will send email to the servers root user (configured email address). I will submit a PR to fix this.

[jblaine]

Ideally the script should not handle mailing at all.

It should spit its output to stdout and stderr. If it does that when run as a cron job, then the output will go to the mailto address via cron's handling. If it does that in any other context, one has control over where the output goes.