from:"GitBox"

[GitHub] [incubator-pagespeed-ngx] Tiltanus commented on issue #1752: Problem with automatic installer.

2022-07-06 Thread GitBox



Tiltanus commented on issue #1752:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1752#issuecomment-1176039081

   @eilandert I saw your psol build for Ubuntu 22, thanks for your work!
   Please, can you make a tip how to build ngx_pagespeed from git sources for 
nginx 1.23.0? Automatic installer from repo fails on this issue too :c


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2092: Prioritize Critical CSS strips leading / from asset urls

2022-07-06 Thread GitBox



Lofesa commented on issue #2092:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2092#issuecomment-1175967170

   Have you tried to put absolutes url's?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1752: Problem with automatic installer.

2022-07-06 Thread GitBox



eilandert commented on issue #1752:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1752#issuecomment-1175883241

   nginx 1.23.0 comes with some api changes, the git repo has been fixed but I 
do not think the automatic installer is fetching the git repo but a .tar.gz 
instead, which obviously hasn't been updated


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Tiltanus commented on issue #1752: Problem with automatic installer.

2022-07-06 Thread GitBox



Tiltanus commented on issue #1752:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1752#issuecomment-1175870607

   I have the same issue. ngx_pagespeed version 1.13.35.2, nginx version 1.23.0


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] hunterpl opened a new issue, #1752: Problem with automatic installer.

2022-07-05 Thread GitBox



hunterpl opened a new issue, #1752:
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1752

   Hello guys,
   
   I have a problem with installing latest version of nginx, never had that 
issue before, all the time shown for me same error, never had the problem.
   
   [code]/root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc: In 
function ‘ngx_int_t 
net_instaweb::{anonymous}::ps_set_cache_control(ngx_http_request_t*, char*)’:
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:407:36: 
error: request for member ‘elts’ in 
‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, 
which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you 
meant to use ‘->’ ?)
 407 |   if (r->headers_out.cache_control.elts == NULL) {
 |^~~~
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:408:35: 
error: cannot convert ‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to 
‘ngx_array_t*’
 408 | ngx_int_t rc = ngx_array_init(>headers_out.cache_control, 
r->pool,
 |   ^
 |   |
 |   ngx_table_elt_t** {aka 
ngx_table_elt_s**}
   In file included from src/core/ngx_core.h:65,
from src/http/ngx_http.h:13,
from 
/root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.h:33,
from 
/root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:26:
   src/core/ngx_array.h:32:29: note:   initializing argument 1 of ‘ngx_int_t 
ngx_array_init(ngx_array_t*, ngx_pool_t*, ngx_uint_t, size_t)’
  32 | ngx_array_init(ngx_array_t *array, ngx_pool_t *pool, ngx_uint_t n, 
size_t size)
 |~^
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:415:22: 
error: cannot convert ‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to 
‘ngx_array_t*’
 415 |   ngx_array_push(>headers_out.cache_control));
 |  ^
 |  |
 |  ngx_table_elt_t** {aka ngx_table_elt_s**}
   In file included from src/core/ngx_core.h:65,
from src/http/ngx_http.h:13,
from 
/root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.h:33,
from 
/root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:26:
   src/core/ngx_array.h:27:35: note:   initializing argument 1 of ‘void* 
ngx_array_push(ngx_array_t*)’
  27 | void *ngx_array_push(ngx_array_t *a);
 |  ~^
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc: In 
function ‘bool 
net_instaweb::{anonymous}::ps_get_cache_control(ngx_http_request_t*, 
GoogleString*)’:
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:439:74: 
error: request for member ‘elts’ in 
‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, 
which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you 
meant to use ‘->’ ?)
 439 |   auto ccp = 
static_cast(r->headers_out.cache_control.elts);
 |  
^~~~
   /root/incubator-pagespeed-ngx-latest-stable/src/ngx_pagespeed.cc:444:59: 
error: request for member ‘nelts’ in 
‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, 
which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you 
meant to use ‘->’ ?)
 444 |   for (ngx_uint_t i = 0; i < r->headers_out.cache_control.nelts; 
i++) {
 |   ^
   make[1]: *** [objs/Makefile:1439: objs/addon/src/ngx_pagespeed.o] Error 1
   make[1]: Leaving directory '/root/nginx-1.23.0'
   make: *** [Makefile:10: build] Error 2
   Error: Failure running 'make', exiting.[/code]


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] azt3k opened a new issue, #2092: Prioritize Critical CSS strips leading / from asset urls

2022-07-04 Thread GitBox



azt3k opened a new issue, #2092:
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2092

   I have font face references in my css that look like this after optimisation:
   
   
`@font-face{font-family:helvetica-black;font-weight:400;font-style:normal;src:url(/resources/font/helvetica-black/5ba229ac-ba95-456c-9c0d-9e10cc8de8d6.eot);src:url(/resources/font/helvetica-black/5ba229ac-ba95-456c-9c0d-9e10cc8de8d6.eot?#iefix)
 format("eot") , 
url(/resources/font/helvetica-black/8ede94c2-af21-4381-92e8-e40c05f98715.woff2) 
format("woff2") , 
url(/resources/font/helvetica-black/4550aa51-8a1d-40f2-a8c3-13384b0bb1e2.woff) 
format("woff") , 
url(/resources/font/helvetica-black/e9b5da31-e566-4f36-a41f-31b1101a16b5.ttf) 
format("truetype")}`
   
   These are fine in the optimised css file, however when critical css is 
prioritised it removes the leading / from the urls which breaks them on sub 
pages:
   
   
`@font-face{font-family:helvetica-black;font-weight:400;font-style:normal;src:url(resources/font/helvetica-black/5ba229ac-ba95-456c-9c0d-9e10cc8de8d6.eot);src:url(resources/font/helvetica-black/5ba229ac-ba95-456c-9c0d-9e10cc8de8d6.eot?#iefix)
 format("eot") , 
url(resources/font/helvetica-black/8ede94c2-af21-4381-92e8-e40c05f98715.woff2) 
format("woff2") , 
url(resources/font/helvetica-black/4550aa51-8a1d-40f2-a8c3-13384b0bb1e2.woff) 
format("woff") , 
url(resources/font/helvetica-black/e9b5da31-e566-4f36-a41f-31b1101a16b5.ttf) 
format("truetype")}`
   
   trim urls is not enabled
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf closed issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-30 Thread GitBox



oschaaf closed issue #1749: incubator-pagespeed-ngx-module not compatible with 
nginx 1.23.0
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf merged pull request #1751: Getpagespeed nginx.1.23.0 fix

2022-06-30 Thread GitBox



oschaaf merged PR #1751:
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1751


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf commented on pull request #1750: Fixes for NGINX nginx 1.23.0

2022-06-30 Thread GitBox



oschaaf commented on PR #1750:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/pull/1750#issuecomment-1171078472

   closing this in favor of 
https://github.com/apache/incubator-pagespeed-ngx/pull/1751
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf closed pull request #1750: Fixes for NGINX nginx 1.23.0

2022-06-30 Thread GitBox



oschaaf closed pull request #1750: Fixes for NGINX nginx 1.23.0
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1750


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf commented on pull request #1751: Getpagespeed nginx.1.23.0 fix

2022-06-30 Thread GitBox



oschaaf commented on PR #1751:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/pull/1751#issuecomment-1171077874

   awesome   lgtm


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] dvershinin opened a new pull request, #1751: Getpagespeed nginx.1.23.0 fix

2022-06-30 Thread GitBox



dvershinin opened a new pull request, #1751:
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1751

   This pull request adds changes from #1750 and adds the necessary 
conditionals to support building against both stable and mainline NGINX 
branches.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-30 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1170999534

   Is there anyway to enable serialization and compression when using pagespeed 
and redis?
   In two days has filled my redis instance
   I have set redis for wordpress with igbinary and lz4 support so a redis DDBB 
of 1GB is typically now around 200MB
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



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

   > Thanks for the heads up, i've added that directive to the list.
   > 
   > Would you mind if i send you an email with the htpasswd credentials to my 
pagespeed console and give it a look to know if all caching mechanism are set 
up correctly?
   
   I have an email in the profile.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



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

   > what about this?
   > 
   > needs to be added at pagespeed as well?
   > 
   > ```
   > location ~* \.(eot|otf|ttf|woff|woff2)$ {
   >add_header Access-Control-Allow-Origin *;
   > }
   > ```
   
   Not. As far as pagespeed do nothing with these files (same with .svg and 
.ico files)


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1170213494

   what about this?
   
   needs to be added at pagespeed as well?
   
   location ~* \.(eot|otf|ttf|woff|woff2)$ {
  add_header Access-Control-Allow-Origin *;
   }


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1170035189

   Thanks for the heads up, i've added that directive to the list.
   
   Would you mind if i send you an email with the htpasswd credentials to my 
pagespeed console and give it a look to know if all caching mechanism are set 
up correctly?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



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

   Well . php stuff is not loaded by LoadFromFile
   
   And Pragma header I think is in the default headers pagespeed set.
   
   But in the statics files, think you need to add, for example, a  
x-content-type-options: nosniff, and you add in the location bloc
   
   add_header x-content-type-options nosniff;
   
   This header is not set in the optimized resources. If you wish to add it to 
