added basic logout functionality

Author: Khoi D. Nguyen

app/routers/views.py

@@ -91,16 +91,10 @@ async def logout(request: Request, response: Response):
     I usually open a new tab/window to reset basic auth. We likely
     need a logout button to be handled somehow in javascript.
     """
-    response.delete_cookie("basic")
-    response.delete_cookie("bearer")
-    response.delete_cookie("access_token")
-    data = helpers.get_page("index.md")
-    return templates.TemplateResponse(
-        "index.html",
-        {
-            "request": request,
-            "data": data,
-        },
+    raise HTTPException(
+        status_code=401,
+        detail="Logged out successfully",
+        headers={"WWW-Authenticate": "Basic"},
     )
 
 

templates/include/topnav.html

@@ -1,25 +1,38 @@
 <nav class="navbar navbar-expand-lg navbar-light bg-light">
-    <div class="container-fluid">
-        <button type="button" id="sidebarCollapse" class="btn btn-info">
-            <i class="fas fa-align-left"></i>
-        </button>
-        {% if title %}<h3 style="margin-left:20px">{{ title }}</h3>{% endif %}
+  <div class="container-fluid">
+    <button type="button" id="sidebarCollapse" class="btn btn-info">
+      <i class="fas fa-align-left"></i>
+    </button>
+    {% if title %}
+    <h3 style="margin-left: 20px">{{ title }}</h3>
+    {% endif %}
 
-        <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarSupportedContent" aria-controls="navbarSupportedContent" aria-expanded="false" aria-label="Toggle navigation">
-            <span class="navbar-toggler-icon"></span>
-        </button>
-        <div class="collapse navbar-collapse" id="navbarSupportedContent">
-            <ul class="navbar-nav me-auto mb-2 mb-lg-0">
-                <li class="nav-item {{'active' if active_page == 'submit_job' }}">
-                    <a class="nav-link" href="/jobs/submit">Submit</a>
-                </li>
-                <li class="nav-item {{'active' if active_page == 'jobs' }}">
-                    <a class="nav-link" href="/jobs">Jobs</a>
-                </li>
-                <li class="nav-item {{'active' if active_page == 'api' }}">
-                    <a class="nav-link" href="/docs">API</a>
-                </li>
-            </ul>
-        </div>
+    <button
+      class="navbar-toggler"
+      type="button"
+      data-bs-toggle="collapse"
+      data-bs-target="#navbarSupportedContent"
+      aria-controls="navbarSupportedContent"
+      aria-expanded="false"
+      aria-label="Toggle navigation"
+    >
+      <span class="navbar-toggler-icon"></span>
+    </button>
+    <div class="collapse navbar-collapse" id="navbarSupportedContent">
+      <ul class="navbar-nav me-auto mb-2 mb-lg-0">
+        <li class="nav-item {{'active' if active_page == 'submit_job' }}">
+          <a class="nav-link" href="/jobs/submit">Submit</a>
+        </li>
+        <li class="nav-item {{'active' if active_page == 'jobs' }}">
+          <a class="nav-link" href="/jobs">Jobs</a>
+        </li>
+        <li>
+          <a class="nav-link" href="/logout">Logout</a>
+        </li>
+        <li class="nav-item {{'active' if active_page == 'api' }}">
+          <a class="nav-link" href="/docs">API</a>
+        </li>
+      </ul>
     </div>
+  </div>
 </nav>