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

add / to user-redirect

Browse Source 
Some proxies may not correctly route /user/name, only /user/name/...
This commit is contained in:
Min RK
2017-08-07 15:23:58 +02:00
parent 6ba3090cd5
commit 546268809f
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
jupyterhub/handlers/base.py
View File

@@ -631,6 +631,8 @@ class UserSpawnHandler(BaseHandler):
@gen.coroutine @gen.coroutine
def get(self, name, user_path): def get(self, name, user_path):
if not user_path:
user_path = '/'
current_user = self.get_current_user() current_user = self.get_current_user()
if current_user and current_user.name == name: if current_user and current_user.name == name:

2
jupyterhub/tests/test_pages.py
View File

@@ -218,7 +218,7 @@ def test_user_redirect_deprecated(app):
r.raise_for_status() r.raise_for_status()
print(urlparse(r.url)) print(urlparse(r.url))
path = urlparse(r.url).path path = urlparse(r.url).path
assert path == ujoin(app.base_url, '/user/%s' % name) assert path == ujoin(app.base_url, '/user/%s/' % name)
r = yield get_page('/user/baduser/test.ipynb', app, cookies=cookies, hub=False) r = yield get_page('/user/baduser/test.ipynb', app, cookies=cookies, hub=False)
r.raise_for_status() r.raise_for_status()