a optimized resource, you need to use
   
   pagespeed AddResourceHeader "x-content-type-options" "nosniff";
   
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169887277

   This is what i typically have inside a server block for a particular domain
   
 location ~ \.php$ {
   includefastcgi_params;
   fastcgi_pass   unix:/run/php-fpm/www.sock;
   fastcgi_intercept_errors off;
   fastcgi_param  SCRIPT_FILENAME  
/var/www/wordpress$fastcgi_script_name;
   
   #NGINX FASTCGI-CACHE-HEADERS
   fastcgi_cache_bypass $skip_cache;
   fastcgi_no_cache $skip_cache;
   fastcgi_cache phpcache;
   fastcgi_cache_valid 200 301 302 7d;
   add_header X-FastCGI-Cache $upstream_cache_status;
   
   #SECURITY HEADERS
   add_header X-Frame-Options "SAMEORIGIN";
   add_header X-Content-Type-Options "nosniff";
   add_header Strict-Transport-Security "max-age=31536000; 
includeSubDomains; preload";
   add_header X-XSS-Protection "1; mode=block" always;
fastcgi_hide_header X-Powered-By;
   proxy_pass_header Server;
add_header Referrer-Policy "no-referrer-when-downgrade";
#add_header Content-Security-Policy "default-src 'self' https: data: 
'unsafe-inline' 'unsafe-eval';" always;
   }   
   
   
  
   location ~* 
\.(?:ico|css|js|gif|jpe?g|png|jpg|woff|woff2|ttf|mp4|mov|ogg)$ {
 expires 366d;
 #expires 1s;   
 add_header Pragma public;
 add_header Cache-Control "public, max-age=31536000";
 access_log off;
 log_not_found off;
   }


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



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

   @danidorado 
   
   > I had this directive set up inside the http block at nginx.conf file
   > pagespeed LoadFromFileCacheTtlMs 3153600;
   That's ok. I mentioned this because I don't see it in the config you posted. 
Only one apreciation, maybe  1 year is too much time.
   That's supposed to be the time pagespeed stores the optimized resource in 
their cache . 
   Forgot the LoadFromFile stuff for a momment. If you serve a resource 
with 1 week as cache-control header and is optimized by pagespeed, 2 things go 
on:
   1.- The cache-control for the optimized resource is changed to 1 year 
(unless you configured pagespeed to preserver the original header)
   3.- The original cache-control, 1 week in the example, is taken by pagespeed 
as the time the resource lives in their cache.
   
   > if i'm not wrong having that TTL set then I could skip doing this part you 
mentioned, right?
   > Or any other custom header (those added to request with the nginx 
add_header), with
   >pagespeed AddResourceHeader
   
   Ummm... Not. That only affect cache-control.  If you whish to add a custom 
header to the resource ( say strict-transport as example) this must be added 
with  `pagespeed AddResourceHeader`. 
   As far as the resource is loaded from disk and don't have any http headers, 
when the optimized resource is serverd it only have the default headers 
pagespeed put in the optimized resource, and strict-transport is not in these 
defaults, and don't have it even you set an add_header nginx directive.
   
   >pagespeed SslCertDirectory directory;
   >pagespeed SslCertFile file;
   
   You are rigth, don't need these directives as far as the request for an 
optimized resource not file based goes by http, not https.
   
   
   
   
   
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169715900

   @Lofesa
   
   I had this directive set up inside the http block at nginx.conf file
   pagespeed LoadFromFileCacheTtlMs 3153600;
   
   if i'm not wrong having that TTL set then I could skip doing this part you 
mentioned, right?
   _Or any other custom header (those added to request with the nginx 
add_header), with
   pagespeed AddResourceHeader_
   
   I read on the documentation pagespeed FetchHttps enable; is enabled by 
default but i've added to the set up at the http nginx block
   
   In relation to this two directives:
   pagespeed SslCertDirectory directory;
   pagespeed SslCertFile file;
   
   I'm exactly doing what you mentioned here
   
AWS ACM ->  ALB  -> EC2 -> NGINX
user -> 443 -> Proxy -> 80 -> Backend
   
   So i'm not installing certs directly on the instance, instead they are 
attached to the ALB through AWS console so i have that line that i guess 
resolves that
   pagespeed RespectXForwardedProto on;
   
   And btw, is anyone willing to spend 30-60 minutes on a call to review the 
integration and give some feedback. 
   -open to another suitable way to do this without taking too much time of 
your lives-
   I'm happy to pay for that time if neccessary.
   I have some doubts if i have properly set up the caching mechanisms and also 
related to the data shown at the pagespeed console panel.
   
   Thanks for letting me know if i'm doing it right.
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-29 Thread GitBox



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

   @danidorado 
   Normally I use some like
   
   `pagespeed Domain http*://*.surfcampmaldives.com;`
   
   This covers both versions, http and https and all subdomains
   By default pagespeed Domain only covers the http version
   
   Take care of pagespeed LoadFromFile.
   When you use it, there is no http request so resources have no http headers.
   If pagespeed don't see any cache-control header, then apply a default of 300 
(if I remember) so the optimized resource only lives these 300s in the 
pagespeed cache.
   
   You need to addç
   
   pagespeed LoadFromFileCacheTtlMs implicit_cache_ttl_in_milliseconds;
   
   Or any other custom header (those added to request with the nginx 
add_header), with
   
   pagespeed AddResourceHeader
   
   In the config you posted I can't see
   
   pagespeed FetchHttps enable;
   pagespeed SslCertDirectory directory;
   pagespeed SslCertFile file;
   
   https://www.modpagespeed.com/doc/https_support#configuring_ssl_certificates
   
   
   So maybe these are causing troubles.
   
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169249549

   This tree guys do the job.  https://surfcampmaldives.com/
   
   pagespeed MapOriginDomain localhost https://cdn.surfcampmaldives.com;
   pagespeed Domain surfcampmaldives.com;
   pagespeed LoadFromFile "https://cdn.surfcampmaldives.com; 
"/var/www/wordpress/";
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169247636

   It doesnt work, i already had that directive set up
   i'm trying with this: pagespeed MapOriginDomain localhost 
https://cdn.surfcampmaldives.com;
   
   
https://cdn.surfcampmaldives.com/wp-content/themes/uncode/library/js/init.js?ver=114949620'
 id='uncode-init-js'>

[GitHub] [incubator-pagespeed-ngx] jmarantz commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



jmarantz commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169241798

   You can try to authorize that domain with:
   ```
   pagespeed Domain https://cdn.surfcampmaldives.com;
   ```
   This will result in  the browser attempting to fetch something like 
`https://cdn.surfcampmaldives.com/wp-content/themes/uncode/library/js/init.js,qver=2122898389.pagespeed.HASH.jm.js`
   
   That should work if your CDN is set up to pull content from your nginx 
server where the URL would be decoded successfully.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1169238004

   
https://cdn.surfcampmaldives.com/wp-content/themes/uncode/library/js/init.js?ver=2122898389'
 id='uncode-init-js'>
   
   
   
   Any idea why it's not optimizing resources from the CDN?
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



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

   @jmarantz 
   Is a know trick when you have
   
   user -> 443 -> Proxy -> 80 -> Backend
   
   w/o pagespeed RespectXForwardedProto on; pagespeed respond with the protocol 
that the proxy request..
   
   Other software have similar issues with this, like Fail2ban.
   Fail2ban is a software to ban ip for bad actors and ir works reading log 
files, so must run in the backend server, but w/o apropiate headers, the log 
file only record the proxy ip.
   In this case headers like X-Real-IP and/or X-Forwarded-For must be inplace 
and then the real user ip gets recorden in the backend server.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] jmarantz commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



jmarantz commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168719936

   Nice! I should remember that for the next time this comes up :)
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168707271

   Found it!
   
   I knew it was related to the way the SSL works
   
   Adding this line unleashes pagespeed features
   pagespeed RespectXForwardedProto on;


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168695501

   
   
   
   --
     | 
   
     | 
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] jmarantz commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



jmarantz commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168689342

   Is it possible that when the HTML is rewritten, ngx_pagespeeed sees the URL 
of that page as something other than https://shopdiggers.com? Maybe 
www.shopdiggers.com?
   
   Other than that, your config looks right to me. It might also help to turn 
on debug:
   ```
   pagespeed EnableFilters debug;
   ```
   and that will annotate your HTML with more details explaining why rewriting 
is not happening as expected.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-28 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168503753

   It's compiled but i can't make js and css minification, probably images also.
   
   I'm getting this info message for almost all resources
   [Mon, 27 Jun 2022 21:52:58 GMT] [Info] [19134] No permission to rewrite 
'https://shopdiggers.com/wp-includes/blocks/navigation/view.min.js'
   
   Because i have an AWS ALB in front of the instance i do all the security 
layer at the ALB with AWS SSL certificates, so by then traffic goes under port 
80 inside the vpc
   
   I hope someone could through some light about what is going on
   
   This are my global settings
   
   pagespeed CreateSharedMemoryMetadataCache "/mnt/ramdisk/pagespeedcache/" 
