[Bug 837676] [NEW] Getting the roster or configuration of a nonexistent list returns a 500 from REST API.

Public bug reported: The REST API doesn't return a 404 when the roster or configuration is requested for a nonexistent list, instead it returns a 500 due to an uncaught exception. ** Affects: mailman Importance: Undecided Status: New ** Tags: mailman3 -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837676 Title: Getting the roster or configuration of a nonexistent list returns a 500 from REST API. To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837676/+subscriptions

Here are some tests for correct behavior which fail. ** Patch added: "837676_tests.patch" https://bugs.launchpad.net/mailman/+bug/837676/+attachment/2334794/+files/83... -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837676 Title: Getting the roster or configuration of a nonexistent list returns a 500 from REST API. To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837676/+subscriptions

Here is a fix. ** Patch added: "837676_fix.patch" https://bugs.launchpad.net/mailman/+bug/837676/+attachment/2334803/+files/83... -- You received this bug notification because you are a member of Mailman Coders, which is subscribed to GNU Mailman. https://bugs.launchpad.net/bugs/837676 Title: Getting the roster or configuration of a nonexistent list returns a 500 from REST API. To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837676/+subscriptions

Nicely done, thanks. I moved the tests to unittests though. ** 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.0a8 ** 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/837676 Title: Getting the roster or configuration of a nonexistent list returns a 500 from REST API. To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837676/+subscriptions

** 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/837676 Title: Getting the roster or configuration of a nonexistent list returns a 500 from REST API. To manage notifications about this bug go to: https://bugs.launchpad.net/mailman/+bug/837676/+subscriptions
participants (2)
-
Barry Warsaw
-
Stephen A. Goss