Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FPM-NGINX 7.4 & 8.0: unknown directive "http2" in 7.4-fpm-nginx-alpine #426

Open
JanStorm opened this issue Sep 22, 2024 · 1 comment
Open
Assignees
Labels
🥶 Bug: Upstream Dependency Bugs that affect this project, but need to be fixed in an upsteam library.

Comments

@JanStorm
Copy link

Steps To Reproduce

Run docker run --rm -e SSL_MODE="full" serversideup/php:7.4-fpm-nginx-alpine. The directive http2 is not know in this version of nginx and therefore the https server cannot start.

Outcome

What did you expect?

A valid nginx config which can be interpreted by the running nginx.

What happened instead?

The containers cant start because of a recurring nginx error:

nginx: [emerg] unknown directive "http2" in /etc/nginx/site-opts.d/https.conf:3

Affected Docker Images

serversideup/php:7.4-fpm-nginx-alpine, serversideup/php:8.0-fpm-nginx-alpine

Anything else?

This commit did break it: 9999c09
Reverting it for the few containers worked for me.

@JanStorm JanStorm added the 🧐 Bug: Needs Confirmation Something isn't working, but needs to be confirmed by a team member. label Sep 22, 2024
@jaydrogers jaydrogers added 🐛 Bug: Confirmed Bugs that are able to be replicated. and removed 🧐 Bug: Needs Confirmation Something isn't working, but needs to be confirmed by a team member. labels Sep 26, 2024
@jaydrogers
Copy link
Member

🔀 Workaround

  • Use Debian based images for now (serversideup/php:7.4-fpm-nginx or serversideup/php:8.0-fpm-nginx

👉 Potential Cause

The base OS for Alpine for these versions is Alpine Linux v3.16. This operating system reached "end of life" 4 months ago https://endoflife.date/alpine

⚠️ Warning

People who are continuing to use PHP 7.4 or 8.0 should upgrade to a supported version of PHP as soon as possible: https://www.php.net/supported-versions.php

It's issues like this that make it challenging to support a legacy release. Our intention to offer these older versions were to make it easier for a team to containerize their application, then simplify the change to a modern version of PHP.

🗣️ Further discussion

Because the operating system is end of life, I am hesitant to keep shipping it. Should we just remove the alpine variations for 7.4? If they need 7.4, the Debian images still work.

@jaydrogers jaydrogers moved this to In review in serversideup/php v3.4 Sep 27, 2024
@jaydrogers jaydrogers added 🥶 Bug: Upstream Dependency Bugs that affect this project, but need to be fixed in an upsteam library. and removed 🐛 Bug: Confirmed Bugs that are able to be replicated. labels Sep 27, 2024
@jaydrogers jaydrogers added 🐛 Bug: Confirmed Bugs that are able to be replicated. and removed 🐛 Bug: Confirmed Bugs that are able to be replicated. labels Oct 17, 2024
@jaydrogers jaydrogers changed the title unknown directive "http2" in 7.4-fpm-nginx-alpine FPM-NGINX 7.4 & 8.0: unknown directive "http2" in 7.4-fpm-nginx-alpine Nov 7, 2024
jaydrogers added a commit that referenced this issue Nov 7, 2024
…and messages (#490)

* Force do not exit on error for the try loop on migrations. Fixes #489

* Update notice for custom commands

* Removed Alpine from 7.4.33 because Alpine 3.16 reached EOL (#426 (comment))

* Removed Alpine from PHP 8.0 because Alpine 3.16 reached EOL (#426 (comment))

* Added PHP version and image version to container info start up

* Switch to GitHub runners on Ubuntu 24.04

* Set GHA cache

* Re-added alpine for 7.4 and 8.0 (#426 (comment))

* Converted to buildx and added platform support

* Adjusted PRs to run automatically

* Deprecated and removed linux/arm/v7

* Sign into docker registry only if we're pushing the image

* Added docs to clarify why we don't use subshells (#481 (comment))

* Fixed link
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🥶 Bug: Upstream Dependency Bugs that affect this project, but need to be fixed in an upsteam library.
Projects
None yet
Development

No branches or pull requests

2 participants