hazza/jupyterhub
1
0
Fork 0
mirror of https://github.com/jupyterhub/jupyterhub.git synced 2025-10-16 06:22:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1303 from minrk/message-202

Browse Source 
consolidate server APIHandlers
This commit is contained in:
Carol Willing
2017-08-04 07:43:40 -07:00
committed by GitHub
parent e666261434 57c8ad6b92
commit 0b34e13dd4
1 changed files with 26 additions and 59 deletions
Download Patch File Download Diff File Expand all files Collapse all files

71
jupyterhub/apihandlers/users.py
View File

@@ -177,75 +177,41 @@ class UserAPIHandler(APIHandler):
class UserServerAPIHandler(APIHandler):
"""Create and delete single-user servers
This handler should be used when c.JupyterHub.allow_named_servers = False
"""
"""Start and stop single-user servers"""
@gen.coroutine
@admin_or_self
def post(self, name):
def post(self, name, server_name=''):
user = self.find_user(name)
if user.running:
# include notify, so that a server that died is noticed immediately
state = yield user.spawner.poll_and_notify()
if state is None:
raise web.HTTPError(400, "%s's server is already running" % name)
options = self.get_json_body()
yield self.spawn_single_user(user, options=options)
status = 202 if user.spawner._spawn_pending else 201
self.set_status(status)
@gen.coroutine
@admin_or_self
def delete(self, name):
user = self.find_user(name)
if user.spawner._stop_pending:
self.set_status(202)
return
if not user.running:
raise web.HTTPError(400, "%s's server is not running" % name)
# include notify, so that a server that died is noticed immediately
status = yield user.spawner.poll_and_notify()
if status is not None:
raise web.HTTPError(400, "%s's server is not running" % name)
yield self.stop_single_user(user)
status = 202 if user.spawner._stop_pending else 204
self.set_status(status)
class UserNamedServerAPIHandler(APIHandler):
"""Manage named single-user servers
This handler should be used when c.JupyterHub.allow_named_servers = True
"""
@gen.coroutine
@admin_or_self
def post(self, name, server_name):
if server_name:
if not self.allow_named_servers:
raise web.HTTPError(400, "Named servers are not enabled.")
user = self.find_user(name)
if user is None:
raise web.HTTPError(404, "No such user '%s'" % name)
spawner = user.spawners[server_name]
if spawner.ready:
# include notify, so that a server that died is noticed immediately
state = yield spawner.poll_and_notify()
if state is None:
raise web.HTTPError(400, "%s's server %s is already running" % (name, server_name))
options = self.get_json_body()
yield self.spawn_single_user(user, server_name, options=options)
spawner = user.spawners[server_name]
status = 202 if spawner._spawn_pending else 201
status = 202 if spawner.pending == 'spawn' else 201
self.set_header('Content-Type', 'text/plain')
self.set_status(status)
@gen.coroutine
@admin_or_self
def delete(self, name, server_name):
def delete(self, name, server_name=''):
user = self.find_user(name)
if server_name:
if not self.allow_named_servers:
raise web.HTTPError(400, "Named servers are not enabled.")
user = self.find_user(name)
if user is None:
raise web.HTTPError(404, "No such user '%s'" % name)
if server_name not in user.spawners:
raise web.HTTPError(404, "%s has no server named '%s'" % (name, server_name))
spawner = user.spawners[server_name]
if spawner._stop_pending:
self.set_header('Content-Type', 'text/plain')
self.set_status(202)
return
if not spawner.ready:
@@ -256,6 +222,7 @@ class UserNamedServerAPIHandler(APIHandler):
raise web.HTTPError(400, "%s's server %s is not running" % (name, server_name))
yield self.stop_single_user(user, server_name)
status = 202 if spawner._stop_pending else 204
self.set_header('Content-Type', 'text/plain')
self.set_status(status)
@@ -284,6 +251,6 @@ default_handlers = [
(r"/api/users", UserListAPIHandler),
(r"/api/users/([^/]+)", UserAPIHandler),
(r"/api/users/([^/]+)/server", UserServerAPIHandler),
(r"/api/users/([^/]+)/servers/([^/]*)", UserNamedServerAPIHandler),
(r"/api/users/([^/]+)/servers/([^/]*)", UserServerAPIHandler),
(r"/api/users/([^/]+)/admin-access", UserAdminAccessAPIHandler),
]