Skip to content

[Kritisch] Web-Profiler aus Produktion entfernen#529

Open
maltehuebner wants to merge 1 commit into
mainfrom
security/disable-web-profiler-in-prod
Open

[Kritisch] Web-Profiler aus Produktion entfernen#529
maltehuebner wants to merge 1 commit into
mainfrom
security/disable-web-profiler-in-prod

Conversation

@maltehuebner

Copy link
Copy Markdown
Contributor

Behebt #521.

Problem

Der Symfony-Web-Profiler war in der Produktionsumgebung erreichbar:

  1. config/bundles.php aktivierte WebProfilerBundle für prod.
  2. config/routing/dev/web_profiler.yaml registrierte /_profiler und /_wdt ohne when@dev:-Schutz und wurde über den Verzeichnis-Import in jeder Umgebung geladen.

Bestätigt via bin/console debug:router --env=prod — sichtbar waren u. a. /_profiler/open (liest Serverdateien), /_profiler/phpinfo und /_wdt/{token}.

Änderung

  • config/bundles.php: WebProfilerBundle nur noch für dev/test.
  • config/routing/dev/web_profiler.yaml: Inhalt in when@dev: gekapselt (identisch zum Symfony-Standardmuster).

Hinweis: Die dev/-Routing-Datei ist die tatsächlich geladene Profiler-Routendefinition (die when@dev-geschützte config/routes/web_profiler.yaml wird im aktuellen Setup gar nicht importiert). Deshalb wurde die aktive Datei gekapselt statt gelöscht — sonst wäre der Profiler auch in dev verschwunden.

Verifikation

debug:router --env=prod  → keine _profiler/_wdt-Routen mehr
debug:router --env=dev   → Profiler-Routen unverändert vorhanden

🤖 Generated with Claude Code

Der Web-Profiler war in prod erreichbar: WebProfilerBundle war in
config/bundles.php für 'prod' aktiviert, und config/routing/dev/web_profiler.yaml
registrierte /_profiler und /_wdt ohne when@dev-Schutz. Bestätigt via
`bin/console debug:router --env=prod` (u. a. /_profiler/open Dateizugriff,
/_profiler/phpinfo).

- bundles.php: WebProfilerBundle nur noch für dev/test
- routing/dev/web_profiler.yaml: in when@dev gekapselt

Verifiziert: keine _profiler/_wdt-Routen mehr in prod, in dev unverändert
vorhanden.

Fixes #521

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant