[Mailman-Developers] Probe messages should not sent Precedenceheader
mark at msapiro.net
Sat Feb 18 18:22:53 CET 2012
Aamir Khan wrote:
>I guess in MM3, there is no function as sendNextNotification(), because i
>used command, "grep sendNextNotification() -r *" and the result was only
Which means that bin/disabled.py won't work because it hasn't been
fully converted from 2.1 yet. It should be using the send_probe()
function instead of sendNextNotification().
>There is a file 'src/mailman/email/messages.py', in which
>UserNotification() class is defined. It has a send function, which sets the
>In file 'src/mailman/app/bouces.py', I guess the probe messages are sent
>directly using send() function of UserNotification() class. I think that
>from line 211 to 228, probe message is used to send VERP probe. But then, i
>am not able to find out the place from where probe message is sent after
>disabling of the address because of over bouncing.
There should be two places. bin/disabled.py should be calling
send_probe() instead of the non-existent sendNextNotification() list
Also, send_probe() should be called when delivery is initially disabled
as a result of a call to IBounceProcessor(), but I am unable to follow
how this works or even if it is implemented yet at least in part
because I haven't yet learned anything about how things like
>I have tried to make some changes to code and pushed it to my local branch
>. Ran through all the tests, and none of them failed. Is there any way
>to run through a subset of test rather than running them using 'bin/test'
>Comments, feedback required on patch.
The patch itself looks good except for style. We normally don't put
whitespace around the = in keyword=value arguments in definitions and
calls. PEP 8 <http://www.python.org/dev/peps/pep-0008/> says "Don't
use spaces around the '=' sign when used to indicate a keyword
argument or a default parameter value."
Mark Sapiro <mark at msapiro.net> The highway is for gamblers,
San Francisco Bay Area, California better use your sense - B. Dylan
More information about the Mailman-Developers