[GitHub] [incubator-pagespeed-mod] datamints-off commented on issue #2085: Images don't get converted to webp in ajax requests

2021-11-03 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-958755112 Thanks for all your input: I found that the accept header of the request fetching the banner is not stating that webp images are accepted. I will change that o

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
Lofesa commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958835298 Hi I found this old thread https://groups.google.com/g/mod-pagespeed-discuss/c/6g_YjUeDyNU Can you try `pagespeed PreserveUrlRelativity on;` ? https://www.mod

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
Lofesa commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958835298 Hi I found this old thread https://groups.google.com/g/mod-pagespeed-discuss/c/6g_YjUeDyNU Can you try `pagespeed PreserveUrlRelativity on;` ? https://www.mod

[GitHub] [incubator-pagespeed-mod] datamints-off commented on issue #2085: Images don't get converted to webp in ajax requests

2021-11-03 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-958755112 Thanks for all your input: I found that the accept header of the request fetching the banner is not stating that webp images are accepted. I will change that o

[GitHub] [incubator-pagespeed-ngx] mch0lic commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
mch0lic commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958716126 > Where are the config directives, server block, location block.. ? Server block. > What certificate is in use at img.domain.com, Is a selsigned certific

[GitHub] [incubator-pagespeed-ngx] mch0lic edited a comment on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
mch0lic edited a comment on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958716126 -- 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 co

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
Lofesa commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958835298 Hi I found this old thread https://groups.google.com/g/mod-pagespeed-discuss/c/6g_YjUeDyNU Can you try `pagespeed PreserveUrlRelativity on;` ? https://www.mod

[GitHub] [incubator-pagespeed-mod] datamints-off commented on issue #2085: Images don't get converted to webp in ajax requests

2021-11-03 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-958755112 Thanks for all your input: I found that the accept header of the request fetching the banner is not stating that webp images are accepted. I will change that o

[GitHub] [incubator-pagespeed-ngx] mch0lic commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
mch0lic commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958716126 -- 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.

[GitHub] [incubator-pagespeed-ngx] mch0lic edited a comment on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
mch0lic edited a comment on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958716126 > Where are the config directives, server block, location block.. ? Server block. > What certificate is in use at img.domain.com, Is a selsigned c

[GitHub] [incubator-pagespeed-ngx] Lofesa commented on issue #1742: 1.13.35.2-stable suddenly stopped working

2021-11-03 Thread GitBox
Lofesa commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-958835298 Hi I found this old thread https://groups.google.com/g/mod-pagespeed-discuss/c/6g_YjUeDyNU Can you try `pagespeed PreserveUrlRelativity on;` ? https://www.mod

[GitHub] [incubator-pagespeed-mod] datamints-off commented on issue #2085: Images don't get converted to webp in ajax requests

2021-11-03 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-958755112 Thanks for all your input: I found that the accept header of the request fetching the banner is not stating that webp images are accepted. I will change that o

[GitHub] [incubator-pagespeed-mod] acachy commented on issue #2069: PSOL with bazel

2021-11-06 Thread GitBox
acachy commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962558016 > @acachy how did you manage to build? > > I get a lot of errors like this: pagespeed/kernel/base/string_util.cc: In function 'GoogleString net_instaweb::intern

[GitHub] [incubator-pagespeed-mod] acachy edited a comment on issue #2069: PSOL with bazel

2021-11-06 Thread GitBox
acachy edited a comment on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962558016 > @acachy how did you manage to build? > > I get a lot of errors like this: pagespeed/kernel/base/string_util.cc: In function 'GoogleString net_instaweb:

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-07 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962562306 Thanks! There are a few things I can use / didn't know When I have some time I'll give it another try. Same as you, trying to fix what I encounter. Never us

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-07 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962696232 @acachy Thanks for your pointers, I used them in a slightly different approach but I am a lot further ahead than I was, but I am stuck on the same error now

[GitHub] [incubator-pagespeed-mod] eilandert edited a comment on issue #2069: PSOL with bazel

