[Bug 973790] [NEW] 'approved' rule should not modify the message

[Barry Warsaw]

Public bug reported:

In Mailman 3, technically rules should not have side effects.  The
'approved' rule breaks this because it removes the Approved header (and
other supported headers) from the message.  There should instead be a
handler that does the header cleaning, and the rule should not do this.

>From an implementation standpoint, the same code can be used in both the
handler and in the rule.  You can use a slightly different API wrapper
and use a flag to indicate whether the header should be removed or not.
OTOH, some clever refactoring might make better sense.

** Affects: mailman
     Importance: Undecided
         Status: New


** Tags: mailman3

-- 
You received this bug notification because you are a member of Mailman
Coders, which is subscribed to GNU Mailman.
https://bugs.launchpad.net/bugs/973790

Title:
  'approved' rule should not modify the message

To manage notifications about this bug go to:
https://bugs.launchpad.net/mailman/+bug/973790/+subscriptions