simplify handler setup with default_handlers in modules

like IPython's, but a bit simpler since we don't have
so many services to deal with.

jupyterhub/apihandlers/__init__.py

@@ -1 +1,7 @@
+from . import auth, users
 from .auth import *
+from .users import *
+
+default_handlers = []
+for mod in (auth, users):
+    default_handlers.extend(mod.default_handlers)

jupyterhub/apihandlers/auth.py

@@ -20,3 +20,7 @@ class AuthorizationsAPIHandler(BaseHandler):
         self.write(json.dumps({
             'user' : orm_token.user.name,
         }))
+
+default_handlers = [
+    (r"/api/authorizations/([^/]+)", AuthorizationsAPIHandler),
+]

jupyterhub/app.py

@@ -14,29 +14,19 @@ from jinja2 import Environment, FileSystemLoader
 import tornado.httpserver
 import tornado.options
 from tornado.ioloop import IOLoop
-from tornado.log import LogFormatter, app_log
+from tornado.log import LogFormatter
 from tornado import gen, web
 
 from IPython.utils.traitlets import (
-    Unicode, Integer, Dict, TraitError, List, Instance, Bool, Bytes, Any,
-    DottedObjectName,
+    Unicode, Integer, Dict, TraitError, List, Bool, Bytes, Any,
+    DottedObjectName, Set,
 )
 from IPython.config import Application, catch_config_error
 from IPython.utils.importstring import import_item
 
 here = os.path.dirname(__file__)
 
-from .handlers import (
-    Template404,
-    RootHandler,
-    LoginHandler,
-    LogoutHandler,
-    UserHandler,
-)
-
-from .apihandlers import (
-    AuthorizationsAPIHandler,
-)
+from . import handlers, apihandlers
 
 from . import orm
 from ._data import DATA_FILES_PATH
@@ -220,19 +210,19 @@ class JupyterHubApp(Application):
         return handlers
     
     def init_handlers(self):
-        handlers = [
-            (r"/", RootHandler),
-            (r"/login", LoginHandler),
-            (r"/logout", LogoutHandler),
-            (r"/api/authorizations/([^/]+)", AuthorizationsAPIHandler),
-        ]
-        self.handlers = self.add_url_prefix(self.hub_prefix, handlers)
+        h = []
+        h.extend(handlers.default_handlers)
+        h.extend(apihandlers.default_handlers)
+
+        self.handlers = self.add_url_prefix(self.hub_prefix, h)
+
+        # some extra handlers, outside hub_prefix
         self.handlers.extend([
-            (r"/user/([^/]+)/?.*", UserHandler),
+            (r"/user/([^/]+)/?.*", handlers.UserHandler),
             (r"/?", web.RedirectHandler, {"url" : self.hub_prefix, "permanent": False}),
         ])
         self.handlers.append(
-            (r'(.*)', Template404)
+            (r'(.*)', handlers.Template404)
         )
     
     def init_db(self):

jupyterhub/handlers/__init__.py

@@ -1,2 +1,8 @@
 from .base import *
 from .login import *
+
+from . import base, login
+
+default_handlers = []
+for mod in (base, login):
+    default_handlers.extend(mod.default_handlers)

jupyterhub/handlers/base.py

@@ -294,3 +294,7 @@ class UserHandler(BaseHandler):
             self.redirect(url_concat(self.settings['login_url'], {
                 'next' : self.request.path,
             }))
+
+default_handlers = [
+    (r"/", RootHandler),
+]

jupyterhub/handlers/login.py

@@ -56,3 +56,8 @@ class LoginHandler(BaseHandler):
                 username=username,
             )
             self.finish(html)
+
+default_handlers = [
+    (r"/login", LoginHandler),
+    (r"/logout", LogoutHandler),
+]