rwasef1830 edited a comment on issue #876:
URL: 
https://github.com/apache/incubator-pagespeed-mod/issues/876#issuecomment-907868790


   There's lot of problems I discovered with pagespeed which made me pretty 
much abandon it entirely except for the webp conversion part and even that I am 
considering just doing it in the application layer directly. It seems this 
project was given to the Apache foundation to be its "grave".
   
   1. the image resizing feature doesn't consider aspect ratio (the resize to 
rendered dimensions filter).
   2. the css inlining doesn't work at all if you use any advanced css features 
(variables, keyframes), the css parser is outdated.
   3. it doesn't do anything with csp except ignore some filters when you set 
honor csp on. it can't even read the current nonce from http header and apply 
it on inline scripts it writes. and to make things worse, the inline script it 
writes contains an ever changing embedded part (server generated nonce), so you 
can't even try to work around this by using unsafe-hashes and hashing the 
content of the inline script yourself.
   
   About the only useful feature it has right now maybe to recompress images to 
be more efficient and resize images to tag specified width and length and 
conversion to webp. That's it in my opinion.
   
   I tried to modify / fork it but the build system is complex, and the 
projects are not isolated, you are forced to compile an apache and an nginx and 
both modules and even chromium just to build it, and the scripts are already 
broken on newer distros. I gave up on trying, and just implemented some of 
those features in my own apps by hand.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

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

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


Reply via email to