Spaze wrote:
my $filename = $transaction->body_filename; [...] unlink($filename);
Is it a leftover from the clamav plugin (which is presented as an inspiration) or is it intended (and "clear")? Will the functionality of qpsmtpd not be affected?
You're right that the unlink() probably shouldn't be there. In reality, I believe qpsmtpd still must have the file open (so the unlink is not performed), since I have been running this code in production for several weeks now. :0
John