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

Feature request: GUI to Increase Nginx FastCGI read timeout #45

Open
batonac opened this issue Jun 14, 2022 · 3 comments
Open

Feature request: GUI to Increase Nginx FastCGI read timeout #45

batonac opened this issue Jun 14, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@batonac
Copy link
Contributor

batonac commented Jun 14, 2022

I'd like to see the FastCGI increased on some sites to work around the 504 Gateway Timeout error on Nginx with long-running php-fpm scripts.

The applicable PHP directives are already exposed in the PHP panel, but it appears that we need an interface to manage the fastcgi_read_timeout nginx fastcgi directive, presumably under the "Tweaks" menu.

@elindydotcom elindydotcom added the enhancement New feature or request label Jun 14, 2022
@elindydotcom elindydotcom added the good first issue Good for newcomers label Nov 30, 2022
@batonac
Copy link
Contributor Author

batonac commented Jan 19, 2023

It occurs to me that client_max_body_size should really be added in addition to fastcgi_read_timeout.

What should really happen here, IMO, is a unified setting for 'Max Upload Size' and 'Script Timeout' that sets the PHP and webserver parameters in tandem with one another:

  • upload_max_filesize (and possibly post_max_size) PHP directive should match client_max_body_size NGINX directive.
  • max_execution_time (and possibly max_input_time) PHP directive should match fastcgi_read_timeout NGINX directive.

@batonac
Copy link
Contributor Author

batonac commented Jan 19, 2023

Sorry, I spoke a bit out of turn. I just realized from your release notes that the FILE UPLOAD SIZE setting takes care of most of this last comment. I'll start a new ticket.

@batonac
Copy link
Contributor Author

batonac commented Jan 19, 2023

Arrg again. I just realized that this is already completely taken care of with the existing FILE UPLOAD SIZE setting. I was unfortunately looking at the PHP tab instead of tweaks.

I guess that reiterates/further-clarifies the original request to have a SCRIPT TIMEOUT setting in the Tweaks tab, in addition to the FILE UPLOAD SIZE setting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants