Quarantine Summary Report and multiple recipients

Report bugs and workarounds
Post Reply
stusmith
Posts: 63
Joined: 27 Jan 2017 15:24

Quarantine Summary Report and multiple recipients

Post by stusmith »

Looking at what for me is line 207 in /usr/local/bin/mailwatch/tools/Cron_jobs/quarantine_report.php I see:
((to_address=%s) OR (to_domain=%s))
which is a problem because I have confirmed it does not handle any Email with multiple recipients. The end result being that if something is caught in quarantine and sent to two recipients, it will never be included in a quarantine summary report.

I'm guessing what we want to do instead is ((to_address LIKE %{%s}%) OR (to_domain LIKE %{%s})). Although that may require generation of a full text index on the maillog table to prevent being bogged down.

I'm still using EFA-3.0.1.8, so this may have been addressed already.

Thoughts?
User avatar
shawniverson
Posts: 3644
Joined: 13 Jan 2014 23:30
Location: Indianapolis, Indiana USA
Contact:

Re: Quarantine Summary Report and multiple recipients

Post by shawniverson »

I see enhancements with the quarantine report in the pipe on the MailWatch project. I plan to pull them in on 3.0.2.2 :)
stusmith
Posts: 63
Joined: 27 Jan 2017 15:24

Re: Quarantine Summary Report and multiple recipients

Post by stusmith »

Awesome!

What I ended up doing was:

/usr/local/bin/mailwatch/tools/Cron_Jobs/quarantine_report.php

Code: Select all

207  ((to_address LIKE %s) OR (to_domain LIKE %s))

215     $result = dbquery(sprintf($sql, quote_smart("%" . $to_address . "%" ), quote_smart("%" . $to_domain . "%" )));
Post Reply