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

2021-12-06 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-986941239 @borishinzer Unfortunately this doesn't work for us due to versioning. The URL changes every time we deploy so the file is not found with LoadFromFile on

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

2021-11-16 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-970629508 Yes exactly, Magento only sets headers for the application dynamic content. For static files it's a combination of rewrites plus .htaccess rules For

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

2021-11-16 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-970585694 For now I've added using the add_header for the cache control and it seems to show ok I also added a header to the location directive for static files to

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

2021-11-16 Thread GitBox
ioweb-gr commented on issue #1897: URL: https://github.com/apache/incubator-pagespeed-mod/issues/1897#issuecomment-970545822 To be honest I'm at a loss here as I'm not sure how to do that. Will it do the trick if in the location block for the static files I temporarily add 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-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] 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 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] 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 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] 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-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] 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] 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] 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] 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 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] 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] 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 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 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-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] 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 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] 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] 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] 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] 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] 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] 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;