s3-website
s3-website copied to clipboard
klaemo
Add gzip
It's ugly but it works. Replaces original files with gzipped. Adds Encoding header.
@matyunya This is an awesome addition! I will make sure I get a good look at it tomorrow. Thanks again for the addition!
@matyunya Sorry to be slow, I have been swamped this week. I was testing your branch and ran into a few issues. I made a comment on the code that I think is at least partly responsible for the issue I was seeing. I am also happy to dig into the issue as I get more time throughout the weekend. Thanks again for the contribution. I am really excited about this feature.
@nick-benoit14 Hey! So the problem was the context of execution: it's node_modules in my case, so I found no better solution than to check the base dir path. Still ugly though :) It should work now.
@matyunya Thanks for pushing a fix. I still haven't had a chance to check it out. I will do my very best to get it tested tonight though. If all goes well I will get it merged and deployed! Thanks for your patience, I know it can be really frustrating to put up with busy slow maintainers.
@matyunya Many apologies again for being slow. After thinking about it for a bit I kind of like the idea of copying the files then gzipping them instead of replacing them. I think it is easy to forget that it is going to gzip everything and inadvertently mess something up. What are your thoughts on that? Don't feel like you need to add that feature yourself, I am happy to work on it as well. I think it may just delay getting a new version out a bit.