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

Adding your own "php.ini" needs to be documented #242

Closed
jaydrogers opened this issue Dec 4, 2023 — with Bugflow.io · 3 comments
Closed

Adding your own "php.ini" needs to be documented #242

jaydrogers opened this issue Dec 4, 2023 — with Bugflow.io · 3 comments
Assignees
@jaydrogers
Labels
📝 Documentation Items that relate to documentation improvements. 👨‍🔬 Release: Pre-release Items that relate to a release that isn't considered stable.

Comments

@jaydrogers Bugflow.io
Copy link
Member

🌎 URL

https://release-v3-0.docker-php.pages.dev/docs/guide/changing-common-php-settings

⚡️ Reported By

Jay Rogers

🔎 More details

Browser: Firefox 120 (120.0)

Screen Resolution: 834x1194

Viewport Dimensions: 1302x1280
@jaydrogers jaydrogers self-assigned this Dec 4, 2023
@jaydrogers jaydrogers added 📝 Documentation Items that relate to documentation improvements. 👨‍🔬 Release: Pre-release Items that relate to a release that isn't considered stable. labels Dec 4, 2023
@pbromb
Copy link

pbromb commented Dec 30, 2023

Hi @jaydrogers, it would be nice to have a php.ini file from the official PHP image and not modify it.
You can override parameters by adding a file such as $PHP_INI_DIR/etc/conf.d/my-parameters.ini

jaydrogers added a commit that referenced this issue Apr 18, 2024
@jaydrogers
Renamed provided PHP ini file per #242 (comment)
ce32611
@jaydrogers
Copy link
Member Author

Thanks for the suggestion on the best practices @pbromb!

I made that change here: ce32611

I'm working on improving the docs for adding your own PHP ini as well

jaydrogers added a commit that referenced this issue Apr 18, 2024
@jaydrogers
Documented PHP ini customizations. Fixes #242
b827e8c
@jaydrogers jaydrogers mentioned this issue Apr 18, 2024
Merged
@jaydrogers
Copy link
Member Author

This has been made available on this PR:

The specific link to the preview site is here: https://improve-file-permissions.docker-php.pages.dev/docs/customizing-the-image/changing-common-php-settings#setting-your-own-phpini

jaydrogers added a commit that referenced this issue Apr 22, 2024
@jaydrogers
Change execution to be unprivileged (#311)
bcc5b18 
* Update PHP patch versions to 8.1.28, 8.2.18, and 8.3.6

* Removed NGINX_USER variable

* Changed FPM-NGINX to be unprivileged

* Improved readability of health check

* Improved health check

* Simplified NGINX config

* Set cert to not be a CA cert

* Configured Apache directories for changing permissions

* Changed Apache to run as unprivileged

* Converted NGINX Unit to be unprivileged

* Dropped permissions on CLI image

* Improve UX with better healthcheck notifications

* Added empty line for easier expansion by scripts

* Refactor .env.example file in docs directory

* Remove beta notice

* Add unprivileged by default support for Docker images

* Added hetzner

* Add default configurations for unprivileged Docker images

* Update Dockerfile examples in upgrade guide

* Improved syntax highlighting

* Updated versions in README

* Renamed provided PHP ini file per #242 (comment)

* Documented PHP ini customizations. Fixes #242

* Restructured the docs

* Update redirect paths in DocumentDrivenNotFound.vue

* Updated docs on Laravel Automations script

* Update Laravel task scheduler documentation and Docker Compose example

* Update Laravel task scheduler documentation and Docker Compose example

* Update links in Laravel Automations documentation

* Update Laravel task scheduler documentation and Docker Compose example

* Update Laravel queue and Laravel Horizon documentation

* Clear config cache before running migrations. Fixes #303

* Remove comment

* Added file permissions docs

* Update Docker image migration guide

* Removed old beta docs

* Added privileges

* Update command reference documentation

* Update contributing guide with link to docs contribution instructions

* Fixed links

* Update static site generation command in service_deploy-static-site.yml

* Added helpful notes to the migration guide

* Changed format of docker-php-serversideup-set-id

* Added /var/www/html/ to chown

* Update migration guide with PHP_FPM_POOL_NAME rename

* Fix SSL keypair generation in 10-init-webserver-config.sh

* Add PHP_OPCACHE_ENABLE=1 to production environment for increased performance

* Improved get-php-versions script to accept null values and rely more on the Official PHP data
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jaydrogers jaydrogers

Labels
📝 Documentation Items that relate to documentation improvements. 👨‍🔬 Release: Pre-release Items that relate to a release that isn't considered stable.
Projects
No open projects
v3.0 - PHP Docker Images
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jaydrogers @pbromb