add /v2/{path} redirect

kernelboard/__init__.py

@@ -115,6 +115,13 @@ def unauthorized():
         api = create_api_blueprint()
         app.register_blueprint(api)
 
+    # Redirect /v2/* routes to /* for SEO (v2 was removed but Google still indexes it)
+    @app.route("/v2/")
+    @app.route("/v2/<path:path>")
+    def redirect_v2(path=""):
+
+        return redirect(f"/{path}", code=301)
+
     @app.errorhandler(401)
     def unauthorized(_error):
         return redirect("/401")

kernelboard/api/auth.py

@@ -188,7 +188,7 @@ def callback(provider: str):
     stores display fields in session, logs the user in, and redirects to the SPA.
     """
     if not current_user.is_anonymous:
-        return redirect("/v2/")
+        return redirect("/")
 
 
     provider_data = app.config["OAUTH2_PROVIDERS"].get(provider)

tests/api/test_auth_api.py

@@ -35,7 +35,7 @@ def test_callback_already_logged_in(client):
 
     response = client.get("/api/auth/discord/callback")
     assert response.status_code == 302
-    assert response.headers.get("Location") == "/v2/"
+    assert response.headers.get("Location") == "/"
 
 
 def test_callback_unknown_provider(client):
@@ -153,7 +153,7 @@ def test_callback_happy_path(client):
                 "api/auth/discord/callback?state=123&code=456"
             )
             assert response.status_code == 302
-            assert response.headers.get("Location") == "/v2/"
+            assert response.headers.get("Location") == "/"
 
 
 def test_logout(client):