cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Kinsella <...@stratosec.co>
Subject Re: Review Request 12228: static resource compression
Date Thu, 20 Mar 2014 22:19:27 GMT
Laszlo, can you reference any other open source projects that have similar solutions to this
issue? Anything I’ve read states dynamic compression in tomcat/httpd/nginx does not add
significant CPU overhead.

On Mar 20, 2014, at 12:53 PM, Laszlo Hornyak <laszlo.hornyak@gmail.com<mailto:laszlo.hornyak@gmail.com>>
wrote:


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12228/
-----------------------------------------------------------

(Updated March 20, 2014, 7:53 p.m.)


Review request for cloudstack, Brian Federle, Darren Shepherd, and Prasanna Santhanam.


Changes
-------

spaces


Repository: cloudstack-git


Description
-------

CloudStack at first use downloads some 3.5 MB of css and javascript to the client. With a
weak internet connection, this might take a long time. With gzip compression content can be
compressed to 850 KB.

This version of the patch uses a custom plugin to compress static resources, so that no dynamic
compression is needed at runtime. When the static resource servlet notices that there is gzipped
version of the resource and the client accepts gzipped content, then it is going to send the
gziped version, while still respects http caching.


Diffs (updated)
-----

 client/WEB-INF/web.xml 1af38e1
 client/pom.xml d8dbde7
 server/src/com/cloud/servlet/StaticResourceServlet.java PRE-CREATION
 server/test/com/cloud/servlet/StaticResourceServletTest.java PRE-CREATION

Diff: https://reviews.apache.org/r/12228/diff/


Testing
-------

yes, tested with firefox and chrome


Thanks,

Laszlo Hornyak


Stratosec<http://stratosec.co/> - Compliance as a Service
o: 415.315.9385
@johnlkinsella<http://twitter.com/johnlkinsella>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message