2021-11-07 Thread GitBox
eilandert edited a comment on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962696232 @acachy Thanks for your pointers, I used them in a slightly different approach but I am a lot further ahead than I was, but I am stuck on the same er

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-07 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-962703264 Or we need a @glog//:glog dependency bazel build -c fastbuild @glog//:glog seems to do something ;-) -- This is an automated message from the Apache Git S

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-962983135 I also face the same issue with 4xx status code preventing rewriting. I'm using the following setup ``` pagespeed RewriteLevel PassThrough; pag

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-962983135 I also face the same issue with 4xx status code preventing rewriting. I'm using the following setup ``` pagespeed RewriteLevel PassThrough;

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963011339 Hi | pagespeed LoadFromFile ... |directive, things start working but I can't do it because my static resources are using a dynamic versioning system.

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963024870 > But you have a "fixed" part: pagespeed LoadFromFile "https://www.example.com/pub/media/"; "/path-to-the fixed-one/pub/media/" Not always, some paths will b

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963024870 > But you have a "fixed" part: pagespeed LoadFromFile "https://www.example.com/pub/media/"; "/path-to-the fixed-one/pub/media/" Not always, some paths

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963058524 > > Not always, some paths will be rendered like this `pub/static/version1636311072/frontend/./image.png ` > But you still have a "fixed" part,

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963066407 >This last directive cover all others. OK I'm sticking with just this then >You need to clear the pagespeed cahe I cleared the cache from purge c

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963066407 >This last directive cover all others. OK I'm sticking with just this then >You need to clear the pagespeed cahe I cleared the cache from

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963075582 >> But nope they're not getting served as webp. >> I no longer see the error for http though And still have the 404? Not serves as webp, but are rewrited b

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963079438 Yes the request for each image has the header for webp ![image](https://user-images.githubusercontent.com/20220341/140737811-73721907-c971-4784-9bd0-90fb07b2d

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963080981 > Not serves as webp, but are rewrited by pagespeed? The url have pagespeed in it? like: ` xshoes_tennis_bg_eng.png.pagespeed.ic.oIjD0NVeCW.png` No they do

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963096659 Ummm In the image I see a cache-control header: `cache-control: s-maxage=10` but pagespeed need resources public cacheable `cache-control: max-age=600` or `cac

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963099670 But the yhimg is the 404. optimized resources are in the https cache, so why the 404? Test again the log and see if request whit "Serf" in the UA are here. Some li

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963099670 But the thing is the 404. optimized resources are in the https cache, so why the 404? Test again the log and see if request whit "Serf" in the UA are here. S

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963103829 The problem is with the current setup, I don't see anything in the proxy_error_log like before and there's no Serf match in any log, not nginx access or error log,

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963106186 The weird thing is that on pagespeed_console, everything is 0 ![image](https://user-images.githubusercontent.com/20220341/140742672-b3086165-90b0-47a5-935f-2

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963107208 And what you can see in the pagespeed "Message History"? `https://example.com/pagespeed_global_admin/message_history` is the url you can view log messages from pag

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963110093 For example for some jpg files ``` [Mon, 08 Nov 2021 12:36:15 GMT] [Info] [1698645] Trying to serve rewritten resource in-place: https://subdomain.example

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963111692 Where are those messages stored in the logs? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963128183 These messages are stored in files, don´t remember where are stored, but are non human readable. Nothing to do whit server logs. But they says that pagespeed fail r

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963235129 Well since PLESK autoconfigures most of them I'm not exactly sure about every detail but let me post the nginx conf and the vhost conf and maybe we can shed some li

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963235129 Well since PLESK autoconfigures most of them I'm not exactly sure about every detail but let me post the nginx conf and the vhost conf and maybe we can shed

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963239577 Let me note that most nginx directives for pagespeed which are common for all vhosts, are directly inserted in the nginx conf files and then I modify on a per vhost

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-08 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963459358 This config is a lot confusing for me. You have a server block for each IP/port, and is supossed that ipv4:443 must have the exact same config that ipv6:443, both a

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-09 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963947963 Indeed I could find some apache 404s ``` grep -ri '\.pagespeed' ../logs/access_ssl_log* | grep -i ' 404 ' ../logs/access_ssl_log.processed:37.6.136

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-09 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963968735 Yes, not "under certain cases" but in all cases file extension match the nginx location. URL rewrited by pagespeed have the same file extension that the original f

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-09 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963968735 -- 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 com

[GitHub] [incubator-pagespeed-mod] acachy commented on issue #2069: PSOL with bazel

2021-11-11 Thread GitBox
acachy commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-966149681 > Or we need a @glog//:glog dependency > > bazel build -c fastbuild @glog//:glog seems to do something ;-) Looks like your way is the good one I've

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-11 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-966219159 > > Or we need a @glog//:glog dependency > > bazel build -c fastbuild @glog//:glog seems to do something ;-) > > Looks like your way is the good one >

[GitHub] [incubator-pagespeed-mod] eilandert edited a comment on issue #2069: PSOL with bazel

2021-11-11 Thread GitBox
eilandert edited a comment on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-966219159 > > Or we need a @glog//:glog dependency > > bazel build -c fastbuild @glog//:glog seems to do something ;-) > > Looks like your way is the good one

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-12 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-967521102 ok, I do this: https://github.com/eilandert/psol-bazel/blob/master/docker/build.sh ``` bazel build -c fastbuild \ @glog//:glog \ @com_googlesou

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2069: PSOL with bazel

2021-11-13 Thread GitBox
Lofesa commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-968075205 Hi @eilandert I´m not a developer, so can´t help here. But I have a clue Greping in and old copy of the repo I have found that icu_whatever_things are in /t

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-13 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-968079554 Hi @Lofesa I am not a dev either ;-) But I like to puzzle. I have it working now with latest master, well at least it passes ./configure from nginx.

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2069: PSOL with bazel

