diff options
| author | Barry Warsaw | 2014-08-12 20:12:17 -0400 |
|---|---|---|
| committer | Barry Warsaw | 2014-08-12 20:12:17 -0400 |
| commit | bc4e33da9a97df35d147e58535358b2468cd1cc8 (patch) | |
| tree | 4aa21daa4614ff8e7ecc9c00c39267a456cf8c36 /src/mailman/rest/addresses.py | |
| parent | 61ad6fde0a9cff213a347838f19878a7b52f5466 (diff) | |
| download | mailman-bc4e33da9a97df35d147e58535358b2468cd1cc8.tar.gz mailman-bc4e33da9a97df35d147e58535358b2468cd1cc8.tar.zst mailman-bc4e33da9a97df35d147e58535358b2468cd1cc8.zip | |
Diffstat (limited to 'src/mailman/rest/addresses.py')
| -rw-r--r-- | src/mailman/rest/addresses.py | 19 |
1 files changed, 11 insertions, 8 deletions
diff --git a/src/mailman/rest/addresses.py b/src/mailman/rest/addresses.py index 6e9eb8872..d42a66aa7 100644 --- a/src/mailman/rest/addresses.py +++ b/src/mailman/rest/addresses.py @@ -37,8 +37,7 @@ from mailman.interfaces.address import ( ExistingAddressError, InvalidEmailAddressError) from mailman.interfaces.usermanager import IUserManager from mailman.rest.helpers import ( - BadRequest, CollectionMixin, NotFound, child, etag, path_not_found, - path_to) + BadRequest, CollectionMixin, NotFound, child, etag, path_to) from mailman.rest.members import MemberCollection from mailman.rest.preferences import Preferences from mailman.rest.validator import Validator @@ -115,7 +114,8 @@ class AnAddress(_AddressBase): def on_get(self, request, response): """Return a single address.""" if self._address is None: - path_not_found(request, response) + falcon.responders.path_not_found( + request, response, b'404 Not Found') else: response.status = falcon.HTTP_200 response.body = self._resource_as_json(self._address) @@ -178,7 +178,8 @@ class UserAddresses(_AddressBase): def on_get(self, request, response): """/addresses""" if self._user is None: - path_not_found(request, response) + falcon.responders.path_not_found( + request, response, b'404 Not Found') else: response.status = falcon.HTTP_200 resource = self._make_collection(request) @@ -190,7 +191,7 @@ class UserAddresses(_AddressBase): Add a new address to the user record. """ if self._user is None: - path_not_found(request, response) + falcon.responders.path_not_found(request, response) return user_manager = getUtility(IUserManager) validator = Validator(email=unicode, @@ -199,11 +200,13 @@ class UserAddresses(_AddressBase): try: address = user_manager.create_address(**validator(request)) except ValueError as error: - return http.bad_request([], str(error)) + falcon.responders.bad_request(request, response, body=str(error)) except InvalidEmailAddressError: - return http.bad_request([], b'Invalid email address') + falcon.responders.bad_request( + request, response, body=b'Invalid email address') except ExistingAddressError: - return http.bad_request([], b'Address already exists') + falcon.responders.bad_request( + request, response, body=b'Address already exists') else: # Link the address to the current user and return it. address.user = self._user |
