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

[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

[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

[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] 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,

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

[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

[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

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

[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

[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

[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

[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

[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

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

[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

[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

[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

[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

[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

[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

[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

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

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

[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] 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-968767756 I modified and now it looks like this ### nginx.conf ``` #ATTENTION! # #DO NOT MODIFY THIS FILE BECAUSE IT WAS GENERATED

[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

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

[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

[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

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

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

[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

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

[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

[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

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

[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

[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

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

[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

[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

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

[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

[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

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

[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

[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

[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

[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

[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

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

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

[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 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] 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;

[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

[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

[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

[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

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

2021-11-07 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

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

2021-11-07 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

[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

[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;` ?

[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

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

[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;` ?

[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

[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

[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

[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;` ?

[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;` ?

[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

[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

[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

[GitHub] [incubator-pagespeed-mod] coralsnake commented on issue #963: ModPagespeed* options in .htaccess don't work for fastcgi processes (PHP-FPM, php-cgi, etc)

2021-10-30 Thread GitBox
coralsnake commented on issue #963: URL: https://github.com/apache/incubator-pagespeed-mod/issues/963#issuecomment-955589467 I have "ModPagespeedEnableFilters remove_comments" in my .htaccess and when I run the php script the comment are actualy missing if I view the html code , maybe it

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

2021-10-29 Thread GitBox
Lofesa commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954959467 Hi Even in `https://www.whisky.de/` banner images are not webp converted but images in the html code get it. Son I´m still thinking is how javascript request

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

2021-10-29 Thread GitBox
Lofesa commented on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-954935119 Hi HttpCacheCompressionLevel stores optimized resources gziped in pagespeed cache and serve it as gzip with content-encoding: gzip header. If you set

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

2021-10-29 Thread GitBox
jmarantz commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954742259 One other subtlety around this: there are few caches in the path, potentially that may need to be warmed by retrying a few times. * mod_pagespeed's server-side

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

2021-10-29 Thread GitBox
datamints-off edited a comment on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954717017 This is the configuration for mod_pagespeed: ``` ModPagespeed on AddOutputFilterByType MOD_PAGESPEED_OUTPUT_FILTER text/html

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

2021-10-29 Thread GitBox
datamints-off edited a comment on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954717017 This is the configuration for mod_pagespeed: IfModule pagespeed_module ModPagespeed on AddOutputFilterByType

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

2021-10-29 Thread GitBox
datamints-off edited a comment on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954717017 This is the configuration for mod_pagespeed: ModPagespeed on AddOutputFilterByType MOD_PAGESPEED_OUTPUT_FILTER text/html

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

2021-10-29 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954717017 This is the configuration for mod_pagespeed: ` ModPagespeed on AddOutputFilterByType MOD_PAGESPEED_OUTPUT_FILTER text/html

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

2021-10-29 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954715945 @Lofesa The site consists of three systems behind a Nginx reverse proxy. mod_pagespeed is not enabled for the one at /forum, but for the one at / (you can see

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

2021-10-29 Thread GitBox
datamints-off commented on issue #2085: URL: https://github.com/apache/incubator-pagespeed-mod/issues/2085#issuecomment-954714560 @jmarantz Yes, I checked the content type: the browser accepts image/webp,image/apng,image/svg+xml,image/*, the server returns image/jpeg. -- This is an

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

2021-10-29 Thread GitBox
mch0lic edited a comment on issue #1742: URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1742#issuecomment-954701661 > Try pagespeed Domain http*://www.domain.com; or allmost pagespeed Domain https://www.domain.com; This did work to an extend, I could see that css/js

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