[Bug 710182] [NEW] REST API: Subribing to more then one list raises AssertionError / returns HTTP 500

Public bug reported: Subscribing to more than one mailing list with the same email address via the REST API raises an AssertionError in app/membership.py and and subsequently returns a 500 HTTP status code. System Info: Ubuntu 10.10 Python 2.6.6 Mailman 3.0.0a7 (Rev. 6976) I added a patch file containing a test for multiple list subscriptions and a possible fix. Cheers Florian ** Affects: mailman Importance: Undecided Status: New ** Tags: mailman3 rest-api -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/710182 Title: REST API: Subribing to more then one list raises AssertionError / returns HTTP 500

** Patch added: "rest_api_membership_error.diff" https://bugs.launchpad.net/bugs/710182/+attachment/1817393/+files/rest_api_m... -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/710182 Title: REST API: Subribing to more then one list raises AssertionError / returns HTTP 500

Confirmed, and thanks for the patch Florian. I'm going to do the test a little differently though because this is a bug in the lower level API, specifically mailman.app.membership.add_member(). It affects more than the REST API so I'm going to create a new unittest in mailman/app. ** Changed in: mailman Status: New => Confirmed ** Changed in: mailman Importance: Undecided => Medium ** Changed in: mailman Assignee: (unassigned) => Barry Warsaw (barry) ** Changed in: mailman Milestone: None => 3.0.0a7 ** Summary changed: - REST API: Subribing to more then one list raises AssertionError / returns HTTP 500 + Typo in add_member() prevents subscribing an email already linked to an existing user ** Changed in: mailman Status: Confirmed => Fix Committed -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/710182 Title: Typo in add_member() prevents subscribing an email already linked to an existing user

r6977 ** Tags removed: rest-api -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/710182 Title: Typo in add_member() prevents subscribing an email already linked to an existing user

** Changed in: mailman Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/710182 Title: Typo in add_member() prevents subscribing an email already linked to an existing user
participants (2)
-
Barry Warsaw
-
Florian Fuchs