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
phpfpm: add options #63962
phpfpm: add options #63962
Conversation
I want to highlight that there was an option to add configuration globally across all
I'll state that this PR should not be merged, in my opinion, because it is based on a PR I'm planning to revert with 63931. Note that #63931 will reintroduce the ability to set As an aside @Izorkin what is your use case for using a tcp connection instead of a unix socket? I'm curious because one has much better performance than the other. |
@aanderse globalPhpOptions allows you to set global parameters.
|
@Izorkin Setting global php options is what you removed from your last PR. Now you are adding it back. I understand how to use tcp connection I'm just curious why you would do that when socket is faster. What benefit do you get from using tcp instead of socket in this example? |
Yes, I missed the moment with the global php.ini configuration. |
Closing in favor of #63931 |
Motivation for this change
Add options globalPhpOptions to configure php.ini for all PHP-FPM pools.
Add option listenTCP to enable work PHP-FPM with network address.
cc @etu
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)