2021-11-14 Thread GitBox
Lofesa commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-968332337 Hi If I´m not wrong, chromium was changed by envoy in the bazel brach. And I think nginx module is a wrapper around the psol lib to work with nginx, so if you ha

[GitHub] [incubator-pagespeed-mod] eilandert commented on issue #2069: PSOL with bazel

2021-11-14 Thread GitBox
eilandert commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-968358107 Having the pagespeed_automatic.a is not enough, we need the headerfiles to be copied, otherwise the NGINX module is not aware of the functions and how to use them.

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #2069: PSOL with bazel

2021-11-15 Thread GitBox
Lofesa commented on issue #2069: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2069#issuecomment-968726076 I only intend to help as much as I can reading the code. The files in the include, in fact all files under "base/debug", don´t exist anymore. I think is better to d

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968746800 I struggled a bit on how to remove the location directive which precedes the pagespeed one and now I have this new nginx.conf ``` server {

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968746800 I struggled a bit on how to remove the location directive which precedes the pagespeed one and now I have this new nginx.conf ``` server

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968749692 On next step I'll try moving it up as you suggested -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968767756 I modified and now it looks like this ### nginx.conf ``` #ATTENTION! # #DO NOT MODIFY THIS FILE BECAUSE IT WAS GENERATED AUTOMATICALLY

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968784299 You still viewing *pagespeed* request in the apache log? Like these in https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-963947963 Have

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968796195 I've modified my configuration file template so that all custom directives I write will go directly below the command `proxy_read_timeout` and above

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968796195 I've modified my configuration file template so that all custom directives I write will go directly below the command `proxy_read_timeout` and

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968820782 If you try some like: `https://subdomain.example.com/pub/media/wysiwyg/espa.png` what you get? Any instance of "Serf" in the logs? What do you see in the me

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968833345 OK let me try and sort it out I'm going to try URL /storeviewgreek/tennis-balls.html?PageSpeedFilters=+debug Let's answer the questions one by

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa700fbab

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968846650 `[Warning] [153873] [x601313-dunlop-atp-official-tennis-balls_1.jpg:0] Resource based on https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968882127 Let me answer quickly the last part about CoreFilters which is easier , I'll answer the second part too when I investigate It' smy understanding that when I use

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968887356 The file exists ``` ll pub/media/catalog/product/cache/9cb80aaa700fbabda1d30deb1d8f7ff5/6/0/601313-dunlop-atp-official-tennis-balls_1.jpg -h | sed 's eten

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968893173 > > So only the ones I enable should be active right? Yes. I´m asking because I can´t see these config in the vhos_nginx.conf file here https://github.com/

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968895233 > > > The file exists > > ``` > ll pub/media/catalog/product/cache/9cb80aaa700fbabda1d30deb1d8f7ff5/6/0/601313-dunlop-atp-official-tennis-balls_1.jpg

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968895233 > > > The file exists > > ``` > ll pub/media/catalog/product/cache/9cb80aaa700fbabda1d30deb1d8f7ff5/6/0/601313-dunlop-atp-official-tennis-ball

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968912037 >Yes. I´m asking because I can´t see these config in the vhos_nginx.conf file here #1897 (comment) I've put them in the http context which is affecting all vh

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968912037 >Yes. I´m asking because I can´t see these config in the vhos_nginx.conf file here #1897 (comment) I've put them in the http context which is affecting

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968914830 I forgot to respond that for an example image the headers are ``` HTTP/2 200 OK server: nginx date: Mon, 15 Nov 2021 13:27:59 GMT content-type: im

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968957374 Well, this etag header tell me that this request is not managed by pagespeed. Optimized resources have a etag like etag: W/"0" or etag: "PSA/X" if optimized by

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968974811 I myself read the Varnish article and I wanted to note that I'm not using varnish. Just nginx as reverse proxy to apache. Regarding the headers for the url I

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-968978217 Without changing any config this time I saw these images converted for example ![image](https://user-images.githubusercontent.com/20220341/141800888-c60ddd3

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969062799 Well think pagespeed is a proxy too, don´t store the html but some metadata about it. When pagespeed is working, html pages are served with cache-control: max-

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969125690 I didn't mention that in my OP but when I use LoadFromFile everything worked a lot better. But because in production we deploy with a versioning system, this

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969129009 I reverted to normal fetch and based on your suggestion I enabled only pngs and jpgs to reduce the message clutter pagespeed Disallow .*; pagespeed Allow

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969133635 No, don´t do that. The rules I hace posted don´t work like these you havr. Whit your pagespeed Disallow .*; disables all rewites. and the no furter process is do

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969140401 Oh Ok I see, however it does rewrite with the commands above... So many weird behaviors I reverted to disallowing each extension separately -- This is an a

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969143014 Again the typical 4xx status code ![image](https://user-images.githubusercontent.com/20220341/141827058-34b7cdb5-5be1-49d8-a149-19927f7a1027.png) Incid

[GitHub] [incubator-pagespeed-mod] ioweb-gr edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969143014 Again the typical 4xx status code ![image](https://user-images.githubusercontent.com/20220341/141827058-34b7cdb5-5be1-49d8-a149-19927f7a1027.png)

[GitHub] [incubator-pagespeed-mod] Lofesa edited a comment on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa edited a comment on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969133635 No, don´t do that. The rules I have posted don´t work like these you have. Whit your pagespeed Disallow .*; disables all rewites. and then no furter proce

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969181614 Now I can see mayority of files writed, unless svg, gif and some jpg/png. Others image files, like gallery.png is not rewrited becase is loaded by a css file that y

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969213042 I have added the location directive already. But the thing is I can see like 20 `.jpg` images rewrited, but another 30 not rewrited. Messages

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
jmarantz commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969215732 Sorry I haven't been following all the details but I can tell you that the "Allow" and "Disallow" commands work with wildcards, not regexes. So I don't think any of

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969220450 Yes I read the docs and adjusted thanks for the input now they are like this ``` pagespeed Disallow "*.svg"; pagespeed Disallow "*.woff2"; pagespeed

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-96941 After reducing the clutter I notice this error message also ` Too busy to rewrite image.` What does this mean, how do I fix it? -- This is an automa

[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
jmarantz commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969232367 That's not necessarily a problem. It's an "Info" message not an Error. Image optimizations are cached, and when the cache is cold and there are a lot of concurrent

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969243930 I see, it's beginning to fill more and more images. Almost all jpgs and jpegs have converted now. Some pngs refuse to convert even though they're rewrited. They sti

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969247598 Then I visit another category page of the website, and the 4xx status code errors come back up again for all images I have no idea what is going on honestly.

[GitHub] [incubator-pagespeed-mod] Lofesa commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
Lofesa commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969247915 Not importat. Pagespeed tries to optimize 8 images at once and take 10ms. to do it, is there are more images or take more than 10ms to do the work, then it do a backg

[GitHub] [incubator-pagespeed-mod] ioweb-gr commented on issue #1897: 4xx status code, preventing rewriting

2021-11-15 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-969251352 here's one example ``` [Mon, 15 Nov 2021 19:33:37 GMT] [Info] [507507] HTTPCache key=https://subdomain.example.com/pub/media/catalog/product/cache/9cb80aaa

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