simplify handler patterns

PATH pattern will now sometimes include leading slash,
but that's okay because nbmanager will strip them.

fewer redirect patterns needed
MinRK 13 years ago
parent a31cfdc8a8
commit dc65b33d14

@ -74,7 +74,7 @@ class NotebookRedirectHandler(IPythonHandler):
default_handlers = [
(r"/notebooks/?%s" % _notebook_path_regex, NotebookHandler),
(r"/notebooks/?%s" % _path_regex, NotebookRedirectHandler),
(r"/notebooks%s" % _notebook_path_regex, NotebookHandler),
(r"/notebooks%s" % _path_regex, NotebookRedirectHandler),
]

@ -229,18 +229,18 @@ class ModifyNotebookCheckpointsHandler(IPythonHandler):
#-----------------------------------------------------------------------------
_path_regex = r"(?P<path>.*)"
_path_regex = r"(?P<path>(?:/.*)*)"
_checkpoint_id_regex = r"(?P<checkpoint_id>[\w-]+)"
_notebook_name_regex = r"(?P<name>[^/]+\.ipynb)"
_notebook_path_regex = "%s/%s" % (_path_regex, _notebook_name_regex)
default_handlers = [
(r"/api/notebooks/?%s/copy" % _notebook_path_regex, NotebookCopyHandler),
(r"/api/notebooks/?%s/checkpoints" % _notebook_path_regex, NotebookCheckpointsHandler),
(r"/api/notebooks/?%s/checkpoints/%s" % (_notebook_path_regex, _checkpoint_id_regex),
(r"/api/notebooks%s/copy" % _notebook_path_regex, NotebookCopyHandler),
(r"/api/notebooks%s/checkpoints" % _notebook_path_regex, NotebookCheckpointsHandler),
(r"/api/notebooks%s/checkpoints/%s" % (_notebook_path_regex, _checkpoint_id_regex),
ModifyNotebookCheckpointsHandler),
(r"/api/notebooks/?%s" % _notebook_path_regex, NotebookHandler),
(r"/api/notebooks/?%s/?" % _path_regex, NotebookHandler),
(r"/api/notebooks%s" % _notebook_path_regex, NotebookHandler),
(r"/api/notebooks%s" % _path_regex, NotebookHandler),
]

@ -64,10 +64,8 @@ class TreeRedirectHandler(IPythonHandler):
default_handlers = [
(r"/tree/(.*)/", TreeRedirectHandler),
(r"/tree/?%s" % _notebook_path_regex, TreeHandler),
(r"/tree/?%s" % _path_regex, TreeHandler),
(r"/tree/", TreeRedirectHandler),
(r"/tree%s" % _notebook_path_regex, TreeHandler),
(r"/tree%s" % _path_regex, TreeHandler),
(r"/tree", TreeHandler),
(r"/", TreeRedirectHandler),
]

Loading…
Cancel
Save