500 error code shown by postorius instead of 409 error code
Bug #1425359 reported by
Ankush Sharma
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
GNU Mailman |
Fix Released
|
High
|
Ankush Sharma |
Bug Description
Bug discussed in detail at : https:/
When a new mailman subscriber is created with the email of an already existing mailman subscriber or user a `409` code with a message `user already exists` is returned by the Mailman REST API but when the same email but now case sensitive is used a `500` code with message `server error code` is returned.
The latter should also return a `409` instead of `500` as it is misleading.
Related branches
lp:~black-perl/mailman/fix-mailman
- Barry Warsaw: Needs Fixing
-
Diff: 32 lines (+10/-1)2 files modifiedsrc/mailman/app/membership.py (+1/-1)
src/mailman/rest/tests/test_membership.py (+9/-0)
description: | updated |
description: | updated |
description: | updated |
description: | updated |
tags: | added: postorius |
Changed in mailman: | |
status: | New → Confirmed |
assignee: | nobody → Ankush Sharma (black-perl) |
Changed in mailman: | |
importance: | Undecided → High |
milestone: | none → 3.0.0b6 |
status: | Confirmed → Fix Committed |
Changed in mailman: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
This Bug was result of treating case sensitive and case insensitive addresses differently which in both cases mailman treats as identical. So always comparing the strings by making the cases identical (lower case here) , will solve the bug. The solution thus provided to make the cases similar seems apt.