Sicherheit: Offene Redirects via REQUEST_URI fixen #43

New Issue

greggy · 2026-05-19T15:42:56+02:00

greggy commented

2026-05-19 15:42:56 +02:00

Schwachstelle #3 – Offene Redirects via `$_SERVER["REQUEST_URI"]`

header("Location: " . $_SERVER["REQUEST_URI"] . "#form-result");

REQUEST_URI wird vom Client kontrolliert. In den meisten Konfigurationen sicher, aber in Edge-Cases (Proxy-Setups) könnte das umgeleitet werden.

Empfehlung

Statt REQUEST_URI einen festen Pfad verwenden:

header("Location: /index.php#form-result");

Priorität

🟠 Mittel

## Schwachstelle #3 – Offene Redirects via `$_SERVER["REQUEST_URI"]` ```php header("Location: " . $_SERVER["REQUEST_URI"] . "#form-result"); ``` `REQUEST_URI` wird vom Client kontrolliert. In den meisten Konfigurationen sicher, aber in Edge-Cases (Proxy-Setups) könnte das umgeleitet werden. ### Empfehlung Statt `REQUEST_URI` einen festen Pfad verwenden: ```php header("Location: /index.php#form-result"); ``` ### Priorität 🟠 Mittel

greggy added the

KI

label 2026-05-19 15:42:56 +02:00

greggy added the

KI

label 2026-05-22 01:03:20 +02:00

greggy commented

2026-05-22 01:03:49 +02:00

Phase 1: Analyse abgeschlossen ✅

Komplexität: S (Small)

Spezifikation

Alle 3 Vorkommen von $_SERVER["REQUEST_URI"] in HomeController ersetzen durch festen Pfad:

header("Location: /#form-result");

Sicher, da Kontaktformular nur auf der Startseite existiert.

Akzeptanzkriterien

Kein $_SERVER["REQUEST_URI"] mehr in Redirects
Redirect korrekt zur Startseite + #form-result
PRG-Pattern funktioniert weiterhin

Status: ReadyForDev → Weiter zu Phase 2 (Implementierung)

## Phase 1: Analyse abgeschlossen ✅ **Komplexität: S (Small)** ### Spezifikation Alle 3 Vorkommen von `$_SERVER["REQUEST_URI"]` in HomeController ersetzen durch festen Pfad: ```php header("Location: /#form-result"); ``` Sicher, da Kontaktformular nur auf der Startseite existiert. ### Akzeptanzkriterien - [ ] Kein `$_SERVER["REQUEST_URI"]` mehr in Redirects - [ ] Redirect korrekt zur Startseite + #form-result - [ ] PRG-Pattern funktioniert weiterhin **Status: ReadyForDev** → Weiter zu Phase 2 (Implementierung)

greggy referenced this issue from a commit

2026-05-22 01:06:21 +02:00

fix(security): replace REQUEST_URI with fixed path in redirects (#43)

greggy referenced a pull request that will close this issue

2026-05-22 01:07:46 +02:00

Fix #43: Offene Redirects via REQUEST_URI fixen #51

greggy closed this issue

2026-05-22 08:32:24 +02:00

greggy referenced this issue from a commit

2026-05-22 08:32:25 +02:00

Merge pull request 'Fix #43: Offene Redirects via REQUEST_URI fixen' (#51) from feature/issue-43-open-redirect-fix into main

Sign in to join this conversation.