Skip to content

Feat: Setting php.ini values in php-fpm.conf#261

Merged
ShadowArcanist merged 3 commits intocoollabsio:nextfrom
tomhatzer:main
Apr 9, 2025
Merged

Feat: Setting php.ini values in php-fpm.conf#261
ShadowArcanist merged 3 commits intocoollabsio:nextfrom
tomhatzer:main

Conversation

@tomhatzer
Copy link
Contributor

@tomhatzer tomhatzer commented Apr 9, 2025

Hey there! 👋

Just a small improvement for people deploying PHP applications using coolify, so they can customize their php.ini settings easily inside the php-fpm.conf FPM worker config. Imho cleaner than setting it in the fastcgi_params.

I did not yet update the existing docs or examples. If I'm allowed to do that too, I'll also include this in my PR.

Thank you for all your work, you are doing an incredible job 🤗

Have a lovely day,
Tom

Adds description on how to update php.ini settings directly in the php-fpm.conf file.
Adds description on how to update php.ini settings directly in the php-fpm.conf file.
@ShadowArcanist
Copy link
Member

Hey @tomhatzer thanks for the PR!

I did not yet update the existing docs or examples. If I'm allowed to do that too, I'll also include this in my PR.

Yes!! feel free to include them on this PR (mention me once this PR is ready for review)

@ShadowArcanist ShadowArcanist added ✨ Enhancement Suggestions to improve or add detail to existing docs. ⚖️ Medium Priority Important tasks to be addressed soon, but not urgent. labels Apr 9, 2025
@tomhatzer
Copy link
Contributor Author

Thank you @ShadowArcanist , I've updated the laravel.md nixpacks file and also the Laravel examples here: coollabsio/coolify-examples#42

Please feel free to approve/merge whenever you've got time available or comment if I missed something.

Thank you very much!

@ShadowArcanist ShadowArcanist changed the base branch from main to next April 9, 2025 15:54
@ShadowArcanist
Copy link
Member

Thanks again for the PR @tomhatzer !!

This will be merged to main branch via #260

@ShadowArcanist ShadowArcanist merged commit 359544d into coollabsio:next Apr 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Enhancement Suggestions to improve or add detail to existing docs. ⚖️ Medium Priority Important tasks to be addressed soon, but not urgent.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants