May 9, 2000
10:51 a.m.
[Harald Meland]
[Brian Edmonds]
The change is on line 91 of Handlers/Sendmail.py[1], where msg.GetSender needs to become mlist.GetAdminEmail.
As I mentioned some weeks ago on this same issue, I think it would be better to override the GetSender() method appropriately for the various subclasses of Message.
If you hardcode in GetAdminEmail as envelope sender, you can get "interesting" results when combined with MTA sender verification and e.g. newly created lists. It will also cause e.g. password reminder bounces to go to some unsuspecting (public) list's admin and not to mailman-owner.
Some might consider that a feature, of course ;)
I finally got round to backporting these changes to CVS mailman:
As usual, I haven't tested the patch with CVS Mailman -- it's just a backporting of the code we've been running here during the last months.
Harald