diff options
author | Prashanth Pai <ppai@redhat.com> | 2013-11-22 12:13:09 +0530 |
---|---|---|
committer | Luis Pabon <lpabon@redhat.com> | 2013-12-05 09:15:40 -0800 |
commit | fc9124caf45949dfcc0732536c6825c12d74582a (patch) | |
tree | ad9871cda75a8c5f08e4a06d88a38836b674c8c5 /test/functional_auth/gswauth/test_gswauth.py | |
parent | 0eb79aad3658ca519143029f219c9efe3591e724 (diff) |
gswauth: Fix 403 being returned instead of 401
- 401(Unauthorized) is to be returned when user credentials are
wrong where as 403(Forbidden) is to be returned when user
credentials are correct but the user doesn't have the priveleges
to carry out the operation.
- Also error messages displayed when using swauth-* command line
utilities have been updated.
Change-Id: I485786896ad14d3263f4325d1857cacc93adab96
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Reviewed-on: http://review.gluster.org/6336
Reviewed-by: Luis Pabon <lpabon@redhat.com>
Tested-by: Luis Pabon <lpabon@redhat.com>
Diffstat (limited to 'test/functional_auth/gswauth/test_gswauth.py')
-rw-r--r-- | test/functional_auth/gswauth/test_gswauth.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/functional_auth/gswauth/test_gswauth.py b/test/functional_auth/gswauth/test_gswauth.py index 30ecfeb..3ee3f5d 100644 --- a/test/functional_auth/gswauth/test_gswauth.py +++ b/test/functional_auth/gswauth/test_gswauth.py @@ -159,7 +159,7 @@ class TestGSWauth(unittest.TestCase): conn = http_connect(config['auth_host'], config['auth_port'], 'PUT', path, headers) resp = conn.getresponse() - self.assertTrue(resp.status == 403) + self.assertTrue(resp.status == 401) def test_change_user_password(self): # check and register account @@ -235,7 +235,7 @@ class TestGSWauth(unittest.TestCase): conn = http_connect(config['auth_host'], config['auth_port'], 'PUT', path, headers) resp = conn.getresponse() - self.assertTrue(resp.status == 403) + self.assertTrue(resp.status == 401) finally: try: |