Abmelden fuehrte im Docker-Betrieb auf 0.0.0.0:3000 - eine Adresse, die kein Browser aufrufen kann. Ursache war hausgemacht: ENV HOSTNAME=0.0.0.0 aus #031 reparierte die Zustandspruefung und lenkte zugleich alle Weiterleitungen aus Server Actions dorthin, weil Next sie serverseitig absolut macht. Vorher stand dort der Containername - derselbe Fehler, nur unauffaelliger. Neuer Helfer eigenerPfad() baut die Adresse aus Einstellung, APP_URL oder den Kopfzeilen der Anfrage. Betroffen waren Abmelden, die Sitzungspruefung und das Loeschen eines Kunden. Zusaetzlich uebergeht oeffentlicheBasis() eine als Lausch-Adresse eingetragene APP_URL, statt sie zu uebernehmen. Gegen einen Docker-Stapel geprueft: Mit APP_URL=http://0.0.0.0:3000 leitet die Anwendung jetzt auf den tatsaechlich aufgerufenen Namen. Dazu #032: Das ct-Skript setzt die Herkunft der App-Skripte selbst, und die Speichervorgabe steht auf 4 GB. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
+38
-19
@@ -9,31 +9,34 @@
|
||||
* sich fälschen; für eine Weiterleitung auf die eigene Seite ist das unkritisch, als erste Wahl aber
|
||||
* unnötig.
|
||||
*/
|
||||
export function oeffentlicheBasis(request: Request, eingetragen?: string): URL {
|
||||
if (eingetragen) {
|
||||
try {
|
||||
return new URL(eingetragen);
|
||||
} catch {
|
||||
// Unbrauchbarer Eintrag: weiter mit den übrigen Quellen.
|
||||
}
|
||||
}
|
||||
/**
|
||||
* Lausch-Adressen taugen nicht als Ziel für einen Browser: `0.0.0.0` heißt „alle Schnittstellen“.
|
||||
* Steht so etwas in der Konfiguration, wird es übergangen statt eine unerreichbare Seite zu bauen.
|
||||
*/
|
||||
const UNBRAUCHBAR = new Set(["0.0.0.0", "::", "[::]", "", "0"]);
|
||||
|
||||
if (process.env.APP_URL) {
|
||||
try {
|
||||
return new URL(process.env.APP_URL);
|
||||
} catch {
|
||||
// ebenso
|
||||
}
|
||||
function brauchbar(adresse: string | undefined): URL | null {
|
||||
if (!adresse) return null;
|
||||
try {
|
||||
const ergebnis = new URL(adresse);
|
||||
return UNBRAUCHBAR.has(ergebnis.hostname) ? null : ergebnis;
|
||||
} catch {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
export function oeffentlicheBasis(request: Request, eingetragen?: string): URL {
|
||||
const ausEinstellung = brauchbar(eingetragen);
|
||||
if (ausEinstellung) return ausEinstellung;
|
||||
|
||||
const ausUmgebung = brauchbar(process.env.APP_URL);
|
||||
if (ausUmgebung) return ausUmgebung;
|
||||
|
||||
const gastgeber = request.headers.get("x-forwarded-host") ?? request.headers.get("host");
|
||||
if (gastgeber) {
|
||||
const schema = request.headers.get("x-forwarded-proto") ?? new URL(request.url).protocol.replace(":", "");
|
||||
try {
|
||||
return new URL(`${schema}://${gastgeber}`);
|
||||
} catch {
|
||||
// ebenso
|
||||
}
|
||||
const ausKopfzeile = brauchbar(`${schema}://${gastgeber}`);
|
||||
if (ausKopfzeile) return ausKopfzeile;
|
||||
}
|
||||
|
||||
return new URL(request.url);
|
||||
@@ -43,3 +46,19 @@ export function oeffentlicheBasis(request: Request, eingetragen?: string): URL {
|
||||
export function eigeneAdresse(request: Request, pfad: string, eingetragen?: string): URL {
|
||||
return new URL(pfad, oeffentlicheBasis(request, eingetragen));
|
||||
}
|
||||
|
||||
/**
|
||||
* Dasselbe für Server Actions und Seiten, die keine `Request` zur Hand haben.
|
||||
*
|
||||
* Nötig, weil Next.js eine Weiterleitung aus einer Server Action serverseitig absolut macht – und
|
||||
* dafür die Adresse nimmt, auf die der Server lauscht. Im Container ist das `0.0.0.0` (oder der
|
||||
* Containername); im Browser landet man damit auf einer Seite, die es nicht gibt.
|
||||
*/
|
||||
export async function eigenerPfad(pfad: string, eingetragen?: string): Promise<string> {
|
||||
const { headers } = await import("next/headers");
|
||||
const kopfzeilen = await headers();
|
||||
|
||||
// Die Anfrage selbst gibt es hier nicht; eine leere mit den echten Kopfzeilen genügt.
|
||||
const basis = oeffentlicheBasis(new Request("http://platzhalter.invalid", { headers: kopfzeilen }), eingetragen);
|
||||
return new URL(pfad, basis).toString();
|
||||
}
|
||||
|
||||
@@ -2,12 +2,16 @@
|
||||
|
||||
import { cookies } from "next/headers";
|
||||
import { redirect } from "next/navigation";
|
||||
import { eigenerPfad } from "@/lib/adresse";
|
||||
import { SITZUNGS_COOKIE } from "@/lib/sitzung";
|
||||
import { protokolliere } from "@/server/audit";
|
||||
import { einstellungen } from "@/server/einstellungen";
|
||||
|
||||
/** Abmelden aus dem Benutzermenü. Die Anmeldung selbst läuft über /api/auth/anmelden. */
|
||||
export async function abmelden() {
|
||||
(await cookies()).delete(SITZUNGS_COOKIE);
|
||||
await protokolliere("abmeldung", "Benutzer", null, {});
|
||||
redirect("/login");
|
||||
|
||||
const { dashboardUrl } = await einstellungen();
|
||||
redirect(await eigenerPfad("/login", dashboardUrl));
|
||||
}
|
||||
|
||||
@@ -6,6 +6,7 @@ import { z } from "zod";
|
||||
import { prisma } from "@/lib/db";
|
||||
import { protokolliere } from "@/server/audit";
|
||||
import { sitzungErforderlich } from "@/server/auth/sitzungspflicht";
|
||||
import { eigenerPfad } from "@/lib/adresse";
|
||||
|
||||
export interface AktionsErgebnis {
|
||||
fehler?: string;
|
||||
@@ -79,7 +80,7 @@ export async function kundeLoeschen(formular: FormData) {
|
||||
|
||||
revalidatePath("/kunden");
|
||||
revalidatePath("/hosts/nicht-zugeordnet");
|
||||
redirect("/kunden");
|
||||
redirect(await eigenerPfad("/kunden"));
|
||||
}
|
||||
|
||||
const standortSchema = z.object({
|
||||
|
||||
@@ -2,6 +2,7 @@ import { cookies } from "next/headers";
|
||||
import { redirect } from "next/navigation";
|
||||
import { cache } from "react";
|
||||
import type { Rolle } from "@/generated/prisma/enums";
|
||||
import { eigenerPfad } from "@/lib/adresse";
|
||||
import { prisma } from "@/lib/db";
|
||||
import { type GeleseneSitzung, SITZUNGS_COOKIE, sitzungLesen } from "@/lib/sitzung";
|
||||
import { darf, type Recht } from "./rollen";
|
||||
@@ -68,10 +69,10 @@ export const angemeldet = cache(async (): Promise<AngemeldeterBenutzer | null> =
|
||||
*/
|
||||
export async function sitzungErforderlich(recht?: Recht): Promise<AngemeldeterBenutzer> {
|
||||
const benutzer = await angemeldet();
|
||||
if (!benutzer) redirect("/login?fehler=Bitte+anmelden.");
|
||||
if (!benutzer) redirect(await eigenerPfad("/login?fehler=Bitte+anmelden."));
|
||||
|
||||
if (recht && !darf(benutzer.rolle, recht)) {
|
||||
redirect("/?fehler=Für+diesen+Bereich+fehlt+die+Berechtigung.");
|
||||
redirect(await eigenerPfad("/?fehler=Für+diesen+Bereich+fehlt+die+Berechtigung."));
|
||||
}
|
||||
|
||||
return benutzer;
|
||||
|
||||
Reference in New Issue
Block a user