fix(auth): store tokens before verifying ID token (#168) #170

Merged

admin-mrrm merged 3 commits from 168-fix-data-loading into main

2026-05-03 00:13:30 +02:00

admin-mrrm commented

2026-05-02 23:41:42 +02:00

Owner

Problem

Nach dem Login konnten keine Daten geladen werden. Root cause: In handleCallback wurde verifyIdToken vor setTokens aufgerufen. Wenn die Verifikation geworfen hat (z.B. crypto.subtle nicht verfügbar in Hermes), wurden die Tokens nie gespeichert → alle API-Anfragen schlugen mit 401 fehl.

Fix

setTokens und clearPendingLogin werden jetzt vor verifyIdToken aufgerufen. Tokens sind damit immer gespeichert, sobald der Code-Exchange erfolgreich war — unabhängig davon, ob die ID-Token-Verifikation gelingt.

Closes #168

## Problem Nach dem Login konnten keine Daten geladen werden. Root cause: In `handleCallback` wurde `verifyIdToken` **vor** `setTokens` aufgerufen. Wenn die Verifikation geworfen hat (z.B. `crypto.subtle` nicht verfügbar in Hermes), wurden die Tokens nie gespeichert → alle API-Anfragen schlugen mit 401 fehl. ## Fix `setTokens` und `clearPendingLogin` werden jetzt **vor** `verifyIdToken` aufgerufen. Tokens sind damit immer gespeichert, sobald der Code-Exchange erfolgreich war — unabhängig davon, ob die ID-Token-Verifikation gelingt. Closes #168

admin-mrrm added 1 commit

2026-05-02 23:41:42 +02:00

fix(auth): store tokens before verifying ID token to prevent login data-loading failure

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

07db19d9da

Previously verifyIdToken was called before setTokens, so if verification
threw (e.g. crypto.subtle unavailable in Hermes) the access token was
never persisted and all subsequent API calls returned 401.

Fixes #168.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

admin-mrrm added 1 commit

2026-05-02 23:46:52 +02:00

Merge branch 'main' into 168-fix-data-loading

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

99e5064c92

admin-mrrm added 1 commit

2026-05-02 23:53:46 +02:00

Merge branch 'main' into 168-fix-data-loading

continuous-integration/drone/push Build is passing