2048000;
   pagespeed DefaultSharedMemoryCacheKB 2048000;
   pagespeed FileCacheSizeKb 2048000;
   pagespeed FileCacheCleanIntervalMs 3600;
   pagespeed FileCacheInodeLimit 50;
   pagespeed LRUCacheKbPerProcess 8192;
   pagespeed LRUCacheByteLimit 16384;
   pagespeed RedisServer "X";
   pagespeed RedisDatabaseIndex 5;
   pagespeed RedisReconnectionDelayMs 1;
   pagespeed RedisTimeoutUs 50;
   pagespeed RedisTTLSec 10;
   pagespeed ImplicitCacheTtlMs 3153600;
   pagespeed LoadFromFileCacheTtlMs 3153600;
   pagespeed ImageMaxRewritesAtOnce -1;
   pagespeed HttpCacheCompressionLevel 9;
   
   pagespeed StatisticsPath /ngx_pagespeed_statistics;
   pagespeed GlobalStatisticsPath /ngx_pagespeed_global_statistics;
   pagespeed MessagesPath /ngx_pagespeed_message;
   pagespeed ConsolePath /pagespeed_console;
   pagespeed AdminPath /pagespeed_admin;
   pagespeed GlobalAdminPath /pagespeed_global_admin;
   
   pagespeed IproMaxConcurrentRecordings 0;   
   pagespeed EnableCachePurge on;
   pagespeed PurgeMethod PURGE;
   pagespeed Statistics on;
   pagespeed StatisticsLogging on;
   pagespeed UsePerVhostStatistics on;
   pagespeed LogDir /var/log/pagespeed;
   pagespeed MessageBufferSize 20;
   
   pagespeed XHeaderValue "Optimized by Golden Gem";
   pagespeed on;
   pagespeed FileCachePath "/mnt/ramdisk/pagespeedcache/";
   pagespeed EnableFilters insert_dns_prefetch;
   pagespeed EnableFilters extend_cache;
   pagespeed EnableFilters strip_image_meta_data;
   pagespeed EnableFilters jpeg_subsampling;
   pagespeed EnableFilters hint_preload_subresources;
   pagespeed UseNativeFetcher on;
   pagespeed FetcherTimeoutMs 2000;
   pagespeed RewriteDeadlinePerFlushMs 5;
   resolver 8.8.8.8;
   
   
   
   This are my settings at the VHOST
   
   server {
   #listen  443 ssl http2;
   #listen  [::]:443 ssl http2;
   listen 80;
   server_name  [shopdiggers.com](http://shopdiggers.com/);
   root   /var/www/shopdiggers;
   
   index  index.php index.html index.htm;
   expires $expires;
   
   location / {
   try_files $uri $uri/ /index.php?$args;
   }
   
   # Needs to exist and be writable by nginx.  Use tmpfs for best 
performance.
   
   pagespeed RedisReconnectionDelayMs 500;
   pagespeed ImplicitCacheTtlMs 3153600;
   pagespeed LoadFromFileCacheTtlMs 3153600;
   
   # Ensure requests for pagespeed optimized resources go to the pagespeed 
handler
   # and no extraneous headers get set.
   location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
  add_header "" "";
   }
   
   location ~ "^/pagespeed_static/" { }
   location ~ "^/ngx_pagespeed_beacon$" { }
   
  pagespeed on;
  pagespeed MaxCombinedJsBytes 30;
  pagespeed MaxCombinedCssBytes -1;
  pagespeed MaxSegmentLength 1500;
  #pagespeed UseExperimentalJsMinifier on;
   
  pagespeed CombineAcrossPaths on;
  pagespeed FetchHttps 
enable,allow_self_signed,allow_unknown_certificate_authority,allow_certificate_not_yet_valid;
  pagespeed MapOriginDomain http://shopdiggers.com/ 
https://shopdiggers.com/;
  pagespeed Domain [shopdiggers.com](http://shopdiggers.com/);
  pagespeed LoadFromFile "https://shopdiggers.com/; "/var/www/shopdiggers/";
   
  pagespeed EnableFilters strip_image_color_profile;
  pagespeed EnableFilters 
move_css_to_head,move_css_above_scripts,flatten_css_imports;
  pagespeed EnableFilters rewrite_javascript,rewrite_css,rewrite_images;
  pagespeed EnableFilters combine_javascript,combine_css;
  pagespeed EnableFilters collapse_whitespace,elide_attributes;
  pagespeed EnableFilters remove_comments;
  pagespeed EnableFilters inline_google_font_css;
  pagespeed EnableFilters sprite_images;
  pagespeed EnableFilters extend_cache;
  pagespeed EnableFilters insert_dns_prefetch;
  pagespeed EnableFilters trim_urls;


-- 
This is an automated message from the Apache Git Service.
To

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox



eilandert commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1168029872

   ok, no sleep! fixed some things.
   
   Packages here: 
   http://edge.deb.myguard.nl:/pool/main/n/nginx/
   http://edge.deb.myguard.nl:/openssl3/pool/main/n/nginx/
   (building in progress right now. please note: edge.deb.myguard.nl is 
experimental)
   
   Docker here https://hub.docker.com/r/eilandert/nginx-modsecurity3-pagespeed
   (currently only the debian ones are with 1.23.0, when all packages are built 
ubuntu will follow tomorrow)
   
   Some documentation here:
   https://deb.myguard.nl/nginx-modules/
   
   Happy testing


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox



eilandert commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167881999

   Thanks @oschaaf , it compiles.
   
   However, my build fails thanks to openresty unpatched modules (with the same 
problem), so I have no packages at all to test. And I need those to run on my 
staging environments :-(


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167132866

   @oschaaf. I'm happy to give it a try on my servers, let me know how i can 
proceed


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-27 Thread GitBox



oschaaf commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1167015184

   @eilandert & @Lofesa -- 
https://github.com/apache/incubator-pagespeed-ngx/pull/1750 has an (untested) 
draft that aims to resolve this. It's still a draft, because we probably need 
some ifdef's to make the change conditional (I guess older nginx versions swill 
still need the old way of manipulating cache control headers). 
   Anyone willing to take that change for a spin? I'd be happy to hear if it 
works well. 


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf opened a new pull request, #1750: [DRAFT] Fixes for NGINX nginx 1.23.0

2022-06-27 Thread GitBox



oschaaf opened a new pull request, #1750:
URL: https://github.com/apache/incubator-pagespeed-ngx/pull/1750

   As of nginx 1.23 the cache control headers are re-implemented as al linked 
list.
   This reworks our code that manipulates this header to work with that.
   
   Still a draft: this change probably needs to be conditioned against nginx 
version
   with ifdefs, so older version can still be built as well. Hence, still a 
draft.
   
   Attempts to fix https://github.com/apache/incubator-pagespeed-ngx/issues/1749


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-26 Thread GitBox



eilandert commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1166679017

   @dvershinin, care to do a pull request for headers.patch? ;-)


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-23 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1164308050

   I hope this issue push an effort of developing a stable version again as the 
module for nginx has been in RC1 for years.
   
   The issue with the "double compression and mess with brotli and gzip" has 
been around since then i had to switch off pagespeed more than a year ago due 
those content enconding issues that were not allowing the browser to properly 
render CSS files even enabling this directive 
ModPagespeedHttpCacheCompressionLevel 0
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-23 Thread GitBox



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

   @oschaaf  Can take alook here?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] shigechika commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-23 Thread GitBox



shigechika commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1164144963

   I found "r->headers_out.cache_control.elts" in diff of 
nginx-1.22.0/src/http/modules/ngx_http_headers_filter_module.c and 
nginx-1.23.0/src/http/modules/ngx_http_headers_filter_module.c.
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] janjoosse commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-23 Thread GitBox



janjoosse commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1164095016

   I have the same issue


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-22 Thread GitBox



eilandert commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1163631854

   Changes with nginx 1.23.021 Jun 2022
   
   *) Change in internal API: now header lines are represented as linked
  lists.
   
   I suspect this is the issue here.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] danidorado commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-22 Thread GitBox



danidorado commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1163482943

   I have the same issue


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] luispabon commented on issue #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-22 Thread GitBox



luispabon commented on issue #1749:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1749#issuecomment-1162856051

   Facing the same issue. This project is abandonded though isnt' it? No 
commits for 2 years now


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] tomazurro commented on issue #1740: Failing to build as dynamic module

2022-06-19 Thread GitBox



tomazurro commented on issue #1740:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1740#issuecomment-1159662047

   the build error in your case is caused by the undefined references, which in 
turn are caused by an outdated build environment. I had the very same error 
with default gcc-Version 4.8.5 on centos 7 and solved it by updating to version 
7+ using slc:
   
   yum -y install centos-release-scl
   yum -y install devtoolset-7-gcc devtoolset-7-gcc-c++ devtoolset-7-binutils
   scl enable devtoolset-7 bash
   gcc -v  
   [resp. your build command]


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] icebluey opened a new issue, #1749: incubator-pagespeed-ngx-module not compatible with nginx 1.23.0

2022-06-18 Thread GitBox



