[ mailman-Bugs-1913963 ] Better error checking for MM list in accept_these_nonmembers

SourceForge.net noreply at sourceforge.net
Fri Mar 14 01:38:19 CET 2008


Bugs item #1913963, was opened at 2008-03-13 17:38
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1913963&group_id=103

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: mail delivery
Group: 2.1 beta
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: James E. Blair (corvus_at_gnu)
Assigned to: Nobody/Anonymous (nobody)
Summary: Better error checking for MM list in accept_these_nonmembers

Initial Comment:
As of the 2.1.10 beta series, accept_these_nonmembers and other related fields accept the name of another mailing list in the form "@listname", or "@listname at hostname".

Currently a user can specify a list's own name in its accept_these_nonmembers fields.  This should not be allowed for two reasons:

1) It's confusing for users -- it would be better to simply use a list's own moderation feature rather than specifying that members of a list can post by listing the name of the list in a field where you specify non-members that can post.

2) It causes a new MailList object for a list to be created inside of a section of code that already has a locked MailList object for the same list.  This is a source of potential lock problems that does not occur elsewhere in the Mailman source.

I'm attaching a patch that handles this condition as well as improving the error reporting for the case where the user specifies a list that doesn't exist.

Additionally, the patch adds protection in the GUI so that users are not permitted to enter either the name of the list they are editing, or a list that does not exist (the latter was already a suggestion in the comments).




----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1913963&group_id=103


More information about the Mailman-coders mailing list