f091_PIN-verstecken-in-URL #36

Merged
mina merged 5 commits from f091_PIN-verstecken-in-URL into development 2025-10-13 10:44:44 +02:00
Owner

vorgangPIN wurde aus URLs entfernt
PIN wird nun in Cookies gesetzt und ausgelesen
inklusive Tests

vorgangPIN wurde aus URLs entfernt PIN wird nun in Cookies gesetzt und ausgelesen inklusive Tests
trachi93 added 3 commits 2025-10-06 09:44:38 +02:00
mina reviewed 2025-10-06 16:46:52 +02:00
mina left a comment
Owner

in der vite.config.ts ergänzen: $root: path.resolve(__dirname, './src')

in der vite.config.ts ergänzen: $root: path.resolve(__dirname, './src')
mina requested changes 2025-10-06 17:07:04 +02:00
@@ -4,3 +1,3 @@
} from '$lib/server/vorgangService';
import { vorgangPINValidation, vorgangExists } from '$lib/server/vorgangService';
import { redirect } from '@sveltejs/kit';
import type { PageServerLoad } from './list/[vorgang]/$types';
Owner

da layout.server müsste es LayoutServerLoad sein

da layout.server müsste es LayoutServerLoad sein
trachi93 marked this conversation as resolved
@@ -15,3 +12,3 @@
const vorgangToken = params.vorgang;
const vorgangPIN = url.searchParams.get('pin');
const COOKIE_NAME = `token-${vorgangToken}`;
Owner

Auf Typen achten, vorgangPIn/vorgangToken darf nicht undefined sein, ist aber möglich

Auf Typen achten, vorgangPIn/vorgangToken darf nicht undefined sein, ist aber möglich
trachi93 marked this conversation as resolved
@@ -0,0 +1,143 @@
import { describe, it, expect, vi } from 'vitest';
import { actions } from '$root/routes/anmeldung/+page.server';
Owner

in der vite.config.ts: $root: path.resolve(__dirname, './src')

in der vite.config.ts: $root: path.resolve(__dirname, './src')
trachi93 marked this conversation as resolved
@@ -0,0 +34,4 @@
}
};
let thrownRedirect;
Owner

Typ richtig setzen! Scheint ein Redirect | undefined zu sein.

Typ richtig setzen! Scheint ein Redirect | undefined zu sein.
trachi93 marked this conversation as resolved
trachi93 added 1 commit 2025-10-09 13:05:13 +02:00
mina added 1 commit 2025-10-13 10:44:23 +02:00
mina merged commit 0ec9692db3 into development 2025-10-13 10:44:44 +02:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: innohub/tatort#36