icebluey opened a new issue, #1749:
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1749

   Nginx version: nginx 861f076eab72
   
   ```
   gcc -c -O2 -fexceptions -g -grecord-gcc-switches -pipe -Wall 
-Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS 
-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -m64 
-mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection 
-fcf-protection  -Wno-deprecated-declarations  -D_GLIBCXX_USE_CXX11_ABI=0 
--std=c++11  -I src/core -I src/event -I src/event/modules -I src/os/unix -I 
../modules/ngx_rtmp_module -I ../pcre2/src/ -I ../openssl/.openssl/include -I 
../zlib -I /usr/include/libxml2 -I objs -I src/http -I src/http/modules -I 
src/http/v2 -I src/http/modules/perl -I /usr/include -I 
../modules/ngx_pagespeed/psol/include -I 
../modules/ngx_pagespeed/psol/include/third_party/chromium/src -I 
../modules/ngx_pagespeed/psol/include/third_party/google-sparsehash/src/src -I 
../modules/ngx_pagespeed/psol/include/third_party/google-sparsehash/gen/arch/linux/x64/include
 -I ../modules/ngx_pagespeed/psol/include/third_party/grpc/src/include -I 
 ../modules/ngx_pagespeed/psol/include/third_party/protobuf/src/src -I 
../modules/ngx_pagespeed/psol/include/third_party/re2/src -I 
../modules/ngx_pagespeed/psol/include/out/Release/obj/gen -I 
../modules/ngx_pagespeed/psol/include/out/Release/obj/gen/protoc_out/instaweb 
-I ../modules/ngx_pagespeed/psol/include/third_party/apr/src/include -I 
../modules/ngx_pagespeed/psol/include/third_party/aprutil/src/include -I 
../modules/ngx_pagespeed/psol/include/third_party/apr/gen/arch/linux/x64/include
 -I 
../modules/ngx_pagespeed/psol/include/third_party/aprutil/gen/arch/linux/x64/include
 -I ../modules/ngx_pagespeed/psol/include/url -I src/mail -I src/stream \
   -o objs/addon/src/ngx_rewrite_driver_factory.o \
   ../modules/ngx_pagespeed/src/ngx_rewrite_driver_factory.cc
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc: In function ‘ngx_int_t 
net_instaweb::{anonymous}::ps_set_cache_control(ngx_http_request_t*, char*)’:
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:410:36: error: request for 
member ‘elts’ in 
‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, 
which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you 
meant to use ‘->’ ?)
 410 |   if (r->headers_out.cache_control.elts == NULL) {
 |^~~~
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:411:35: error: cannot convert 
‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to ‘ngx_array_t*’
 411 | ngx_int_t rc = ngx_array_init(>headers_out.cache_control, 
r->pool,
 |   ^
 |   |
 |   ngx_table_elt_t** {aka 
ngx_table_elt_s**}
   In file included from src/core/ngx_core.h:65,
from src/http/ngx_http.h:13,
from ../modules/ngx_pagespeed/src/ngx_pagespeed.h:36,
from ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:29:
   src/core/ngx_array.h:32:29: note:   initializing argument 1 of ‘ngx_int_t 
ngx_array_init(ngx_array_t*, ngx_pool_t*, ngx_uint_t, size_t)’
  32 | ngx_array_init(ngx_array_t *array, ngx_pool_t *pool, ngx_uint_t n, 
size_t size)
 |~^
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:418:22: error: cannot convert 
‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to ‘ngx_array_t*’
 418 |   ngx_array_push(>headers_out.cache_control));
 |  ^
 |  |
 |  ngx_table_elt_t** {aka ngx_table_elt_s**}
   In file included from src/core/ngx_core.h:65,
from src/http/ngx_http.h:13,
from ../modules/ngx_pagespeed/src/ngx_pagespeed.h:36,
from ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:29:
   src/core/ngx_array.h:27:35: note:   initializing argument 1 of ‘void* 
ngx_array_push(ngx_array_t*)’
  27 | void *ngx_array_push(ngx_array_t *a);
 |  ~^
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc: In function ‘bool 
net_instaweb::{anonymous}::ps_get_cache_control(ngx_http_request_t*, 
GoogleString*)’:
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:442:74: error: request for 
member ‘elts’ in 
‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, 
which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you 
meant to use ‘->’ ?)
 442 |   auto ccp = 
static_cast(r->headers_out.cache_control.elts);
 |  
^~~~
   ../modules/ngx_pagespeed/src/ngx_pagespeed.cc:447:59: error: request for 
member ‘nelts’ in

[GitHub] [incubator-pagespeed-ngx] knyzorg commented on issue #1715: Add gtag.js with Insert Google Analytics

2022-05-29 Thread GitBox



knyzorg commented on issue #1715:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1715#issuecomment-1140492687

   [Google Analytics 4 is our next-generation measurement solution, and it's 
replacing Universal Analytics. On July 1, 2023, standard Universal Analytics 
properties will stop processing new hits. If you still rely on Universal 
Analytics, we recommend that you [prepare to use Google Analytics 
4](https://support.google.com/analytics/answer/10759417) going 
forward.](https://support.google.com/analytics/answer/11583528?hl=en)


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] lavacano commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-05-27 Thread GitBox



lavacano commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1139788057

   > Hello @nemihome, try the latest master, recursive. git clone --recursive 
https://github.com/apache/incubator-pagespeed-mod.git
   > 
   > additionally, I uploaded precompiled psol binaries here: 
http://www.tiredofit.nl/psol-jammy.tar.xz (available: psol-bionic.tar.gz 
psol-focal.tar.gz psol-jammy.tar.xz psol-trusty.tar.gz psol-xenial.tar.gz)
   > 
   > and additionally, I'm offering debian and ubuntu packages (including 
jammy) here: https://deb.myguard.nl/nginx-modules/ (there is also a docker, for 
impish and bullseye)
   
   any hope for quic 
https://blog.cloudflare.com/experiment-with-http-3-using-nginx-and-quiche/ ?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] sajjadjafaribojd commented on issue #1748: Parsing JS and CSS error

2022-05-19 Thread GitBox



sajjadjafaribojd commented on issue #1748:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1748#issuecomment-1131479697

   I am using version 1.13.35.2-stable
   Nginx version : 1.20.1
   OS version: ubuntu 20.04.4 TLS
   I used the   `pagespeed RewriteLevel PassThrough;`  command to disable any 
filters.
   I also used your help and added the `pagespeed HttpCacheCompressionLevel 0;` 
option to the configurations and no changes were observed and we still have 
errors.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] epelc commented on issue #1971: rewrite_css filter minifies CSS Custom Properties

2022-05-18 Thread GitBox



epelc commented on issue #1971:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1971#issuecomment-1130587942

   @jmarantz oh I installed latest stable today.
   
   Ended up just working around by renaming all our css variables to be 
lowercase instead. ie `--myVariable` is now `--my-variable`. Not perfect as 
there is extra character but it works easily.
   
   You could disable mod_pagespeed for a specific file though as an alternative 
workaround. One thought was to define all your variables in a separate file 
that mod_pagespeed ignored.
   
   Was more of an issue figuring out the cause of everything breaking than 
actually fixing things.
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #1971: rewrite_css filter minifies CSS Custom Properties

2022-05-18 Thread GitBox



jmarantz commented on issue #1971:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1971#issuecomment-1130578758

   actually 1.14 is available. Not sure if that bug  is fixed.
   
   Can you just disallow the specific CSS file causing problems?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] epelc commented on issue #1971: rewrite_css filter minifies CSS Custom Properties

2022-05-18 Thread GitBox



epelc commented on issue #1971:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1971#issuecomment-1130550858

   Also just ran into this! Took a while to figure out but was basically 
breaking our entire page and there were no error messages or anything to go off 
of.
   
   Also still a bug on latest mod_pagespeed/1.13.35.2-0


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] jmarantz commented on issue #1748: Parsing JS and CSS error

2022-05-18 Thread GitBox



jmarantz commented on issue #1748:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1748#issuecomment-1129957520

   I assume you've tried also `pagespeed off;`? Your configuration has it doing 
basically nothing anyway, especially to CSS and JS, which makes me think the 
config is not really being applied to the request. For example, you might be 
serving CSS and JS from a different domain or `host` configuration than the 
pass-through that you showed.
   
   What version of pagespeed are you using?
   
   There's some control about how mod_pagespeed compresses resources as it 
fetches and caches them here: 
https://www.modpagespeed.com/doc/system#gzip_cache though with your settings I 
don't think that could matter.
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] sajjadjafaribojd commented on issue #1748: Parsing JS and CSS error

2022-05-18 Thread GitBox



sajjadjafaribojd commented on issue #1748:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1748#issuecomment-1129916755

   I also disabled all compression-related configurations
   All modules are also disabled.
   There are no additional configurations.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] sajjadjafaribojd opened a new issue, #1748: Parsing JS and CSS error

2022-05-18 Thread GitBox



sajjadjafaribojd opened a new issue, #1748:
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1748

   I'm getting this in JS and CSS console:
   SyntaxError: illegal character
   And when I open up the offending file, it displays this:
   ôےDTôzXEzزj{�Tئî†Xاُ|ےيMëےë—jWھمû€f;€¬êش8[ïk^
dk"چہژ•ى؟µïصپéu!،¢d¶n‡UœmZ‍�9 aپ§êقف‎؟_pخW
ة‏ےûûٍكTز7Q–£طوè;Gب½×ع{ڈQpk¸4FKْذAْہkح5×¾÷‍sn©»ھZnµZ@ڈ1û€،ح،£0
   Kz(u›€ئجfqüfüJ;® 
„ش‡eêë‏œکaNrU‎‚ْiµ²¹o±زؤ„‎¹÷أ^ںفzù—ôٹˆJً¢¦{I¥خ5ùm÷زق.Bد!ةYFھ•¤‎q±ك6cK„ےâں؟ف‰î·Owîّ
 ½PoŒٍرهôظ%ٍèفèsR°ل8دôB/¦¯«‡z؟ط‹U{dSî,[زc[•+»{®Xµhعمêiژk7è+ک>آق 
گ¥?ٌسڈ9[÷8[E3¾پH�‰نژ”I�ê€Bj¢R��@�4ز@C
   4ذ`گ™Xب‹,(P
   Tىrِ€S{ےڈ¼‎¹يأکu“2
   ´¾zArœ\¬Hp³oï*إ/`C 
v[d[d[fبM¯Q‹l4إVê°’،lب�2ہضعض9عض¶>ldغdکأV†µm†µ‹Œآ.2ٹيq@&چ"ُüû`>©^$¸ظ7=\¥ّlشn‹
a‹aث¹YشâM±U†:¬d(2ہ†°u†¶uژ¶µئ6و°•am›aي"£°‹ŒbغأeRs>‌ذے؟،د«'ôة 
ک‏.•عا°bهگüغ²لqmثف#‡؟"‡ق
   rّھ†«¢€ژٹr¸)بل¤r\ثل،( ‡b‘أ;y{‰,z´Fôƒگ€ùù#C*_dگ
   ùµDئ7~‏بˆإƒک:2|ًk‰
ü%rٹڈـw'¨[„„بiطد9±ْ"§تہ9ر¯%r–ََGخ[¾ب™(ہش‘“ˆ_Kن´à/‘}ن¾ë¶S³¹³ك9}¢نu
   ¨خُب:I2ض1'ٍ
   ëU91ع
   ق%—¶•_ط‹p´ے±I$‘نCRژن¤V%A‚¤ڈô¢W#™صJ6wي‍±ƒ‚ىG*NهA$ثUgھ
R3¤،zf 3³گEإ¼¸
   
g¬Cتا°ٹSyگ@ةr3U©™‚�زP=3گ™Yب¢b^ـ†¯³i؛خشے‌ش>¶‎fُI0فأ±L¹©.قfvï‍Jحط·{gَ<ëcF›ٌè(3نغخV£¶ع¦ٍ
 پ’غذ¨2Hحگ†è™پججBاٹںJ[إ7iظش‚Hr(T¤f
   HC
   ôج@ff!‹ٹں)`zWء'ïنèٌ™n[ےzى_ـ²Nدùشwµ}Ië\Y;h\N/*]¥?ƒَ&œdk/Œù1پسٍِç
س¾i=Œ*©7àt*غ¦-éN²ِم„س®}خضs¶R®¾m´چ+*غOs6•¯ûH¼‹¬پ�ہ�ِa*ك$گ·‍%
„!گ€bPŒؤ]"Xآ@ (إ°KKHC"   ه"،v‰dICiH$،\$”أ.‘,e( …T‹‚jط%ٹ¥
”،گ‚jQP
   »D±ہ�� €°�„a—��`�@v   
°´،پ64زP/êa—h–64ذ†FêEC=ىح2†ئ0ب@³h†]bXئ0ہhح°KثXأ"
يb،v‰eYأkXd،],´أ.±¬آ(@a¨€إRہbط%«0
   
P*`±°vIلy¤|OJ*سح|¶”Nق1@ز©´îENuµAA’:O;Zْïkھµض¦ùدضعةغm4^¥إ/رzھ«Yگµ†2O±}ھض‏ٍzb‡ç™ƒ‡غ‏>+çقçLبµO…
µ‌Y:  7yگ@r»œ,fھجLA€�i¨پ‍بج,dQq¬h
   إ7iا،
   zà ت 5UMTy 
   5ذ3گ™Yب¢â
   -wL¯,l\^Hدؤ7àt*«5+Fى.±ûhg´¼‡(إإc$Fp 
>آˆہBڈâb„1#¸aD`!ٹGq1آ‰\‚ڈ0"°إ£¸aŒؤ.ءGXˆâQ\Œ0Fb—à#Œ,Dٌ(.F#1‚KًF¢x#Œ‘ءُ€ہ‡`Bل،pa†!b\>Cک
   …C0qD5ƒˆتF،*{Pإ]¨ْ×AD…ً \ˆ[،Œ½¬/:h÷`—ّكإnغ÷15‰$’,êعإIٹ  ’>ز‹^
   ة¬–dSٌ@ر½‏»ص؟+y‎د*¦&‘d{ك}»¾[UٌھHگ é#½èصگجjI6×گâ>|0ِ±=ىٌ>¼u2سG;|ڑ–آٌ 
Gœ—u±ةق|غ!kسھٍِsحظhuخ‏ف•ـ‏س·غْ“µ¶‏>غ~†Dv�à‰€à~�سچ'يژYjاصسژï¥زٹ0رJLA;®{v|é¬°mV„fVb~ظqإ²مe…U²"²“اژkچ_+ى‰!ˆ•کv\%¬‹•ط/پ°?پس¾Jè{•XôoxP;پs؛J(s•طâ¯oً6پ¶Jhj•Xرïfگ2پ³±J¨a•ط؟/^0.پSJèX•X®oUذ)پَ¨J(P•طœ¯Lp%پ“¤JhG•X‹ïC!پ3
 J¨>•طy/;°پس›Jè5•Xho2Pپs—J(-•طV¯)ًپ“Jh$•XEï 
گ!cي¾ƒسچvBz†قڈ‚،÷&‘YhڈO(Eë÷œK´قOJ„قِ 
·‘6HفX_8v8ه§شâL‰}œyq%م0‰§b—ZpًإSq…«vƒ½Iتئسp‎خ°ّ¸v‎ء%œژq€م4>×m8†âg`ï±N¤ّûئ¾é±oطwطgv8ىھàqrإڈN!م¦ف¬NS0pDة>~ٍض¼هx+ًVâآ[ہ[چ/,ّقF`ىل\“'p‹<~aـî6¸ي¾«<×8ؤنyw-Nزطأءآ½;àëg|'ژN‏¸ےC
cہےلq_=à„“غ
   8ہc'lىل’ڈا)ڈي]ّçï‹‎وة}…C;>€س~ہw   
ç=v={œ[َ~œ·8W|çsü†8ڈ8g¼اûـˆC;êé`^غüتٹebف¸bC
   ث•IG«k‍,ƒںJ‚Hu¸*_S‏‰ژث?‌8ےï>هص”¦´.«غ}|c$ث/¼r}n]×؛|ù]SKJٌo‹أڈـµڈلmط
…œکOھv*p‡س—ي\غ©z¶§aکèJ(ـèت_”د±أûnêع”c^ؤ6ç“ّزس؟'Sakd'ضA£‡ے•N‏7,ô‍>-ے+ë“م~0²‰Y‏Sڈهٹ<+ًم+6ںµeO–\—£iHi‌$“سzcêأ«²،¥?_rس10ل:Rڈأ2خوة›û©„إک-رçکY‏&‡?ïےةLز©[75±وà¦“MM،“سsںuo<'÷6Aشطڑ+sص‡|y
 ْàv¸-رCظ.·ٍJ=كl¹ز؛:é‍S?)´gN%h8.ضtھ?يحkûyrV„Q»
   N©F×
£b²»…m'C‌ي—âأùکsًهخ’gT~ي¹يسءüظَjEvٌ‘¤`!fô£M£*m‍ïY´Gj–P[ةڑ–ژں.ٍ¥Bˆyث÷é—z8خ[1ُ>ُ�yںx€؛ڈفJشj>O”²ِں½Hm؟éZن]Aô‎أIF؛Œéµ‘‏4Lï‡؛1è´”5jlز¯»zٍ9V1ا'ّةثذآQ‏ضOً“ُط؛îI^صVy6قdط-oi]HXڈlةھٍçش·لT—]ب¹³ïD[³؟Ov"Ugًm«سي“؛}
صّ!نrQ¹ّ»ذدِيپ†،îHzصyگkYôA'«a·‰·VB6j÷×éRي‰‡/ثYد³ك5Fg�كu:ê)h³_ة
   َMت/cڑ]¶؛ُli ز®&8=–‡ؤم!ّ³¬`^¥{لHىمu~¢ًRآ¬گg[
   ذآfNLگب¤D÷,“%س§نƒق$¹Mœwك4Mûھ,N‏w؟َ³tش
   
¶×#بvزfi±شُس‹wW†ƒS?êْْ’qa®ب©çoك§صشWTà³اظن\ز‍ـ÷—~±8شô»–ç&¶b8Kّ^…¶ئèRUُ]Cذ¢َ§Blé•ل|ة†¬¸cç'ک‌[؟’VC>†\T’gf÷;؛JlI5U~×uH1¶K(مف–$à>0چئ‹(%ُP²©€XذE¢؟4jàKكء‹ح¾گ´dؤEصG,46غژîc‏àkخ·¦‹Qé@mO€‎×ًju÷Nصن57b)û�تqہ.j¹Vے¼§‹¦‘ؤçّ‏û<مغثF/GزŒ‎~Wh2®ü.ؤNًثb‘¯?.ç1;}¨پّ*l8"
÷€„`قXإ|¶ù}tتض،ژˆطجï ³jvwsژ°
   aکu§`ہ:هˆىزwِî@O^P@-F[uسéڑ·%طbe=üY{w@neLـ¦n7’o”«oƒ› 
*OY™لˆ„XùُƒM.v¢ڑjûhh”•ہc2%ف+pؤŒtوDہ0c÷aث‡'Mn–g*�چs-إٍ¯%‡\¬ù/ُbدمj
   و¯eD‡وzذا–b[rّXَ£اہ?±)ë^¸uc¢;c*D-ةAظزم£ىس¹‘ëc¥¥
   .–î“0è¨7,W8·“@›ٌ_59‏"َأژز~:ْز²ّ!d³ë¤ 
âگ›D--¨ض‌Vپnه/û·aء?]>ü«yzû¼¾)îآS:ب&ک@o|or†ں½عْڈWLèF0ق· 
ڈo;£¨Sس±\9-‘xû؛'à]ظmnp‡آظrت¤xévu¦.!²r/63ù‘*îƒâo/W؛كè)l¦}$§Yb{hإذضعث¸ 
°پ¤7Yپƒ¥ˆسه¦V›ھثنjةtù½زجل¦Vجأfpت•´9îکم`,ئ[کEb"m'إQ}ُ5JٹLّة³û³Cَ؟çk@‰¼ëê
   
>|ـG=«؟[9#â°¦‡:5نؤjِè¯ٍâu}tu)طüٍZ‏ُ”ü–Lâ~p?v7X5إ¨%™‌#ءq#>Gح‹êXح±*‏Hf>$%ˆإچ3تإè6ê/=‚†¹ˆ»;أ°yi#QEWئêHu·'\_ë´i90à…”x`†9ZIإyں,EVض2;+Œ¼4آGg³[لہ1àدت1?¾¥²4
   ص¸‌فjM÷سئMهN¹ء_ٹذ`nIpa€™i8C\z’&€
   
!é‡گغJ4ىضI·qت?ڈxN>gں“دـ3û¼ئçزïًjجJ'’2j¼2صخ){و@+{-L™’�°ŒِL¦¬(k2†„أژ†PxآD.…ّ!
   W[ھzآdCلco?
   
   My Condifuration:
   pagespeed FileCachePath "/var/cache/ngx_pagespeed/";
   pagespeed on;
   pagespeed RewriteLevel PassThrough;
   pagespeed ForbidAllDisabledFilters true;
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub

[GitHub] [incubator-pagespeed-ngx] leonardocxavier commented on issue #1533: configure: error: module ngx_pagespeed requires the pagespeed optimization library.

2022-05-09 Thread GitBox



leonardocxavier commented on issue #1533:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1533#issuecomment-1121603275

   not work on debian 11


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2088: Changing filenames with webp: does it impact images SEO?

2022-05-07 Thread GitBox



Lofesa commented on issue #2088:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2088#issuecomment-1120258859

   Pagespeed serve rewrited images whit a link rel canonical header, so if 
google do their work, these images don't be indexed but the canonical.
   Anyway, images url don't change unles the image change.
   One jpg image, depending on enabled filters, generate some pagespeed 
rewirted urls (if the images get resized or converte to webp...) but unless the 
image changes, thats urls become allways the same.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] oschaaf closed issue #2091: sdsdsd

2022-05-03 Thread GitBox



oschaaf closed issue #2091: sdsdsd
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2091


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] vndroid commented on issue #1740: Failing to build as dynamic module

2022-04-21 Thread GitBox



vndroid commented on issue #1740:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1740#issuecomment-1104794873

   > With some fiddling this produces an nginx binary for me:
   > 
   > ```
   > #!/bin/bash

 
   > set -x
   > NPS_VERSION=1.14.33.1-RC1
   > NPS_RELEASE_NUMBER=1.14.36.1
   > PS_NGX_EXTRA_FLAGS=--with-cc-opt=-Wno-error
   > 
   > wget -O- 
https://github.com/apache/incubator-pagespeed-ngx/archive/refs/tags/v${NPS_VERSION}.tar.gz
 | tar -xz
   > nps_dir=$(realpath $(find . -name "*pagespeed-ngx-${NPS_VERSION}" -type d))
   > cd "$nps_dir"
   > 
psol_url=https://dl.google.com/dl/page-speed/psol/${NPS_RELEASE_NUMBER}.tar.gz
   > [ -e scripts/format_binary_url.sh ] && 
psol_url=$(scripts/format_binary_url.sh PSOL_BINARY_URL)
   > wget -O- ${psol_url} | tar -xz  # extracts to psol/

 
   > 
   > NGINX_VERSION=1.18.0
   > wget -O- http://nginx.org/download/nginx-${NGINX_VERSION}.tar.gz | tar -xz
   > cd nginx-${NGINX_VERSION}/
   > ./configure --add-module=$nps_dir ${PS_NGX_EXTRA_FLAGS}
   >  make -j 80
   > ```
   > 
   > Notes:
   > 
   > * During the release process, I didn't go ahead and release ngx_pagespeed 
along with mod_pagespeed. This is because that would need ngx_pagespeed to go 
through some more compliance checking to ensure it is on par with ASF standards.
   > * I did however go as far as tagging a release candidate. Unfortunately I 
made a typo (1.14.33.1-RC1 ought to have been 1.14.36.1-RC1). This is what we 
use above.
   >   I think it would be OK for us to update the documentation with a script 
similar to what I am posting here, as long as we add a disclaimer/warning about 
this being a WIP from an ASF policy PoV.
   
   According to this, I still fail to build on Alpine 3.15 (alpine:3.15).
   
   I already install [libuuid] and [util-linux-dev] (contains uuid/uuid.h and 
libuuid.so) and other depends.
   
   The compilation parameters use the parameters of the nginx official alpine 
container.
   
   ```
   ./configure --prefix=/etc/nginx \
   --sbin-path=/usr/sbin/nginx \
   --modules-path=/usr/lib/nginx/modules \
   --conf-path=/etc/nginx/nginx.conf \
   --error-log-path=/var/log/nginx/error.log \
   --http-log-path=/var/log/nginx/access.log \
   --pid-path=/var/run/nginx.pid \
   --lock-path=/var/run/nginx.lock \
   --http-client-body-temp-path=/var/cache/nginx/client_temp \
   --http-proxy-temp-path=/var/cache/nginx/proxy_temp \
   --http-fastcgi-temp-path=/var/cache/nginx/fastcgi_temp \
   --http-uwsgi-temp-path=/var/cache/nginx/uwsgi_temp \
   --http-scgi-temp-path=/var/cache/nginx/scgi_temp \
   --with-perl_modules_path=/usr/lib/perl5/vendor_perl \
   --user=nginx \
   --group=nginx \
   --with-compat \
   --with-file-aio \
   --with-threads \
   --with-http_addition_module \
   --with-http_auth_request_module \
   --with-http_dav_module \
   --with-http_flv_module \
   --with-http_gunzip_module \
   --with-http_gzip_static_module \
   --with-http_mp4_module \
   --with-http_random_index_module \
   --with-http_realip_module \
   --with-http_secure_link_module \
   --with-http_slice_module \
   --with-http_ssl_module \
   --with-http_stub_status_module \
   --with-http_sub_module \
   --with-http_v2_module --with-mail \
   --with-mail_ssl_module --with-stream \
   --with-stream_realip_module \
   --with-stream_ssl_module \
   --with-stream_ssl_preread_module \
   --add-dynamic-module=/usr/src/incubator-pagespeed-ngx-1.14.33.1-RC1 \
   --with-cc-opt='-Os -fomit-frame-pointer -g' \
   --with-ld-opt=-Wl,--as-needed,-O1,--sort-common
   ```
   
   The lib already exists.
   
   ```bash
   # find /usr/src/ -name pagespeed_automatic.a
   
/usr/src/incubator-pagespeed-ngx-1.14.33.1-RC1/psol/lib/Release/linux/x64/pagespeed_automatic.a
   ```
   
   Error log  `/usr/src/nginx-1.20.2/objs/autoconf.err`
   
   ```
   head -n 500 /usr/src/nginx-1.20.2/objs/autoconf.err
   
   
   
   checking for C compiler
   
   
   
   checking for gcc -pipe switch
   
   
   
   checking for --with-ld-opt="-Wl,--as-needed,-O1,--sort-common"
   
   
   
   checking for -Wl,-E switch
   
   
   
   checking for gcc builtin atomic

[GitHub] [incubator-pagespeed-ngx] knyzorg commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-04-18 Thread GitBox



knyzorg commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1101711276

   My way of resolving this was to patch Pagespeed, compile it on the working 
version of glibc, and package the result as a debian package.
   
   I have automated the process here: https://github.com/knyzorg/nginx-pagespeed
   
   And host the resulting debian files both in the repo and on my personal 
server for public use.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-04-18 Thread GitBox



eilandert commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1101698266

   Hi, Thanks for your feedback. 
   I have an ubuntu ppa on 
https://launchpad.net/~eilander/+archive/ubuntu/nginx which may be more 
suitable for your needs. 
   On my own repository I need to make a NGINX-only tree, but have yet to 
figure out how to do that in a sane way. I will look into it sometime soon.
   
   I am happy that my jammy-psol works for you and probably others
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] pablo-silveira commented on issue #1747: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-18 Thread GitBox



pablo-silveira commented on issue #1747:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1747#issuecomment-1101358892

   Thank you all! 


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] nemihome commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-04-17 Thread GitBox



nemihome commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1100845213

   Hello @eilandert,
   
   thank you. I tried your repository. Well Nginx is working with pagespeed but 
that changes half of the ubuntu 22.04 software to non standard. I don't like 
that much because if there are problems showing up like e.g.
   
   "Note that the deactivation of "ssh-rsa" signatures does not necessarily
   require cessation of use for RSA keys. In the SSH protocol, keys may be
   capable of signing using multiple algorithms. In particular, "ssh-rsa"
   keys are capable of signing using "rsa-sha2-256" (RSA/SHA256),
   "rsa-sha2-512" (RSA/SHA512) and "ssh-rsa" (RSA/SHA1). Only the last of
   these is being turned off by default."
   
   There is always the question if this is resulting from changes like + 
openssl3 and so forth or standard. Is there a way just to get the nginx part of 
your packages automatically?
   
   All this updates are not on my list of things I would like to have (I really 
would prefer just nginx in the package):
   
   The following packages will be upgraded:
 apache2 apache2-bin apache2-data apache2-utils libjemalloc2 liblmdb0
 libluajit-5.1-2 libluajit-5.1-common liblzf1 libpcre2-16-0 libpcre2-32-0
 libunbound8 lua-cjson myguard openssh-client openssh-server
 openssh-sftp-server postfix redis redis-server redis-tools ssh zstd


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] oschaaf commented on issue #2090: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-15 Thread GitBox



oschaaf commented on issue #2090:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2090#issuecomment-1100016261

   Should be fixed!


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] oschaaf closed issue #2090: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-15 Thread GitBox



oschaaf closed issue #2090: Failed to install nginx pagespeed module due SSL 
expired Certificate
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2090


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf commented on issue #1747: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-15 Thread GitBox



oschaaf commented on issue #1747:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1747#issuecomment-1100015855

   Should be fixed!


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] oschaaf closed issue #1747: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-15 Thread GitBox



