Compare commits

...
2 Commits
Author SHA1 Message Date
Nick SzaunigandClaude Sonnet 5 1df62f20c8 fix: Klickbarkeit der Wissensdatenbank-Karten sichtbar machen
Docker Image bauen und veroeffentlichen / build-and-push (push) Successful in 2m24s
Die Artikel-Karten auf /wissen waren nur per Hover als klickbar erkennbar
(Unterstreichung der Überschrift) — ohne sichtbaren Hinweis im Ruhezustand
war unklar, dass man draufklicken kann. Zeigt jetzt dauerhaft ein
"Weiterlesen →" je Karte.

Co-Authored-By: Claude Sonnet 5 <[email protected]>
2026-09-11 22:48:26 +02:00
Nick SzaunigandClaude Sonnet 5 d03cada256 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]>
2026-09-11 16:06:56 +02:00
6 changed files with 59 additions and 3 deletions

No files matched your search

+6
View File
@@ -31,6 +31,12 @@ export default async function WissenPage() {
<Link href={`/wissen/${slug}`} className="block group"> <Link href={`/wissen/${slug}`} className="block group">
<h2 className="text-xl text-forest group-hover:underline">{article.title}</h2> <h2 className="text-xl text-forest group-hover:underline">{article.title}</h2>
<p className="mt-2 text-sm text-ink/65 leading-relaxed">{article.excerpt}</p> <p className="mt-2 text-sm text-ink/65 leading-relaxed">{article.excerpt}</p>
<span className="mt-3 inline-flex items-center gap-1 text-sm text-honey-dark tracking-wide-caps">
{t.wissenListPage.readMore}
<span className="transition-transform group-hover:translate-x-1" aria-hidden="true">
→
</span>
</span>
</Link> </Link>
</ScrollReveal> </ScrollReveal>
); );
+1
View File
@@ -436,6 +436,7 @@ export const de: Dictionary = {
wissenListPage: { wissenListPage: {
eyebrow: "Honigwissen", eyebrow: "Honigwissen",
title: "Wissenswertes rund um Honig", title: "Wissenswertes rund um Honig",
readMore: "Weiterlesen",
}, },
wissenArticlePage: { wissenArticlePage: {
backLabel: "Zurück zum Honigwissen", backLabel: "Zurück zum Honigwissen",
+1
View File
@@ -436,6 +436,7 @@ export const en: Dictionary = {
wissenListPage: { wissenListPage: {
eyebrow: "Honey Knowledge", eyebrow: "Honey Knowledge",
title: "Good to know about honey", title: "Good to know about honey",
readMore: "Read more",
}, },
wissenArticlePage: { wissenArticlePage: {
backLabel: "Back to Honey Knowledge", backLabel: "Back to Honey Knowledge",
+1
View File
@@ -234,6 +234,7 @@ export interface Dictionary {
wissenListPage: { wissenListPage: {
eyebrow: string; eyebrow: string;
title: string; title: string;
readMore: string;
}; };
wissenArticlePage: { wissenArticlePage: {
backLabel: string; backLabel: string;
+29 -1
View File
@@ -1,6 +1,17 @@
import { describe, test, expect } from "vitest"; import { describe, test, expect, beforeEach, afterEach } from "vitest";
import { isTrustedOrigin } from "./csrf"; import { isTrustedOrigin } from "./csrf";
const originalSiteUrl = process.env.NEXT_PUBLIC_SITE_URL;
beforeEach(() => {
delete process.env.NEXT_PUBLIC_SITE_URL;
});
afterEach(() => {
if (originalSiteUrl === undefined) delete process.env.NEXT_PUBLIC_SITE_URL;
else process.env.NEXT_PUBLIC_SITE_URL = originalSiteUrl;
});
function makeRequest(url: string, origin?: string): Request { function makeRequest(url: string, origin?: string): Request {
const headers = new Headers(); const headers = new Headers();
if (origin !== undefined) headers.set("origin", origin); if (origin !== undefined) headers.set("origin", origin);
@@ -37,4 +48,21 @@ describe("isTrustedOrigin", () => {
const request = makeRequest("https://wabenhain.com/api/checkout"); const request = makeRequest("https://wabenhain.com/api/checkout");
expect(isTrustedOrigin(request)).toBe(true); expect(isTrustedOrigin(request)).toBe(true);
}); });
test("accepts an origin matching NEXT_PUBLIC_SITE_URL even when it differs from request.url's host", () => {
// Reproduziert den Live-Bug: hinter einem selbst betriebenen
// Reverse-Proxy kann request.url intern einen anderen Host tragen als
// den echten, oeffentlichen Host im Origin-Header — ohne diesen
// Fallback wurde das faelschlich als CSRF-Versuch abgelehnt
// ("Ungueltige Anfrage."), obwohl der Request vom eigenen Frontend kam.
process.env.NEXT_PUBLIC_SITE_URL = "https://wabenhain.com";
const request = makeRequest("http://app:3000/api/admin/upload", "https://wabenhain.com");
expect(isTrustedOrigin(request)).toBe(true);
});
test("still rejects a forged cross-origin request when NEXT_PUBLIC_SITE_URL is configured", () => {
process.env.NEXT_PUBLIC_SITE_URL = "https://wabenhain.com";
const request = makeRequest("http://app:3000/api/admin/upload", "https://attacker.example");
expect(isTrustedOrigin(request)).toBe(false);
});
}); });
+21 -2
View File
@@ -1,3 +1,5 @@
import { getBaseUrl } from "@/lib/url";
/** /**
* Same-Origin-Pruefung fuer zustandsaendernde app/api/.../route.ts-Handler. * 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 * JSON-CSRF-Umgehung) — bei /api/checkout koennte das, sofern der Besuch von
* einer eingeloggten Kundschaft ausgeht, Bestellungen unter deren Konto * einer eingeloggten Kundschaft ausgeht, Bestellungen unter deren Konto
* anlegen und Lagerbestand reservieren, ohne dass die Kundschaft das wollte. * 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 { export function isTrustedOrigin(request: Request): boolean {
const origin = request.headers.get("origin"); const origin = request.headers.get("origin");
@@ -21,9 +38,11 @@ export function isTrustedOrigin(request: Request): boolean {
if (!origin) return true; if (!origin) return true;
try { try {
const requestHost = new URL(request.url).host;
const originHost = new URL(origin).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 { } catch {
return false; return false;
} }