mosaicstack/stack
1
0
Fork 0
Code Issues 13 Pull Requests Actions Packages Projects Releases 11 Wiki Activity

fix(web): admin page role check — stop false redirect to /chat #203

Merged
jason.woltje merged 1 commits from fix/admin-redirect into main 2026-03-17 02:38:26 +00:00
Conversation 0 Commits 1 Files Changed 2 +59 -2
jason.woltje commented 2026-03-17 02:38:19 +00:00
Owner
Copy Link

Admin page was redirecting all users to /chat because the role check was failing. Fixes #196.

Admin page was redirecting all users to /chat because the role check was failing. Fixes #196.
jason.woltje added 1 commit 2026-03-17 02:38:20 +00:00
fix(web): admin page role check — stop false redirect to /chat
All checks were successful
ci/woodpecker/push/ci Pipeline was successful
Details
ci/woodpecker/pr/ci Pipeline was successful
Details
df4b7399ad 
The admin page role check was failing because better-auth v1.5.5 doesn't
automatically include additionalFields (like 'role') in the session response.
This caused the admin guard to treat all users as non-admin and redirect them.

The fix implements a defensive fallback in the AdminGuard that fetches the
role from the database if it's missing from the session, ensuring that admin
users can access the admin panel while protecting against regression.

Fixes #196
jason.woltje merged commit bf668e18f1 into main 2026-03-17 02:38:26 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jason.woltje
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#203
Delete Branch "fix/admin-redirect"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?