oschaaf closed issue #1747: Failed to install nginx pagespeed module due SSL 
expired Certificate
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1747


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1747: Failed to install nginx pagespeed module due SSL expired Certificate

2022-04-15 Thread GitBox



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

   Hi @pablo-silveira 
   
   Noticed the same yesterday and send a message to @oschaaf and tell me maybe 
today the certificate will be updated.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2089: High TTFB with PageSpeed

2022-04-15 Thread GitBox



Lofesa commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1099985385

   hi @Ungov77 
   Take account that your filesystem cache  was near of 40GB, if I remember +30 
(Yes, I´m Longinos in the goolge group email list)
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-14 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1099694906

   Hi,
   
   The config file aas in /etc/sysconfig/memcached
   
   What size should we increase it to? We have 64gb of ram on this server 
   I increased it from 64 to 6400 for now


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] pablo-silveira opened a new issue, #1747: Failed to build nginx pagespeed module due SSL expired Certificate

2022-04-14 Thread GitBox



pablo-silveira opened a new issue, #1747:
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1747

   I noticed that the certificate SSL of https://ngxpagespeed.com/ expired on 
April 13th.
   So I receiving an error when executing `bash <(curl -k -f -L -sS 
https://ngxpagespeed.com/install)` .
   
   Do you have any estimated to renewal this certificate or some workaround to 
solve this issue?
   
   Thanks!


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] pablo-silveira opened a new issue, #2090: Failed to build nginx pagespeed module due SSL expired Certificate

2022-04-14 Thread GitBox



pablo-silveira opened a new issue, #2090:
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2090

   I noticed that the certificate SSL of  https://ngxpagespeed.com expired on 
April 13th.
   So I receiving an error when executing `bash <(curl -k -f -L -sS 
https://ngxpagespeed.com/install)` .
   
   Do you have any estimated to renewal this certificate or some workaround to 
solve this issue?
   
   Thanks!


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2089: High TTFB with PageSpeed

2022-04-14 Thread GitBox



Lofesa commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1099401496

   Hi @Ungov77 @jmarantz 
   
   Take a look on this:
   
   https://www.keinegotterkeinemeister.de/pagespeed_admin/cache#physical_cache
   
   See evictions and current objects stored:
   
   curr_items:53457
   evictions: 25654735
   
   I think is a big amount of evistions.
   
   You are using only a 64Mb memcache database
   
   bytes: 63936177
   
   
   In the config file for memcached (normaly /etc7memcached.conf or 
/etc/system/memcached.conf) you can increase the size of RAM memcached uses:
   
   
   change `m 64` whit the size you will use.


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-13 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1098556400

   Hi,
   
   Makes sense. I didn't understand how it was faster with Memcached failing lol
   
   I don't think the rewrite deadline is being overwritten. I have not modified 
