queuedDelivery should be overridable

Reported by Marc Tardif on 2008-02-12
6
Affects Status Importance Assigned to Milestone
Zope 3
Undecided
Unassigned
zope.sendmail
Medium
Tres Seaver
zope.sendmail (Ubuntu)
Undecided
Unassigned

Bug Description

The use case is where a configuration can be used in the context of a daemon and also in the context of a command line application, such as a cron for example. The problem is that if the configuration happens to define queuedDelivery, the application typically never exits because there is a thread continuously polling the queuePath. The solution would be to provide a mechanism to have includeOverrides replace queuedDelivery with directDelivery. In order to support this feature, the discriminator for both would have to be the same. However, they are currently different which makes the override impossible. Please see the patch attached to this message.

Marc Tardif (cr3) wrote :
Tres Seaver (tseaver) on 2010-04-15
Changed in zope3:
status: New → Won't Fix
Tres Seaver (tseaver) on 2010-04-21
tags: added: bugday20100424
Tres Seaver (tseaver) wrote :

This patch looks sane to me, but I don't know the innards of zope.sendmail very well. A couple of points in its favor:

- All tests continue to pass with the patch applied.

- The change to the discriminator makes it match what the handler actually does at execution time.

One downside: the patch does not add a test which demonstrates the failure case.

Changed in zope.sendmail:
importance: Undecided → Medium
status: New → Confirmed
Tres Seaver (tseaver) on 2012-05-23
Changed in zope.sendmail:
assignee: nobody → Tres Seaver (tseaver)
milestone: none → 3.7.5
Tres Seaver (tseaver) wrote :
Changed in zope.sendmail:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
Logan Rosen (logan) on 2012-07-15
Changed in zope.sendmail (Ubuntu):
status: New → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package zope.sendmail - 3.7.5-0ubuntu1

---------------
zope.sendmail (3.7.5-0ubuntu1) quantal; urgency=low

  * New upstream release.
    - Fixes "queuedDelivery should be overridable." (LP: #191143)
    - Fixes "SMTPRecipientsRefused not handled in zope.sendmail." (LP: #1003288)
  * Bump Standards-Version to 3.9.3.
 -- Logan Rosen <email address hidden> Sun, 15 Jul 2012 19:07:58 -0400

Changed in zope.sendmail (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers