Lofesa edited a comment on issue #1720:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1720#issuecomment-731720301


   Hi
   Some questions about the config files:
   
   1.- There is some files (`pagespeed.conf` in file `uhl.site.conf `line 5, 
`pagespeedstatslog.conf` line 68, `pagespeedhandler.conf `line 69) you have no 
included. Maybe in this files are the directives for nginx_status and  
pagespeed log or the server block for 127.0.0.1?
   
   2.- Why you have fastcgi directives out of a server block in uhl.site.conf? 
If the directives are out of a server block is the same as put it in the http 
block, so why don´t put it at the same level that other fastcgi directives in 
nginx.conf
   
   3.- You have enbled pagespeed in the http block so pagespeed works for all 
the server blocks. You need pagespeed working in phpadmin or in the default 
server? if not, put `pagespeed unplugged;` in it to disable pagespeed in these 
servers
   
   4.- Why you need  `pagespeed MapOriginDomain "http://uhl.site"; 
"https://uhl.site";`?  You serve uhl.site in https protocol and when 
http://uhl.site is requested, is redirected to https://uhl.site (as per the 
uhl.site.conf config)
   
   5.- Why you need `if ($scheme = http)`? If the request go to uhl.site port 
80, the request is allways http scheme, I think the if is not needed and the 
request must be inconditionally redirected to uhl.site port 443.
   
   6.- Why do you need `pagespeed ModPagespeedRespectXForwardedProto on;`? This 
directive is used when you have a ssl endpoint before the server:
   user -> https proxy/ssl endpont -> http server
   and if needed, because you have a ssl endoint, you need  `pagespeed 
RespectXForwardedProto on;` (for nginx. You have used the syntax for apache)
   
   7.- What means this `pagespeed Allow wildcard_spec;`? Here `wildcard_spec` 
here is a regular expresion, not the this literal text. By default (if these 
directive is NOT present) the value is `pagespeed Allow "*";` so maybe you 
don´t need it. See the examples 
[here](https://www.modpagespeed.com/doc/restricting_urls#disallow)
   
   8.- You have `pagespeed ModPagespeedFetchHttps enable;` this directive is 
for apache, for nginx is `pagespeed FetchHttps enable;` 
   
   9.- You say use Redis, but in the config files Memcached is configured, not 
Redis.
   
   10.- Why you need gzip enabled in .pagespeed. reosurces?
   
   
   


----------------------------------------------------------------
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


Reply via email to