any conf file, the entire config is in Plesk GUI except for  
ModPagespeedCreateSharedMemoryMetadataCache
   
   I have looked at the histograms but I'm not sure how to interpret the data
   
![image](https://user-images.githubusercontent.com/2366002/163282842-2fe8cb45-0e27-4301-871e-b18762fc5554.png)
   
   You can check the admin section, I put it public temporarily again
   https://www.keinegotterkeinemeister.de/pagespeed_admin/


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-13 Thread GitBox



jmarantz commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1098084824

   It's possible the reason that TTFB was good for a while is that the 
memcached code is good at failing-fast if it looks like memcached is not 
working.
   
   I can think of three reasons generally that PageSpeed would make TTFB 
noticeably worse:
* HTML parsing overhead: Running with ?PageSpeedFilters=+debug shows us 
that parsing overhead is less than 6ms so I don't think it's that.
* Rewrite Deadlines: in the config you pasted above it looks like that's 
set to 20ms which should be fine. But it might be worth looking at the config 
again to see if anything deadline-related is overridden
* General overhead and queuing on your server.  PageSpeed keeps a limited 
number of threads for each process, with work queues for each thread, and at 
some scale the queue-depths might start growing. There are histograms for that 
on /pagespeed_admin.
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1097268729

   I see this error in /mod_pagespeed_statistics?cache#physical_cache
   `Error getting memcached server status for localhost:11211`
   
   I temporarily re-enabled public statistics on both domains if you want to 
check


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1097249240

   I tried CTRL+F5 many many times but I didn't see any webp images in the 
code. As soon as I turn off Memcached, I see webp images after 2 refreshes. We 
let Memcached be enabled for 18 hours and didn't see any webp images on the 
homepage.
   
   Memcached disabled, many webp images in the code:
   view-source:https://www.keinegotterkeinemeister.de/
   
   Memcached enabled for 18 hours with zero webp in the code:
   view-source:https://www.ni-dieu-ni-maitre.com/


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



jmarantz commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1097233197

   My first suspicion is that they just haven't been (re)converted into webp 
*yet*.  By switching to memcached you effectively are flushing your cache. As 
you may have noticed, PageSpeed optimizes resources in the background when it 
learns they need to be optimized, but may serve the first few requests 
unoptimized. Is it possible that if you refresh your page a few times the 
images will start coming in as webp?
   
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1097225781

   Thanks, we disabled defer_javascript and some other filters
   
   However today we noticed that images are not getting converted to webp 
anymore after switching to Memcached
   When commenting out Memcached in the config, the images are converted again 
to webp


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



jmarantz commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1096667259

   One more thing -- you probably should control access to /pagespeed_admin and 
/mod_pagespeed_statistics with allow/deny directives per 
https://www.modpagespeed.com/doc/admin#handlers


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-12 Thread GitBox



jmarantz commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1096614478

   The shared-memory metadata cache does not get populated with images. It
   just gets populated with metadata mapping the original image name and
   context into which it is to be rendered into an optimized resource name.
   The images are stored in memcached in your configuration. You can see the
   hierarchy pictured here:
   
   https://www.keinegotterkeinemeister.de/pagespeed_admin/cache#cache_struct
   
   It look like you've got Prioritize Critical CSS enabled. I haven't
   thought about this space for quite some time but I don't think we have much
   else in PageSpeed I'd recommend. I"d try disabling defer_javascript -- it's
   possible it's doing more harm than good to that metric.
   
   Good luck!
   
   
   
   
   On Mon, Apr 11, 2022 at 11:52 PM Ungov77 ***@***.***> wrote:
   
   > There could be an IO issue because according to the Plesk guide, the first
   > test I ran should get a speed of 142 MB/s but I got only 183 *kB*/s
   >
   > Memcached is now installed but I'm not sure how to check if it's correctly
   > configured
   > https://www.keinegotterkeinemeister.de/phpinfo.php
   > I enabled statistics and I can see some data related to Memcached so it
   > seems to be working:
   > https://www.keinegotterkeinemeister.de/mod_pagespeed_statistics?memcached
   >
   > ModPagespeedMemcachedServers localhost:11211
   > ModPagespeedCreateSharedMemoryMetadataCache "/var/cache/mod_pagespeed/" 
51200
   >
   > Is 51200 enough? We have plenty of memory on the server and the websites
   > are big with thousands of images
   >
   > It seems to have lowered TTFB, great! Now it is almost the same as with
   > Pagespeed disabled. Thanks!
   >
   > Anything suggestions to improve LCP?
   >
   > —
   > Reply to this email directly, view it on GitHub
   > 
,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you commented.Message ID:
   > ***@***.***>
   >
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-11 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1095951580

   There could be an IO issue because according to the Plesk guide, the first 
test I ran should get a speed of 142 MB/s but I got only 183 **kB**/s
   
   Memcached is now installed but I'm not sure how to check if it's correctly 
configured
   https://www.keinegotterkeinemeister.de/phpinfo.php
   I enabled statistics and I can see some data related to Memcached so it 
seems to be working:
   https://www.keinegotterkeinemeister.de/mod_pagespeed_statistics?memcached
   
   ```
   ModPagespeedMemcachedServers localhost:11211
   ModPagespeedCreateSharedMemoryMetadataCache "/var/cache/mod_pagespeed/" 51200
   ```
   Is 51200 enough? We have plenty of memory on the server and the websites are 
big with thousands of images
   
   
   It seems to have lowered TTFB, great! Now it is almost the same as with 
Pagespeed disabled. Thanks!
   
   Anything suggestions to improve LCP?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-11 Thread GitBox



jmarantz commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1095856844

   I was suspicious there may be spinning-disk latency but as you have an SSD 
I'm not sure. Nevertheless it might be worth trying redis/memcached, as we 
never really tuned the file-cache for SSD.
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #2089: High TTFB with PageSpeed

2022-04-11 Thread GitBox



Ungov77 commented on issue #2089:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1095809062

   Hi, thanks for the reply
   
   The server is a dedicated server in OVH Datacenter in Beauharnois, Quebec, 
Canada
   Intel(R) Xeon(R) CPU D-1541 @ 2.10GHz (16 core(s)) - 2499 MHz
   64 GB Ram
   NVME SSD
   1GBPS network
   
   CPU load is always under 25% https://prnt.sc/Xte_RHbEGSgR
   
   We also integrated a benchmark into the website to calculate the page load 
speed of the different PHP modules and calculate SQL queries execution time. 
You can see it on the bottom of the page when adding ?benchmark=1 to the URL
   
https://www.keinegotterkeinemeister.de/tshirt-sabotage-black-cat-D01004174288P0210/?benchmark=1
   
   Currently, it takes 0.2110 seconds to generate the page with PHP on the 
first page load. But out of this 0.21 seconds, 0.1865 is due to calling an 
external geolocation API. This is just because Cloudflare is currently disabled 
for our testing phase. When Cloudflare is enabled, we get the geolocation from 
$_SERVER["HTTP_CF_IPCOUNTRY"] so we avoid calling the external API and page 
load time goes down to around 0.01s
   
   TTFB is not the only problem, we also have high LCP and it throws warnings 
in Google Search Console: https://prnt.sc/2PjB74ekUgdR
   Our search ranking dropped a lot and we lost 50% of our search traffic since 
June 2021 and we suspect it could be partly due to that
   
   How can I confirm whether the disk is I/O bound or not? I followed Plesk 
guide and here are the results:
   
https://support.plesk.com/hc/en-us/articles/360006437794-Plesk-for-Linux-server-is-running-slow-too-much-IO-interactivity
   `[root@ns508279 ~]# dd if=/dev/zero of=/root/testfile bs=512 count=5000 
oflag=dir   
   ect
   5000+0 records in
   5000+0 records out
   256 bytes (2.6 MB, 2.4 MiB) copied, 13.634 s, 188 kB/s
   [root@ns508279 ~]# dd if=/dev/zero of=/root/testfile bs=512 count=5000 
oflag=direct
   5000+0 records in
   5000+0 records out
   256 bytes (2.6 MB, 2.4 MiB) copied, 14.1703 s, 181 kB/s
   [root@ns508279 ~]# dd if=/dev/zero of=/root/testfile bs=512 count=5000 
oflag=direct
   5000+0 records in
   5000+0 records out
   256 bytes (2.6 MB, 2.4 MiB) copied, 14.0026 s, 183 kB/s
   `
   This seems pretty slow. Could it be because we run pagespeed from too many 
domains? 
   
   Latency seems fine:
   ```
   [root@ns508279 ~]# dd if=/dev/zero of=/root/testfile bs=1024M count=1 
oflag=direct
   1+0 records in
   1+0 records out
   1073741824 bytes (1.1 GB, 1.0 GiB) copied, 2.05548 s, 522 MB/s
   ```
   
   however iotop looks perfectly fine. Monitored it for around 10 minutes and 
IO never went above 1%
   https://prnt.sc/F65mP1MkTeBC
   
   also no high %WA with top:
   https://prnt.sc/Oe1hhgcqrNDC


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #2089: High TTFB with PageSpeed

2022-04-11 Thread GitBox

jmarantz commented on issue #2089:
URL:
https://github.com/apache/incubator-pagespeed-mod/issues/2089#issuecomment-1095540226

Thanks for the data including the config; it's very helpful.

It does look like the performance is pretty good even from the US, despite
the TTFB. I assume the server is in Germany with no US CDN.

Of course TTFB will never get faster with PageSpeed, only slower. The
question is whether the optimizations it makes during that time provide a
better user experience.

I visited
view-source:https://www.keinegotterkeinemeister.de/tshirt-sabotage-black-cat-D01004174288P0210/?PageSpeedFilters=+debug
to see what timing info PageSpeed injected at the end, and found this:
```
#NumFlushes0
#EndDocument after 5906us
#Total Parse duration 5685us
#Total Render duration 411563us
#Total Idle duration 221us
No critical images detected.
The following filters were disabled for this request:
CriticalSelectorFilter: No critical selector info in cache
DelayImages
```

So it looks like your server spent 5.7 seconds in PageSpeed parsing your
HTML -- that's mostly CPU bound. Then it spent 452ms rendering it -- that may
be a combination of image optimization, network operations, waiting for
asynchronous optimizations and URL fetches (depending on the cache setup) That
leaves 1100-270= 378ms unaccounted for.

I'm guessing that most of this comes from the file cache. It's possible your
disk is I/O bound. My main suggestion is to try deploying memcached or redis as
an asynchronous cache. That should be faster than the file-cache.

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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 opened a new issue, #2089: High TTFB with PageSpeed

2022-04-11 Thread GitBox



Ungov77 opened a new issue, #2089:
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2089

   Hi,
   
   TTFB is much higher when PageSpeed is enabled and I don't understand why
   
   264ms TTFB with PageSpeed=off
   https://speedvitals.com/report/www.keinegotterkeinemeister.de/eEuqEG/
   
   1100ms TTFB with PageSpeed ON
   https://speedvitals.com/report/www.keinegotterkeinemeister.de/tg1zwa/
   
   Pagespeed config:
   https://pastebin.com/fLqVcP1M


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Ungov77 commented on issue #1293: Pagespeed increases Time To First Byte

2022-04-10 Thread GitBox



Ungov77 commented on issue #1293:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1293#issuecomment-1094557764

   Same problem here
   
   264ms TTFB with PageSpeed=off
   https://speedvitals.com/report/www.keinegotterkeinemeister.de/eEuqEG/
   
   1100ms TTFB with PageSpeed ON
   https://speedvitals.com/report/www.keinegotterkeinemeister.de/tg1zwa/
   
   pagespeed config:
   https://pastebin.com/fLqVcP1M


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2083: Using automated install for Nginx 1.21.3 - Nginx does not start after successfull build and install - Ubuntu Impish

2022-04-09 Thread GitBox



eilandert commented on issue #2083:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2083#issuecomment-1094110543

   see https://github.com/apache/incubator-pagespeed-ngx/issues/1743


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] eilandert commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-04-09 Thread GitBox



eilandert commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1094110008

   Hello @nemihome, try the latest master, recursive.
   git clone --recursive https://github.com/apache/incubator-pagespeed-mod.git
   
   additionally, I uploaded precompiled psol binaries here: 
http://www.tiredofit.nl/psol-jammy.tar.xz
   (available: psol-bionic.tar.gz  psol-focal.tar.gz  psol-jammy.tar.xz  
psol-trusty.tar.gz  psol-xenial.tar.gz)
   
   and additionally, I'm offering debian and ubuntu packages (including jammy) 
here: https://deb.myguard.nl/nginx-modules/
   (there is also a docker, for impish and bullseye)


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] nemihome commented on issue #1743: Building on newer glibc (eg. Ubuntu 21.10 and above)

2022-04-09 Thread GitBox



nemihome commented on issue #1743:
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1743#issuecomment-1094046200

   Hello eilandert,
   
   I don't see this files in folder structure which you have mentioned - I do 
have incubator-pagespees-ngx-latest-stable/psol/third_party/apr/src but no 
threadproc and locks. There are only the following folders: include, random, 
test and none of them seems to include the files you mentioned.
   
   The only one I have found is:
   sed -i s/"define HAVE_PTHREAD_YIELD 1"/"define HAVE_PTHREAD_YIELD 0"/g 
third_party/apr/gen/arch/linux/x64/include/apr_private.h


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] nemihome commented on issue #2083: Using automated install for Nginx 1.21.3 - Nginx does not start after successfull build and install - Ubuntu Impish

2022-04-09 Thread GitBox



nemihome commented on issue #2083:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/2083#issuecomment-1094042551

   Same with 22.04 LTS. I have removed pagespeed now from Nginx due to missing 
maitenance


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] webtek-dev opened a new issue, #2088: Changing filenames with webp: does it impact images SEO?

2022-04-08 Thread GitBox



webtek-dev opened a new issue, #2088:
URL: https://github.com/apache/incubator-pagespeed-mod/issues/2088

   I have a website where images are very important and we get a lot of traffic 
from google images. When enabling webp images, Mod Pagespeed will constantly 
change the filenames of the images. Does that means it will have a negative 
impact on SEO?
   
   If googlebot finds a webp image on my website and adds it to google images, 
and then some day the file name changes. I assume the ranking will be lost 
because google will think the image was deleted?
   
   If images SEO is very important, should we completely disable webp rewriting?


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-ngx] Rming closed issue #1276: bad http status code 404 work with openresty

2022-04-06 Thread GitBox



Rming closed issue #1276: bad http status code 404 work with openresty
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1276


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] benyaminl commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



benyaminl commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088569097

   It basically the same. Just sometimes when on RHEL the alert are raising
   for some reason. The team should specifically write context for this I
   think.
   
   
   
   
   Regards,
   Benyamin Limanto
   sent from my Sony XPERIA XZ Premium
   
   On Tue, 5 Apr 2022, 17:43 Lofesa, ***@***.***> wrote:
   
   > Hi
   > There is an rpm pakage, maybe you wish to take a look on it
   >
   > 
https://centos.pkgs.org/8/getpagespeed-x86_64/nginx-module-pagespeed-selinux-1.20.0.1.13.35.2-3.el8.gps.x86_64.rpm.html
   >
   > —
   > Reply to this email directly, view it on GitHub
   > 
,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you commented.Message ID:
   > ***@***.***>
   >
   


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



Lofesa commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088546254

   Hi
   There is an rpm pakage, maybe you wish to take a look on it
   
https://centos.pkgs.org/8/getpagespeed-x86_64/nginx-module-pagespeed-selinux-1.20.0.1.13.35.2-3.el8.gps.x86_64.rpm.html


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] bbeckford commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



bbeckford commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088537898

   @Lofesa Thanks for the suggestion, just tried that and no change I'm afraid 
☹️


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



Lofesa commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088500697

   Hi
   Have tried this:
   https://www.modpagespeed.com/doc/faq#selinux_permissions


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] bbeckford commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



bbeckford commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088492818

   Thanks for responding, think I'm just going to remove pagespeed for now then 
路‍♂️


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [incubator-pagespeed-mod] benyaminl commented on issue #1936: Log filling up with Errors: Failed to mkdir /var/cache/mod_pagespeed/ flush

2022-04-05 Thread GitBox



benyaminl commented on issue #1936:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/1936#issuecomment-1088491414

   > Did anyone ever solve this issue? Think I'm going to have to ditch 
Pagespeed altogether on one server as I can't disable SELinux (even when I 
tested it turning off didn't stop the error logs).
   
   last time I tried to add `httpd_sys_rw_content`, sometimes works sometimes 
not, then ditch. Dunno now. 


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

To unsubscribe, e-mail: dev-unsubscr...@pagespeed.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

< 1 2 3 4 5 6 7 8 9 10 >

201 - 300 of 4314 matches

Mail list logo