Re: [Mailman-Users] digest threshold problem

Sean wrote:
Can anyone point me to the code that checks the digest size for the daily senddigests cron job? Much appreciated.
The daily senddigests cron job does not do any checking of digest size. The Mailman/Handlers/ToDigest.py handler is responsible for queueing the digest to be sent when it adds a post that puts the digest mailbox over the threshhold size.
The periodic job (cron/senddigests) attempts to send a digest only for lists for which the digest_send_periodic attribute is true. The code is
for listname in listnames:
mlist = MailList.MailList(listname, lock=0)
if mlist.digest_send_periodic:
mlist.Lock()
try:
mlist.send_digest_now()
mlist.Save()
finally:
mlist.Unlock()
It's hard to see how that could possibly send a digest for a list for which digest_send_periodic is not true.
-- Mark Sapiro msapiro@value.net The highway is for gamblers, San Francisco Bay Area, California better use your sense - B. Dylan

Thanks Mark. Of course the incident has not repeated itself so it looks like it was probably human error that caused the digest to be sent.
Sean
On Fri, 27 May 2005, Mark Sapiro wrote:
Sean wrote:
Can anyone point me to the code that checks the digest size for the daily senddigests cron job? Much appreciated.
The daily senddigests cron job does not do any checking of digest size. The Mailman/Handlers/ToDigest.py handler is responsible for queueing the digest to be sent when it adds a post that puts the digest mailbox over the threshhold size.
The periodic job (cron/senddigests) attempts to send a digest only for lists for which the digest_send_periodic attribute is true. The code is
for listname in listnames: mlist = MailList.MailList(listname, lock=0) if mlist.digest_send_periodic: mlist.Lock() try: mlist.send_digest_now() mlist.Save() finally: mlist.Unlock()
It's hard to see how that could possibly send a digest for a list for which digest_send_periodic is not true.
-- Mark Sapiro msapiro@value.net The highway is for gamblers, San Francisco Bay Area, California better use your sense - B. Dylan
participants (2)
-
Mark Sapiro
-
Sean