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

[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

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

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

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

[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-969277121 @jmarantz Files in the same folder, 1 get rewrited and serve webp version but others not, the file are in place, it can show with a ll bash command, but in

[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

[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-968887356 The file exists ``` ll pub/media/catalog/product/cache/9cb80aaa700fbabda1d30deb1d8f7ff5/6/0/601313-dunlop-atp-official-tennis-balls_1.jpg -h

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

[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-968887356 The file exists ``` ll pub/media/catalog/product/cache/9cb80aaa700fbabda1d30deb1d8f7ff5/6/0/601313-dunlop-atp-official-tennis-balls_1.jpg -h

[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-969256950 @Lofesa Would you mind sanitizing this message url that I missed for privacy reasons?

[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-969263929 RE "serves as jpg instead of web": in what context? - rewritten URL on HTML page? - in-place optimization on a file with that name? - in response to a

[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-969277121 @jmarantz Files in the same folder, 1 get rewrited and serve webp version but others not, the file are in place, it can show with a ll bash command, but in the

[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-969306541 Well these headers are just debug info on whether you're hitting the application Full Page Cache for the response for which the backend can be a anything. Most 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-969260667 What's really puzzling me is that it's doing a shrink in this file ``` [Mon, 15 Nov 2021 19:42:30 GMT] [Info] [519193]

[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-969260667 What's really puzzling me is that it's doing a shrink in this file ``` [Mon, 15 Nov 2021 19:42:30 GMT] [Info] [519193]

[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-969312608 @ioweb-gr Can you take a look in /etc and search for some-file.vlc and grep if there is some like this `unset resp.http.X-Varnish;` This hide the varnish

[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-969520651 ``` $ find /etc/ -name "*.vlc" | wc -l 0 ``` Disabled full page cache and block cache that magento utilizes which affect the frontend. Not turning

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

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

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