diff options
| author | ivarlovlie <git@ivarlovlie.no> | 2022-09-29 12:05:48 +0200 |
|---|---|---|
| committer | ivarlovlie <git@ivarlovlie.no> | 2022-09-29 12:06:18 +0200 |
| commit | a893be89c5e2863ae44af3fa2c2639604ced1278 (patch) | |
| tree | e54f870dc0b4f48884e12e2120cd632602670ccf /apps/kit/src/routes/(main)/+layout.svelte | |
| parent | 7d69290660cf78e96b34483ea066f015af7fe1d9 (diff) | |
| download | greatoffice-a893be89c5e2863ae44af3fa2c2639604ced1278.tar.xz greatoffice-a893be89c5e2863ae44af3fa2c2639604ced1278.zip | |
refactor: Check session and redirect server side
Diffstat (limited to 'apps/kit/src/routes/(main)/+layout.svelte')
| -rw-r--r-- | apps/kit/src/routes/(main)/+layout.svelte | 29 |
1 files changed, 3 insertions, 26 deletions
diff --git a/apps/kit/src/routes/(main)/+layout.svelte b/apps/kit/src/routes/(main)/+layout.svelte index ec50c4b..3107861 100644 --- a/apps/kit/src/routes/(main)/+layout.svelte +++ b/apps/kit/src/routes/(main)/+layout.svelte @@ -1,38 +1,15 @@ <script lang="ts"> import "../../app.pcss"; - import { afterNavigate, beforeNavigate, goto } from "$app/navigation"; - import { is_active } from "$lib/session"; - import type { Navigation } from "@sveltejs/kit"; import { setLocale } from "$lib/i18n/i18n-svelte"; import { onMount } from "svelte"; import type { LayoutData } from "./$types"; import LocaleSwitcher from "$lib/components/locale-switcher.svelte"; export let data: LayoutData; - onMount(() => setLocale(data.locale)); - async function redirect_if_necessary(ticket: Navigation) { - const sessionIsValid = await is_active(); - // TODO: ticket.to can be empty while navigating, so coalesce could probably cause non-public routes to cause a redir to /login... - const isPublicRoute = ticket.to?.routeId?.startsWith("(main)/(public)"); - - console.log("redir: ", { - isPublicRoute, - sessionIsValid, - }); - - if (sessionIsValid && isPublicRoute) { - await goto("/home"); - } else if (!sessionIsValid && !isPublicRoute) { - await goto("/sign-in"); - } - } - - // This should probably be removed in favor of the logic in layout.server.ts. - // That requires a more sophisticated server side implementation of session handling, - // and i don't want that tbh, i want to stay as much in the browser as possible. - afterNavigate(redirect_if_necessary); - beforeNavigate(redirect_if_necessary); + onMount(async () => { + setLocale(data.locale); + }); </script> <LocaleSwitcher /> |
