diff options
| author | Jenkins <jenkins@review.openstack.org> | 2012-01-25 20:01:41 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2012-01-25 20:01:41 +0000 |
| commit | 55a70f32c1a9bbcf4534b314edbebe82542ed4e1 (patch) | |
| tree | 981aa69978b915afb8f818526012adc018398668 /nova | |
| parent | f7346b6ce8a945499d9f2f3eb0d531f9da0705dc (diff) | |
| parent | 4a04d297bdd276b7bd9fbd1565c9c7dcfc87ee95 (diff) | |
Merge "Return 403 instead of 401 when policies reject"
Diffstat (limited to 'nova')
| -rw-r--r-- | nova/api/openstack/wsgi.py | 2 | ||||
| -rw-r--r-- | nova/tests/api/openstack/test_wsgi.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/nova/api/openstack/wsgi.py b/nova/api/openstack/wsgi.py index 8a7a077b8..26f6cc97f 100644 --- a/nova/api/openstack/wsgi.py +++ b/nova/api/openstack/wsgi.py @@ -573,7 +573,7 @@ class ResourceExceptionHandler(object): if isinstance(ex_value, exception.NotAuthorized): msg = unicode(ex_value) - raise Fault(webob.exc.HTTPUnauthorized(explanation=msg)) + raise Fault(webob.exc.HTTPForbidden(explanation=msg)) elif isinstance(ex_value, TypeError): LOG.exception(ex_value) raise Fault(webob.exc.HTTPBadRequest()) diff --git a/nova/tests/api/openstack/test_wsgi.py b/nova/tests/api/openstack/test_wsgi.py index 3dfcb2388..3974971fa 100644 --- a/nova/tests/api/openstack/test_wsgi.py +++ b/nova/tests/api/openstack/test_wsgi.py @@ -204,7 +204,7 @@ class ResourceTest(test.TestCase): req = webob.Request.blank('/tests') app = fakes.TestRouter(Controller()) response = req.get_response(app) - self.assertEqual(response.status_int, 401) + self.assertEqual(response.status_int, 403) def test_dispatch(self): class Controller(object): |