From b9d2e62d9253af5bdafeef77f242682c0a951184 Mon Sep 17 00:00:00 2001 From: Bryan Newbold Date: Wed, 13 Oct 2021 16:46:23 -0700 Subject: web: editor username /u/ helper --- CHANGELOG.md | 1 + python/fatcat_web/routes.py | 8 ++++++++ python/tests/web_editor.py | 8 ++++++++ 3 files changed, 17 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index c9ef0b22..c8636cca 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -35,6 +35,7 @@ command line tool to upgrade old dumps. still actively publishing, discontinued, or "stub" - above API schema changes added to elasticsearch 'release' and 'container' schemas - editor "lookup" API endpoint, for fetching editor object by username +- web `/u/{username}` redirect helper for inspecting editors, bots, etc ### Fixed diff --git a/python/fatcat_web/routes.py b/python/fatcat_web/routes.py index 41027710..e9741b6d 100644 --- a/python/fatcat_web/routes.py +++ b/python/fatcat_web/routes.py @@ -606,6 +606,14 @@ def editor_annotations(ident): return render_template('editor_annotations.html', editor=editor, annotations=annotations) +@app.route('/u/', methods=['GET', 'HEAD']) +def editor_username_redirect(username): + try: + editor = api.lookup_editor(username=username) + except ApiException as ae: + abort(ae.status) + return redirect(f'/editor/{editor.editor_id}') + @app.route('/changelog', methods=['GET']) def changelog_view(): try: diff --git a/python/tests/web_editor.py b/python/tests/web_editor.py index 58b21ddf..0d0679bb 100644 --- a/python/tests/web_editor.py +++ b/python/tests/web_editor.py @@ -22,3 +22,11 @@ def test_change_username(app_admin): assert rv.status_code == 200 rv = app_admin.get('/auth/account') assert b'admin-tmp' not in rv.data + +def test_username_redirect(app_admin): + + rv = app_admin.get('/u/admin') + assert rv.status_code == 302 + + rv = app_admin.get('/u/bogus-not-registered') + assert rv.status_code == 404 -- cgit v1.2.3