-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow serving GZip-ped resources from /? #76
Comments
It is better to have pre-gzipped scripts rather than compressing on the fly. Maybe with a build script? |
I'll modify the build script of Hestia to produce a gzipped version - as part of the building flow. |
As with #76 (comment), We don't like our software to be slow, do we? |
See vnma0/hestia#133 for a building script. |
Also, @dungwinux, can you make a script to clear the source maps produced by CRA? It is not needed, and in some cases, intruders can leverage them to inject scripts & alter the behaviour of Hestia - something pretty much possible since we do not support HTTPS. |
later, not on this issue |
Well I did it anyway - CRA supports doing so. |
The gain from compressing the resources is substantial - at least 30% for short files, and most of the JS files takes 60% less space. For fonts, the gains is nearly 50%.
|
Where's the release, then? |
Feel free to get the artifacts from AppVeyor. Currently I don't have a way to provide the latest build from |
|
It takes pretty long time to load 2.5MB of scripts from the network.
The text was updated successfully, but these errors were encountered: