Skip to content
Snippets Groups Projects
Commit 9772fcb6 authored by Harsha Kethineni's avatar Harsha Kethineni
Browse files

corrected error messages

parent de6b6ad6
No related branches found
No related tags found
No related merge requests found
......@@ -9,7 +9,7 @@ def default(handler, user=None):
pass
elif method == 'PUT' and handler.uid == _id:
if 'root' in payload and payload['root'] != user['root']:
handler.abort(400, 'user cannot alter own superuser privilege')
handler.abort(400, 'user cannot alter own admin privilege')
elif 'disabled' in payload and payload['disabled'] != user.get('disabled'):
handler.abort(400, 'user cannot alter own disabled status')
else:
......@@ -17,7 +17,7 @@ def default(handler, user=None):
elif method == 'PUT' and handler.user_is_admin:
pass
elif method == 'POST' and not handler.superuser_request and not handler.user_is_admin:
handler.abort(403, 'only superuser are allowed to create users')
handler.abort(403, 'only admins are allowed to create users')
elif method == 'POST' and (handler.superuser_request or handler.user_is_admin):
pass
elif method == 'GET':
......
......@@ -103,7 +103,7 @@ class GearHandler(base.RequestHandler):
def post(self, _id):
"""Upsert an entire gear document."""
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
doc = self.request.json
......@@ -126,7 +126,7 @@ class GearHandler(base.RequestHandler):
def delete(self, _id):
"""Delete a gear. Generally not recommended."""
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
return remove_gear(_id)
......@@ -227,7 +227,7 @@ class JobsHandler(base.RequestHandler):
def get(self):
"""List all jobs."""
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
return list(config.db.jobs.find())
def add(self):
......@@ -286,14 +286,14 @@ class JobsHandler(base.RequestHandler):
def stats(self):
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
return Queue.get_statistics()
def next(self):
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
tags = self.request.GET.getall('tags')
......@@ -309,7 +309,7 @@ class JobsHandler(base.RequestHandler):
def reap_stale(self):
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
count = Queue.scan_for_orphans()
return { 'orphaned': count }
......@@ -321,7 +321,7 @@ class JobHandler(base.RequestHandler):
def get(self, _id):
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
return Job.get(_id)
......@@ -406,7 +406,7 @@ class JobHandler(base.RequestHandler):
def add_logs(self, _id):
"""Add to a job's logs"""
if not self.superuser_request and not self.user_is_admin:
self.abort(403, 'Request requires superuser')
self.abort(403, 'Request requires admin')
doc = self.request.json
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment