RFE: have a default for discard_these_nonmembers

[Moritz Baumann]

I would like to have a
DEFAULT_DISCARD_THESE_NONMEMBERS = []

in mm_cfg.py / Defaults.py

and under

/usr/lib/mailman/Mailman# diff MailList.py MailList.py.bak
397c397
< self.discard_these_nonmembers = mm_cfg.DEFAULT_DISCARD_THESE_NONMEMBERS
---
> self.discard_these_nonmembers = []

/usr/lib/mailman/Mailman# diff versions.py versions.py.bak
114c114
< l.discard_these_nonmembers = mm_cfg.DEFAULT_DISCARD_THESE_NONMEMBERS
---
> l.discard_these_nonmembers = []

[Mark Sapiro (msapiro)]

My initial reaction is twofold:
1) Why not all *_these_nonmembers?
2) I don't think this will be of sufficient general interest to justify adding yet more configuration settings.

Please provide more information about your use case so I can see that it might be of general interest.

[Moritz Baumann (flaminmoe)]

I don't understand your question 1)

We will be using a spamfiltering solution called mailcleaner (https://www.mailcleaner.net/)

which will send a spam report about which mails are in quarantaine to each destination email it sees.

So as soon as mailinglist mails get routed through there
this spam report will be sent to the whole list. We want this to be discarded by default but give the list-owner the option to remove this discard.

[Moritz Baumann (flaminmoe)]

a now I see what you mean by 1).
Well the others would not hurt either. It's just that we only need discard_these_nonmembers

[Moritz Baumann (flaminmoe)]

The problem with the report to be sent to the mailing list is that there is an admin link for this quarantine mailbox (and if the list is public) everyone would see this and might modify white/blacklist etc.