fix: CSRF-Origin-Pruefung faelschlich abgelehnt hinter selbst betriebenem Reverse-Proxy
isTrustedOrigin() verglich den Origin-Header ausschliesslich gegen den von Next.js aus request.url rekonstruierten Host. Hinter dem eigenen Caddy/nginx-Reverse-Proxy (docker-compose.yml, k8s/) ist dieser Wert nicht in jeder Konstellation zuverlaessig derselbe wie der echte, oeffentliche Host im Origin-Header - das fuehrte live zu "Ungueltige Anfrage." bei jedem Admin-Upload, obwohl der Request vom eigenen Frontend kam. Prueft jetzt zusaetzlich gegen getBaseUrl() (NEXT_PUBLIC_SITE_URL), genau wie lib/url.ts es fuer denselben Zweck bereits vormacht - als reine ODER-Erweiterung, damit ein falsch gesetztes NEXT_PUBLIC_SITE_URL nicht umgekehrt bislang funktionierende Endpunkte wie /api/checkout blockiert. Co-Authored-By: Claude Sonnet 5 <[email protected]>
This commit is contained in:
1 parent
d0c1d0840c
commit
d03cada256
2 files changed
+50
-3
No files matched your search
+21
-2
@@ -1,3 +1,5 @@
|
||||
import { getBaseUrl } from "@/lib/url";
|
||||
|
||||
/**
|
||||
* Same-Origin-Pruefung fuer zustandsaendernde app/api/.../route.ts-Handler.
|
||||
*
|
||||
@@ -10,6 +12,21 @@
|
||||
* JSON-CSRF-Umgehung) — bei /api/checkout koennte das, sofern der Besuch von
|
||||
* einer eingeloggten Kundschaft ausgeht, Bestellungen unter deren Konto
|
||||
* anlegen und Lagerbestand reservieren, ohne dass die Kundschaft das wollte.
|
||||
*
|
||||
* Akzeptiert den Origin, wenn er zu request.url ODER zu getBaseUrl()
|
||||
* (NEXT_PUBLIC_SITE_URL) passt — nicht nur zu request.url allein. Hinter
|
||||
* einem selbst betriebenen Reverse-Proxy (Caddy/nginx, siehe
|
||||
* docker-compose.yml bzw. k8s/) ist der vom Next.js-Prozess intern
|
||||
* rekonstruierte Host aus request.url nicht in jeder Proxy-Konstellation
|
||||
* zuverlaessig derselbe wie der echte, oeffentliche Host, den der Browser im
|
||||
* Origin-Header sendet (vgl. denselben Grundsatz bereits in lib/url.ts) —
|
||||
* das fuehrte live faelschlich zu "Ungueltige Anfrage." bei jedem
|
||||
* Admin-Upload, obwohl der Request tatsaechlich vom eigenen Frontend kam.
|
||||
* Die zusaetzliche ODER-Bedingung statt eines reinen Ersatzes ist bewusst
|
||||
* risikoarm: sie akzeptiert nur MEHR legitime Faelle als vorher, verengt
|
||||
* nichts — verhindert also, dass ein falsch/veraltet gesetztes
|
||||
* NEXT_PUBLIC_SITE_URL ploetzlich bislang funktionierende Endpunkte wie
|
||||
* /api/checkout fuer echte Kundschaft blockiert.
|
||||
*/
|
||||
export function isTrustedOrigin(request: Request): boolean {
|
||||
const origin = request.headers.get("origin");
|
||||
@@ -21,9 +38,11 @@ export function isTrustedOrigin(request: Request): boolean {
|
||||
if (!origin) return true;
|
||||
|
||||
try {
|
||||
const requestHost = new URL(request.url).host;
|
||||
const originHost = new URL(origin).host;
|
||||
return requestHost === originHost;
|
||||
const requestHost = new URL(request.url).host;
|
||||
if (requestHost === originHost) return true;
|
||||
const trustedHost = new URL(getBaseUrl()).host;
|
||||
return trustedHost === originHost;
|
||||
} catch {
|
||||
return false;
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user