The REST API returns a 500 when requesting subscriptions for a nonexistent address.

Bug #848103 reported by Stephen A. Goss on 2011-09-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Medium
Barry Warsaw

Bug Description

Requests to /<email address hidden>/memberships causes an exception in Mailman so it returns a 500 instead of a 404.

Stephen A. Goss (postfuturist) wrote :

Here's a simple fix.

This sounds like a http.bad_request() not http.not_found().

Stephen A. Goss (postfuturist) wrote :

Possibly, though the request would be valid if someone were to subscribe to a mailing list with that email address, which is why I lean towards 404.

Barry Warsaw (barry) wrote :

Agreed that for consistency, a 404 is a better error code. Thanks for the patch!

Changed in mailman:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Barry Warsaw (barry)
milestone: none → 3.0.0a8
status: In Progress → Fix Committed
Barry Warsaw (barry) on 2011-09-25
Changed in mailman:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers