fix: implement logging out

blue-build · Jul 13, 2024 · dd73079 · dd73079
1 parent 0962713
commit dd73079
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/routes/+layout.svelte b/src/routes/+layout.svelte
@@ -60,9 +60,7 @@
                                 </a>
                             </DropdownMenu.Item>
                             <DropdownMenu.Item>
-                                <button on:click={() => alert("unimplemented, just clear cookies")}>
-                                    Log out
-                                </button>
+                                <a href="/api/login/logout">Log out</a>
                             </DropdownMenu.Item>
                         </DropdownMenu.Group>
                     </DropdownMenu.Content>

diff --git a/src/routes/api/login/logout/+server.ts b/src/routes/api/login/logout/+server.ts
@@ -0,0 +1,11 @@
+// routes/login/github/+server.ts
+import { redirect } from "@sveltejs/kit";
+
+import type { RequestEvent } from "@sveltejs/kit";
+
+export async function GET(event: RequestEvent): Promise<Response> {
+    await event.cookies.delete("github_oauth_state", { path: "/" });
+    await event.cookies.delete("github_oauth_token", { path: "/" });
+
+    redirect(302, "/");
+}