Lofesa edited a comment on issue #1720: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1720#issuecomment-734197053
Issues in confiles files, regarding files in the order they appears, starting in nginx.conf: 1.- File nginx.conf line 19 you don´t need any proxy config, comment this line ( `include proxy.conf;`) 2.- File nginx.conf lines 59-60 these cetificate files are for your host. Pagespeed don´t use these, but the default for your OS. Read https://www.modpagespeed.com/doc/https_support#configuring_ssl_certificates 3.- File nginx.conf line 116 you have `include enable-php.conf;` this file have `include fastcgi.conf;` and `include pathinfo.conf;`. In this last file you have `if ($fastcgi_script_name ~ "^(.+?\.php)(/.+)$") {` this regular expression is malformed. the " are not needed and you have a close parenthesis that is unopened. All the stuff in this file can be changed by `fastcgi_split_path_info ^(.+?\.php)(/.*)$;` More simple, more fast, no if involved.... 4.- File uhl.site.conf (in the vhos dir.) Here you, or the apanel team, make a decission that I don´t agree: Set all traffic in 1 server block and then check all request for the port and rewrite the url. I think is better and faster having 3 separate server block, 1 for all ports www,1 for http (port 80) and unconditionally redirect to https (port 443), so: ``` server { name www.uhl.site redirect 301 https://uhl.site$request_uri; } server { port 80; name uhl.site; redirect 301 https://uhl.site$request_uri; } server{ port 443; name uhl.site; all the stuff here; } ``` or at least change the rewrite in line 20 with a more efective redirect. 5.- File uhl.site.conf (in the vhos dir.) line 20. Can be this the origin for the 2 downloads? Try to comment and see what happens. 6.- File uhl.site.conf (in the vhos dir.) line 39. Same issue that #3 because have `include pathinfo.conf;` 7.- File uhl.site.conf (in the vhos dir.) line 43. Here you include the `rewrite/uhl.site.conf;`. here you have the `try_files $uri $uri/ /index.php?$args;` you need `try_files $uri $uri/ /index.php$is_args$args;` Only include $args if there are any $args. 8.- I see you have a `pagespeed/main.cf` with pagespeed directives, but don´t see it included on any other file. Others advices: A.- You have the include `pagespeed/pagespeed.conf ` at the very end of the file, try to put at the start, just when all the add_headers ends. B.- you use 2 files enable-php.conf and enable-php-74.conf with the exact same content. Why don´t use only 1? And this file must be ``` location ~ [^/]\.php(/|$) { fastcgi_split_path_info ^(.+?\.php)(/.*)$; try_files $uri =404; fastcgi_index index.php; include fastcgi.conf; fastcgi_pass unix:/tmp/php-cgi-74.sock; } ``` I may have missed something.... ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
