[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1720: URL is not in cache

2021-01-24 Thread GitBox


Lofesa commented on issue #1720:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1720#issuecomment-766614357


   I have tested https://uhl.site in chrome browser and in webpagetest and 
can´t see any outgoing connection to www.pagespeed-com.com, only uhl.site and 
stats.uhl.site
   
https://www.webpagetest.org/result/210125_DiT4_a0658f20d9b5e3aa0f60ee1b9c939dd8/
   
   But you have some mistakes:
   1.- All css inlined before the meta og and tewitter. This make Facebook and 
twitter don´t read these labels, the css must be inlined after these labels.
   2.-You have 1 url (https://uhl.site) with 2 cannonicals (https://uhl.site 
and https://uhl.site/de). These is not the way to have multilingual pages. You 
need to explore the hreflang html labels.



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




[GitHub] [incubator-pagespeed-ngx] uhlhosting commented on issue #1720: URL is not in cache

2021-01-24 Thread GitBox


uhlhosting commented on issue #1720:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1720#issuecomment-766492829


   @Lofesa I am also testing CSP with pagespeed and it seems that there is some 
external ongoing connection to www.pagespeed-mod.com ! ? Any idea what would 
pagespeed need to connect to external sources?



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