change(fastapi): remove the GET /logout route; replaced with POST

Had to add some additional CSS in to style a form button the same
as <a> links are styled.

Closes #188

Signed-off-by: Kevin Morris <kevr@0cost.org>

aurweb/routers/auth.py

@@ -77,14 +77,9 @@ async def login_post(request: Request,
     return response
 
 
-@router.get("/logout")
+@router.post("/logout")
 @auth_required()
-async def logout(request: Request, next: str = "/"):
+async def logout(request: Request, next: str = Form(default="/")):
-    """ A GET and POST route for logging out.
-
-    @param request FastAPI request
-    @param next Route to redirect to
-    """
     if request.user.is_authenticated():
         request.user.logout(request)
 
@@ -95,9 +90,3 @@ async def logout(request: Request, next: str = "/"):
     response.delete_cookie("AURSID")
     response.delete_cookie("AURTZ")
     return response
-
-
-@router.post("/logout")
-@auth_required()
-async def logout_post(request: Request, next: str = "/"):
-    return await logout(request=request, next=next)

templates/partials/archdev-navbar.html

@@ -45,9 +45,12 @@
 
             {# All logged in users see Logout #}
             <li>
-                <a href="/logout?next={{ next }}">
+                <form action="/logout" method="post" class="link">
-                    {% trans %}Logout{% endtrans %}
+                    <input type="hidden" name="next" value="{{ next }}" />
-                </a>
+                    <button type="submit">
+                        {% trans %}Logout{% endtrans %}
+                    </button>
+                </form>
             </li>
         {% else %}
             {# All guest users see Register #}

test/test_auth_routes.py

@@ -154,8 +154,9 @@ def test_unauthenticated_logout_unauthorized():
     with client as request:
         # Alright, let's verify that attempting to /logout when not
         # authenticated returns 401 Unauthorized.
-        response = request.get("/logout", allow_redirects=False)
+        response = request.post("/logout", allow_redirects=False)
         assert response.status_code == int(HTTPStatus.SEE_OTHER)
+        assert response.headers.get("location").startswith("/login")
 
 
 def test_login_missing_username():

web/html/css/aurweb.css

@@ -229,3 +229,18 @@ input#search-action-submit {
 .success {
     color: green;
 }
+
+/* Styling used to clone <a> styles for a form.link button. */
+form.link, form.link > button {
+    display: inline-block;
+}
+form.link > button {
+    padding: 0 0.5em;
+    color: #07b;
+    background: none;
+    border: none;
+}
+form.link > button:hover {
+    cursor: pointer;
+    text-decoration: underline;
+}