|
From: Tanstaafl <tan...@li...> - 2013-03-11 11:26:33
|
Ok, first, I can definitely confirm that this is working now. This is awesome. We normally have 5-10 messages per day *per user* (users that have their vacation enabled that is) that get responded to by the vacation script that shouldn't be, and now they are not, and I now have a way to manually add additional strings whenever I see something in the logs getting replied to when it shouldn't. So, many many thanks to Rudi for helping me get this working! As for my below question about adding this into vacation.conf... I understand that vacation.pl is a perl script, but shouldn't it be still be possible to define the string to test for as a variable that the script can call? One other question... Here are two examples of my line tripping: > 2013/03/11 06:20:04 DEBUG> /var/spool/vacation/vacation.pl:544 main:: - Script argument SMTP recipient is : 'tbobbitt#med...@au...' and smtp_sender : 'bo...@ma...' > 2013/03/11 06:20:04 DEBUG> /var/spool/vacation/vacation.pl:549 main:: - Multi-string From header matching mediapost found; exiting > 2013/03/11 06:47:06 DEBUG> /var/spool/vacation/vacation.pl:544 main:: - Script argument SMTP recipient is : 'tbobbitt#med...@au...' and smtp_sender : 'ESC...@in...' > 2013/03/11 06:47:06 DEBUG> /var/spool/vacation/vacation.pl:549 main:: - Multi-string From header matching noreply found; exiting In the first one, it matches 'mediapost' - but since 'bounce' comes before 'mediapost', why didn't it say that it matched that one? In the second, it matches 'noreply', which is not evident in the smtp_sender. So, my question is twofold... 1. Does this test test the mail-from (I'm guessing that is what the 'smtp_sender' is) *first*, *then* any from: headers? From my tests it appears so... and secondly, 2. Why does the first one not match on 'bounce' before it gets to 'mediapost'? Is the string evaluated in reverse order or something? Either way, what are the chances of getting this added to the core vacation.pl? If I have to I can just update mine manually after every postfixadmin update, which admittedly is not all that often, but I would think other admins would really appreciate what it provides. Thanks again guys! On 2013-03-04 12:38 PM, Tanstaafl <tan...@li...> wrote: > On 2013-03-04 12:06 PM, Rudi Floren <rud...@go...> wrote: >> I hope christian will merge this into core. Than you can just change >> your vacation.conf with send_to_auto = true like the other settings. > > Ok, but... I anticipate modifying these strings to test for (usually > adding, but maybe sometimes removing if it results in too many false > positives), so I'd like to get the entire list of test strings to test > for into vacation.conf somehow... so, something like: > > +# Send vacation mails to do-not-reply email adresses. > +# By default vacation email adresses will be sent. > +# For now emails that contain any of the strings specified in > $send_to_auto_false won't > +# be answered when $send_to_auto is set to false. > +our $send_to_auto = true; > +# Strings to test for that if matched, emails will not be sent: > +#$send_to_auto_false = > "bounce|do-not-reply|facebook|linkedin|list-|myspace|twitter"; > >> Please test with a working vacation setup with test.sh I don't have a >> setup with vacation running. > > Will attempt to when I have some time... > > Many thanks again Rudi! |
