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.

Sign up for GitHub

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

Jump to bottom

Add initial "NGINX Unit" based image #875

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add initial "NGINX Unit" based image #875

wants to merge 1 commit into from

Commits on Jan 12, 2024

  1. Add initial "NGINX Unit" based image 

    The Unit configuration I've included here is based on the example provided by the Unit documentation (https://unit.nginx.org/howto/wordpress/), although I did spend time understanding it before blindly trusting it. 😄

The biggest 😬 here (IMO) is that WordPress itself does not know about Unit yet, so the permalink handling is a bit off -- I've added a hack in `wp-config-docker.php` (specifically in the Unit images only) which works around it by teaching WordPress that Unit is effectively NGINX (which does work and is accurate enough for what WordPress needs), but that should probably be an issue or PR to the WordPress community instead.
    @tianon
    tianon committed Jan 12, 2024
    Configuration menu
    Copy the full SHA
    3610d02 View commit details
    Browse the repository at this point in the history