mrhid6andClaude Opus 5
7a8e683d99
Route-group layouts do the guarding. A customer session on /staff/* is redirected to its own home rather than shown a refusal -- there is nothing to tell them about. This is UX only: admin enforces the same boundary with RequireStaff/RequireCustomer and answers 404 rather than 403 for another account's data, which is the layer that actually matters. Signup carries the honeypot the backend expects and reports "check your email" rather than claiming an account exists, matching a backend that creates nothing until the link is opened. Buttons match site/'s .btn--solid and .btn--line, neutral border included. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
48 lines
1.5 KiB
TypeScript
48 lines
1.5 KiB
TypeScript
"use client";
|
|
|
|
import { useQuery } from "@tanstack/react-query";
|
|
import { useRouter } from "next/navigation";
|
|
import { useEffect } from "react";
|
|
import { API_BASE, ApiError, NotConnected, api, type Session } from "./api";
|
|
import { NotConnectedPanel } from "@/components/NotConnected";
|
|
|
|
export function useSession() {
|
|
const { data, error, isLoading } = useQuery<Session>({
|
|
queryKey: ["me"],
|
|
queryFn: api.me,
|
|
staleTime: 60_000,
|
|
});
|
|
return { session: data, error, isLoading };
|
|
}
|
|
|
|
/*
|
|
* The route-group guard. This is UX, not security: admin enforces the same
|
|
* boundary with RequireStaff/RequireCustomer and returns 404 rather than 403
|
|
* for another account's data. A customer hitting a staff route is redirected
|
|
* rather than shown a refusal, because there is nothing to tell them about.
|
|
*/
|
|
export function RequireKind({
|
|
kind,
|
|
children,
|
|
}: {
|
|
kind: Session["kind"];
|
|
children: React.ReactNode;
|
|
}) {
|
|
const router = useRouter();
|
|
const { session, error, isLoading } = useSession();
|
|
|
|
useEffect(() => {
|
|
if (error instanceof ApiError && error.status === 401) {
|
|
router.replace("/login");
|
|
return;
|
|
}
|
|
if (session && session.kind !== kind) {
|
|
router.replace(session.kind === "staff" ? "/staff" : "/");
|
|
}
|
|
}, [error, session, kind, router]);
|
|
|
|
if (error instanceof NotConnected) return <NotConnectedPanel url={API_BASE} />;
|
|
if (isLoading || !session || session.kind !== kind) return null;
|
|
return <>{children}</>;
|
|
}
|