Skip to content

fix(docker): ship a static welcome page instead of phpinfo()#2524

Merged
dunglas merged 1 commit into
mainfrom
fix/ghsa-cj57-c655-p798-phpinfo-default
Jul 18, 2026
Merged

fix(docker): ship a static welcome page instead of phpinfo()#2524
dunglas merged 1 commit into
mainfrom
fix/ghsa-cj57-c655-p798-phpinfo-default

Conversation

@dunglas

@dunglas dunglas commented Jul 18, 2026

Copy link
Copy Markdown
Member

The default /app/public/index.php in both Docker images ran phpinfo(). Derived images that don't provide their own index served it verbatim, exposing environment variables (which commonly hold secrets), php.ini directives, versions and system paths.

Reuse the RPM/deb welcome page (package/content/index.php) as the single default index across every distribution channel, and point both the Debian and Alpine images at it. The page discloses nothing: no PHP, no scripts, no external assets.

See GHSA-cj57-c655-p798.

Welcome page graphic design by @ginifizz.

The default `/app/public/index.php` in both Docker images ran `phpinfo()`.
Derived images that don't provide their own index serve it verbatim,
exposing environment variables (which commonly hold secrets), php.ini
directives, versions and system paths.

Reuse the RPM/deb welcome page (`package/content/index.php`) as the single
default index for every distribution channel, and point both the Debian and
Alpine images at it. The page discloses nothing: no PHP, no scripts, no
external assets. See GHSA-cj57-c655-p798.

Welcome page graphic design by @ginifizz.

Co-authored-by: ginifizz <16579730+ginifizz@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 18, 2026 13:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@dunglas
dunglas merged commit 17940ff into main Jul 18, 2026
69 of 75 checks passed
@dunglas
dunglas deleted the fix/ghsa-cj57-c655-p798-phpinfo-default branch July 18, 2026 13:43
dunglas added a commit that referenced this pull request Jul 18, 2026
The `Lint Code Base` job on `main` is red: the welcome page added in
#2524 ends without a trailing newline, which violates `.editorconfig`
(`insert_final_newline = true`). EDITORCONFIG was the only failing
linter.

Appends the missing newline. No content change.
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.

2 participants