@snokam/auth 3.7.0 → 3.7.1

package/CHANGELOG.md

@@ -3,6 +3,10 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.7.1](https://github.com/snokam/components/compare/@snokam/auth@3.7.0...@snokam/auth@3.7.1) (2026-10-06)
+
+**Note:** Version bump only for package @snokam/auth
+
 # [3.7.0](https://github.com/snokam/components/compare/@snokam/auth@3.6.0...@snokam/auth@3.7.0) (2026-10-06)
 
 ### Features

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@snokam/auth",
   "description": "Snøkam auth components",
-  "version": "3.7.0",
+  "version": "3.7.1",
   "main": "src/index.ts",
   "types": "src/index.ts",
   "files": [

package/src/app-router/index.tsx

@@ -34,6 +34,7 @@ export {
 } from "../impersonation/context";
 export type { ImpersonationActive } from "../impersonation/cookies";
 export { impersonationRequired } from "../impersonation/required";
+export { isAllowedRedirect } from "../impersonation/redirects";
 export { signIn, SessionProvider } from "next-auth/react";
 export {
   Login,

package/src/app-sign-in/handlers.ts

@@ -79,13 +79,13 @@ async function ownSigner(
   return "error" in access ? null : { email, accessToken: access.accessToken };
 }
 
-function signerFor(
+async function signerFor(
   authConfig: Auth.Config,
   request: Request,
   email: string
 ): Promise<Signer | null> {
   if (impersonationRequired(authConfig.environment)) {
-    return Promise.resolve(personaSigner(request, email));
+    return personaSigner(request, email);
   }
   return ownSigner(authConfig, request, email);
 }

package/src/app-sign-in/platform.ts

@@ -1,36 +1,24 @@
 import { getManagedBearer } from "@snokam/identity";
 import {
-  Configuration,
   PlatformClient,
   ResponseError,
   type AuthorizationPreview,
   type IssuedAuthorizationCode,
 } from "@snokam/platform-client";
+import { platformClientFor } from "../impersonation/request";
 import type { AuthorizationParams } from "./consent";
 
 type Outcome<T> = { value: T } | { error: string };
 
 const UNAVAILABLE = "temporarily_unavailable";
 
-function apiBase(): string {
-  return (process.env.NEXT_PUBLIC_PLATFORM_API_BASE_URL ?? "").replace(
-    /\/+$/,
-    ""
-  );
-}
-
 async function asLoginSite<T>(
   call: (client: PlatformClient) => Promise<T>
 ): Promise<Outcome<T>> {
   const loginSite = await getManagedBearer();
-  if (!loginSite) return { error: UNAVAILABLE };
+  const client = loginSite ? platformClientFor(loginSite) : null;
+  if (!client) return { error: UNAVAILABLE };
 
-  const client = new PlatformClient(
-    new Configuration({
-      basePath: apiBase(),
-      accessToken: async () => loginSite,
-    })
-  );
   try {
     return { value: await call(client) };
   } catch (error) {

package/src/impersonation/context.tsx

@@ -12,21 +12,25 @@ import {
 import { IMP_ACTIVE_COOKIE, type ImpersonationActive } from "./cookies";
 import { clearAuthTokenCache } from "../hooks";
 import { useSessionRenewal } from "./renewal";
+import { isAllowedRedirect, platformBaseDomain } from "./redirects";
+
+export interface StartImpersonationInput {
+  targetEmail: string;
+  targetName?: string;
+  targetImage?: string;
+  reason: string;
+  redirectTo?: string;
+  returnTo?: string;
+  landingUrl?: string;
+}
 
 interface ImpersonationContextValue {
   active: ImpersonationActive | null;
   starting: boolean;
   exiting: boolean;
-  start: (input: {
-    targetEmail: string;
-    targetName?: string;
-    targetImage?: string;
-    reason: string;
-    redirectTo?: string;
-    returnTo?: string;
-    landingUrl?: string;
-    handOffOnly?: boolean;
-  }) => Promise<{ ok: boolean; error?: string }>;
+  start: (
+    input: StartImpersonationInput
+  ) => Promise<{ ok: boolean; error?: string }>;
   exit: () => Promise<void>;
 }
 
@@ -34,27 +38,12 @@ const ImpersonationContext = createContext<ImpersonationContextValue | null>(
   null
 );
 
-/** Registrable base domain of the platform (last two labels of the API host). */
-function platformBaseDomain(): string {
-  const api = process.env.NEXT_PUBLIC_PLATFORM_API_BASE_URL;
-  if (!api) {
-    throw new Error("NEXT_PUBLIC_PLATFORM_API_BASE_URL is not configured");
-  }
-  return new URL(api).hostname.split(".").slice(-2).join(".");
-}
-
-/** Guards the exit redirect against open-redirect: same-origin, or https on the platform base domain. */
 function isSafeReturnTo(returnTo: string): boolean {
   if (typeof window === "undefined") return false;
-  const base = platformBaseDomain();
-  try {
-    const url = new URL(returnTo, window.location.origin);
-    if (url.origin === window.location.origin) return true;
-    if (url.protocol !== "https:") return false;
-    return url.hostname === base || url.hostname.endsWith(`.${base}`);
-  } catch {
-    return false;
-  }
+  return isAllowedRedirect(returnTo, {
+    origin: window.location.origin,
+    platformDomain: platformBaseDomain(),
+  });
 }
 
 function readActiveCookie(): ImpersonationActive | null {
@@ -87,47 +76,35 @@ export function ImpersonationProvider({ children }: { children: ReactNode }) {
 
   useSessionRenewal(active, rereadCookie);
 
-  const start = useCallback(
-    async (input: {
-      targetEmail: string;
-      targetName?: string;
-      targetImage?: string;
-      reason: string;
-      redirectTo?: string;
-      returnTo?: string;
-      landingUrl?: string;
-      handOffOnly?: boolean;
-    }) => {
-      setStarting(true);
-      try {
-        const { redirectTo, landingUrl } = input;
-        const res = await fetch("/api/impersonation/start", {
-          method: "POST",
-          headers: { "Content-Type": "application/json" },
-          credentials: "include",
-          body: JSON.stringify(input),
-        });
-        if (!res.ok) {
-          return {
-            ok: false,
-            error: (await res.text()) || `HTTP ${res.status}`,
-          };
-        }
-        clearAuthTokenCache();
-        const data = (await res.json().catch(() => ({}))) as {
-          adoptUrl?: string;
+  const start = useCallback(async (input: StartImpersonationInput) => {
+    setStarting(true);
+    try {
+      const { redirectTo, landingUrl } = input;
+      const res = await fetch("/api/impersonation/start", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        credentials: "include",
+        body: JSON.stringify(input),
+      });
+      if (!res.ok) {
+        return {
+          ok: false,
+          error: (await res.text()) || `HTTP ${res.status}`,
         };
-        if (data.adoptUrl) window.location.assign(data.adoptUrl);
-        else if (landingUrl) window.location.assign(landingUrl);
-        else if (redirectTo) window.location.assign(redirectTo);
-        else window.location.reload();
-        return { ok: true };
-      } finally {
-        setStarting(false);
       }
-    },
-    []
-  );
+      clearAuthTokenCache();
+      const data = (await res.json().catch(() => ({}))) as {
+        adoptUrl?: string;
+      };
+      if (data.adoptUrl) window.location.assign(data.adoptUrl);
+      else if (landingUrl) window.location.assign(landingUrl);
+      else if (redirectTo) window.location.assign(redirectTo);
+      else window.location.reload();
+      return { ok: true };
+    } finally {
+      setStarting(false);
+    }
+  }, []);
 
   const exit = useCallback(async () => {
     setExiting(true);
@@ -164,7 +141,6 @@ export function ImpersonationProvider({ children }: { children: ReactNode }) {
 export function useImpersonation(): ImpersonationContextValue {
   const context = useContext(ImpersonationContext);
   if (!context) {
-    // Safe no-op default so the shared banner can mount in apps that haven't wired the provider yet.
     return {
       active: null,
       starting: false,

package/src/impersonation/cookies.ts

@@ -1,8 +1,6 @@
 import { resolveCookieDomain } from "../session";
 
-/** HttpOnly — carries the impersonation bearer; only the token handler ever reads it. */
 export const IMP_SESSION_COOKIE = "imp_session";
-/** Readable — non-sensitive session facts that drive the banner + provider (no token). */
 export const IMP_ACTIVE_COOKIE = "imp_active";
 
 export interface ImpersonationSession {
@@ -89,7 +87,6 @@ function serialize(
   return parts.join("; ");
 }
 
-/** Set-Cookie values that open a session (server-side, on mint success). */
 export function buildImpersonationCookies(
   minted: MintedToken,
   host: string | null
@@ -123,7 +120,6 @@ export function buildImpersonationCookies(
   ];
 }
 
-/** Set-Cookie values that clear the session (exit or after posting the end record). */
 export function clearImpersonationCookies(host: string | null): string[] {
   return [
     serialize(IMP_SESSION_COOKIE, "", { host, httpOnly: true, maxAge: 0 }),

package/src/impersonation/handlers.ts

@@ -13,35 +13,35 @@ import {
   platformNotConfigured,
   requestHost,
 } from "./request";
+import { isAllowedRedirect, platformBaseDomain } from "./redirects";
+
+interface StartBody {
+  targetEmail?: string;
+  targetName?: string;
+  targetImage?: string;
+  reason?: string;
+  redirectTo?: string;
+  returnTo?: string;
+  landingUrl?: string;
+}
 
 function registrable(host: string): string {
   return host.split(":")[0].toLowerCase().split(".").slice(-2).join(".");
 }
 
-function platformBaseDomain(): string {
-  const api = process.env.NEXT_PUBLIC_PLATFORM_API_BASE_URL;
-  if (!api) {
-    throw new Error("NEXT_PUBLIC_PLATFORM_API_BASE_URL is not configured");
-  }
-  return new URL(api).hostname.split(".").slice(-2).join(".");
+function resolveLanding(returnTo: string, origin: string): string {
+  const allowed = isAllowedRedirect(returnTo, {
+    origin,
+    platformDomain: platformBaseDomain(),
+  });
+  return allowed
+    ? new URL(returnTo, origin).toString()
+    : new URL("/", origin).toString();
 }
 
-// The adopt sets the tenant-domain cookie, then hands back to a same-origin path OR the platform
-// hub (/manage on snosky.no) — anything else falls back to "/" so it can never open-redirect.
-function resolveLanding(returnTo: string, origin: string): string {
-  const base = platformBaseDomain();
-  try {
-    const url = new URL(returnTo, origin);
-    if (url.origin === origin) return url.toString();
-    if (
-      url.protocol === "https:" &&
-      (url.hostname === base || url.hostname.endsWith(`.${base}`))
-    )
-      return url.toString();
-  } catch {
-    // Fall through to the safe default.
-  }
-  return new URL("/", origin).toString();
+function withCookies(response: Response, cookies: string[]): Response {
+  for (const cookie of cookies) response.headers.append("Set-Cookie", cookie);
+  return response;
 }
 
 function cookieScope(host: string): string {
@@ -62,11 +62,6 @@ function handsOffToTarget(request: Request, target: string): boolean {
   }
 }
 
-/**
- * Starts a view-as-employee session: authenticates as the admin, calls the platform mint endpoint
- * (which enforces the platform.impersonate gate + required reason + audit server-side), and, on
- * success, sets the impersonation cookies so subsequent product calls ride the minted token.
- */
 export function createImpersonationStartHandler(authConfig: Auth.Config) {
   return async function POST(request: Request) {
     const caller = await resolveAccessToken(authConfig, request);
@@ -75,16 +70,7 @@ export function createImpersonationStartHandler(authConfig: Auth.Config) {
     const client = platformClient(caller.accessToken);
     if (!client) return platformNotConfigured();
 
-    let body: {
-      targetEmail?: string;
-      targetName?: string;
-      targetImage?: string;
-      reason?: string;
-      redirectTo?: string;
-      returnTo?: string;
-      landingUrl?: string;
-      handOffOnly?: boolean;
-    };
+    let body: StartBody;
     try {
       body = await request.json();
     } catch {
@@ -118,46 +104,33 @@ export function createImpersonationStartHandler(authConfig: Auth.Config) {
       tokenId: result.tokenId,
       expiresOn: result.expiresOn,
     };
-
-    // Custom-domain tenant: hand the session to the tenant domain's adopt route (login's pattern).
-    if (body.redirectTo && handsOffToTarget(request, body.redirectTo)) {
-      const target = new URL(body.redirectTo);
-      const ticket = sealImpersonationTicket(
-        { ...result, business: target.host, returnTo: body.returnTo },
-        authConfig.nextAuthSecret
-      );
-      const adopt = new URL("/api/impersonation/adopt", target.origin);
-      adopt.searchParams.set("t", ticket);
-      adopt.searchParams.set(
-        "returnTo",
-        body.landingUrl ?? target.pathname + target.search
-      );
-      const crossDomain = Response.json({
-        ...summary,
-        adoptUrl: adopt.toString(),
-      });
-      if (body.handOffOnly) return crossDomain;
-      for (const cookie of buildImpersonationCookies(
-        { ...result, returnTo: body.returnTo },
-        requestHost(request)
-      )) {
-        crossDomain.headers.append("Set-Cookie", cookie);
-      }
-      return crossDomain;
-    }
-
-    const out = Response.json(summary);
-    for (const cookie of buildImpersonationCookies(
+    const cookies = buildImpersonationCookies(
       { ...result, returnTo: body.returnTo },
       requestHost(request)
-    )) {
-      out.headers.append("Set-Cookie", cookie);
+    );
+
+    if (!body.redirectTo || !handsOffToTarget(request, body.redirectTo)) {
+      return withCookies(Response.json(summary), cookies);
     }
-    return out;
+
+    const target = new URL(body.redirectTo);
+    const ticket = sealImpersonationTicket(
+      { ...result, business: target.host, returnTo: body.returnTo },
+      authConfig.nextAuthSecret
+    );
+    const adopt = new URL("/api/impersonation/adopt", target.origin);
+    adopt.searchParams.set("t", ticket);
+    adopt.searchParams.set(
+      "returnTo",
+      body.landingUrl ?? target.pathname + target.search
+    );
+    return withCookies(
+      Response.json({ ...summary, adoptUrl: adopt.toString() }),
+      cookies
+    );
   };
 }
 
-// Runs on the tenant domain: opens the sealed ticket and sets the impersonation cookie here.
 export function createImpersonationAdoptHandler(authConfig: Auth.Config) {
   return async function GET(request: Request) {
     const url = new URL(request.url);
@@ -185,21 +158,13 @@ export function createImpersonationAdoptHandler(authConfig: Auth.Config) {
       url.searchParams.get("returnTo") || "/",
       origin
     );
-    const out = new Response(null, {
-      status: 302,
-      headers: { Location: dest },
-    });
-    for (const cookie of buildImpersonationCookies(claims, host)) {
-      out.headers.append("Set-Cookie", cookie);
-    }
-    return out;
+    return withCookies(
+      new Response(null, { status: 302, headers: { Location: dest } }),
+      buildImpersonationCookies(claims, host)
+    );
   };
 }
 
-/**
- * Ends a session: writes the append-only session-end audit record (best effort) and clears the
- * cookies so the admin's own token is restored. Token expiry auto-ends too.
- */
 export function createImpersonationEndHandler(authConfig: Auth.Config) {
   return async function POST(request: Request) {
     const session = readImpersonationSession(request);
@@ -213,10 +178,9 @@ export function createImpersonationEndHandler(authConfig: Auth.Config) {
         .catch(() => undefined);
     }
 
-    const out = Response.json({ ok: true });
-    for (const cookie of clearImpersonationCookies(requestHost(request))) {
-      out.headers.append("Set-Cookie", cookie);
-    }
-    return out;
+    return withCookies(
+      Response.json({ ok: true }),
+      clearImpersonationCookies(requestHost(request))
+    );
   };
 }

package/src/impersonation/redirects.test.ts

@@ -0,0 +1,47 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import { isAllowedRedirect } from "./redirects.ts";
+
+const rules = {
+  origin: "https://www.test.snosky.no",
+  platformDomain: "snosky.no",
+  extraOrigins: ["https://minside.test.snokam.no"],
+};
+
+describe("redirect allow-list", () => {
+  it("allows the same origin, relative or absolute", () => {
+    assert.equal(isAllowedRedirect("/manage", rules), true);
+    assert.equal(
+      isAllowedRedirect("https://www.test.snosky.no/x", rules),
+      true
+    );
+  });
+
+  it("allows https on the platform domain and its subdomains", () => {
+    assert.equal(isAllowedRedirect("https://snosky.no/", rules), true);
+    assert.equal(
+      isAllowedRedirect("https://snokam.test.snosky.no/minside", rules),
+      true
+    );
+  });
+
+  it("allows the extra origins it is given", () => {
+    assert.equal(
+      isAllowedRedirect("https://minside.test.snokam.no/okonomi", rules),
+      true
+    );
+  });
+
+  it("refuses everything else", () => {
+    assert.equal(isAllowedRedirect("https://evil.example/", rules), false);
+    assert.equal(
+      isAllowedRedirect("http://snokam.test.snosky.no/", rules),
+      false
+    );
+    assert.equal(
+      isAllowedRedirect("https://snosky.no.evil.example/", rules),
+      false
+    );
+    assert.equal(isAllowedRedirect("javascript:alert(1)", rules), false);
+  });
+});

package/src/impersonation/redirects.ts

@@ -0,0 +1,31 @@
+export function platformBaseDomain(): string {
+  const api = process.env.NEXT_PUBLIC_PLATFORM_API_BASE_URL;
+  if (!api) {
+    throw new Error("NEXT_PUBLIC_PLATFORM_API_BASE_URL is not configured");
+  }
+  return new URL(api).hostname.split(".").slice(-2).join(".");
+}
+
+interface RedirectRules {
+  origin: string;
+  platformDomain: string;
+  extraOrigins?: string[];
+}
+
+export function isAllowedRedirect(
+  target: string,
+  { origin, platformDomain, extraOrigins = [] }: RedirectRules
+): boolean {
+  try {
+    const url = new URL(target, origin);
+    if (url.origin === origin) return true;
+    if (url.protocol !== "https:") return false;
+    return (
+      url.hostname === platformDomain ||
+      url.hostname.endsWith(`.${platformDomain}`) ||
+      extraOrigins.includes(url.origin)
+    );
+  } catch {
+    return false;
+  }
+}

package/src/impersonation/request.ts

@@ -10,17 +10,20 @@ export function platformBase(): string | null {
   );
 }
 
-export function platformClient(accessToken: string): PlatformClient | null {
+export function platformClientFor(
+  authorization: string
+): PlatformClient | null {
   const basePath = platformBase();
   if (!basePath) return null;
   return new PlatformClient(
-    new Configuration({
-      basePath,
-      accessToken: async () => `Bearer ${accessToken}`,
-    })
+    new Configuration({ basePath, accessToken: async () => authorization })
   );
 }
 
+export function platformClient(accessToken: string): PlatformClient | null {
+  return platformClientFor(`Bearer ${accessToken}`);
+}
+
 export function platformNotConfigured(): Response {
   return new Response("NEXT_PUBLIC_PLATFORM_API_BASE_URL is not configured", {
     status: 500,