Skip to content

NGINX web server setup document contains inconsistent references to both php5 and php7.0, should be php7.3 #1811

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

Closed
adrianyorke opened this issue Feb 3, 2021 · 1 comment · Fixed by #1812
Labels
linked pull request Has a linked Pull Request that will close the issue.

Comments

@adrianyorke
Copy link
Contributor

adrianyorke commented Feb 3, 2021

This issue relates to NGINX web server setup document which can be found here:
https://github.com/raspberrypi/documentation/blob/master/remote-access/web-server/nginx.md

In the section labelled "Edit by removing the # characters on the following lines:" there is a reference to php5 in the following line:

fastcgi_pass unix:/var/run/php5-fpm.sock;

This should read:

fastcgi_pass unix:/run/php/php7.3-fpm.sock;

Also update this line later in the same document which references php7.0 to be consistent:

fastcgi_pass unix:/var/run/php/php7.0-fpm.sock;
@adrianyorke adrianyorke changed the title NGINX web server setup document contains a reference to php5, should be php7.0 NGINX web server setup document contains a reference to php5, should be php7.3 Feb 3, 2021
@adrianyorke adrianyorke changed the title NGINX web server setup document contains a reference to php5, should be php7.3 NGINX web server setup document contains references to php5, should be php7.3 Feb 3, 2021
@adrianyorke adrianyorke changed the title NGINX web server setup document contains references to php5, should be php7.3 NGINX web server setup document contains inconsistent references to both php5 and php7.0, should be php7.3 Feb 3, 2021
@aallan
Copy link
Contributor

aallan commented Jun 7, 2021

This is resolved by #1812 I think?

@aallan aallan added the linked pull request Has a linked Pull Request that will close the issue. label Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linked pull request Has a linked Pull Request that will close the issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants