allow specifying another list in accept_these_nonmembers

Bug #558204 reported by jtittsler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Fix Released
Medium
Unassigned

Bug Description

Add the capability to logically include the addresses
all of the members of another list in the same Mailman
installation to accept_these_nonmembers.

'@listname' checks the poster's address against the
membership of listname.

Revision history for this message
jtittsler (jtittsler-users) wrote :

The file listinclusion.patch was added: patch against 2.1.6 (should also apply to 2.1.5)

Revision history for this message
tkikuchi (tkikuchi-users) wrote :

Logged In: YES
user_id=67709

Marking as an Mailman 2.2 new feature candidate.

Revision history for this message
msapiro (msapiro-users) wrote :

Logged In: YES
user_id=1123998

The 2.1.6 patch also applies without change to Mailman 2.1.7
and 2.1.8. Even though there are minor changes in the
patched modules, the patched line numbers and the patch data
are unchanged.

Revision history for this message
msapiro (msapiro-users) wrote :

Logged In: YES
user_id=1123998
Originator: NO

The patch also applies without change to Mailman 2.1.9.

Revision history for this message
tkikuchi (tkikuchi-users) wrote :

Logged In: YES
user_id=67709
Originator: NO

Because '@' is mandatory in mailman 2.2 listname, I think that '`listname@domain`' is more preferable. Use of '`' is meaning 'evaluate this list member'.

I also think we can merge this patch (along with the sibling list patch #1347962) in the next mailman 2.1.10 because release of 2.2 is delayed considerably.

Thoughts ?

Revision history for this message
libertytrek (libertytrek) wrote :

Logged In: YES
user_id=1881071
Originator: NO

Hello,

I'm very interested in using this feature, but I'm not very good at adding custom patches... any chance of an update on when this will be added to the core code?

Thanks,

Charles

Revision history for this message
msapiro (msapiro-users) wrote :

Logged In: YES
user_id=1123998
Originator: NO

Patch will be in 2.1.10.

Revision history for this message
Mark Sapiro (msapiro) wrote :

The attached list inclusion patch is a one-line change to the original patch plus expansion of some tabs. The change replaces "if mother.members.has_key(sender)" with "if mother.isMember(sender)". Using the isMember() method will work with all MemberAdaptors, not just OldStyleMemberships.py. Also, members.has_key(sender) only tests regular members, not digest members.

This corrected patch is provided for those still using Mailman 2.1.9 or older that want this functionality. The patch is not required for Mailman 2.1.10 and newer.

Mark Sapiro (msapiro)
Changed in mailman:
milestone: none → 2.1-stable
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.