Hi John, Jetty gives this functionality by default without a filter/servlet, simply it is not available in tomcat. I used this functionality a lot in jetty and that plus the massive size of the UI gave me the idea. My approach to the problem was motivated by that I don't like to see anything wasted, but let's make a measurement and see what the impact is. This is what could be interesting in comparison with dynamic compression: - request/sec and average delay for static resoures in different sizes - time taken by full page reload
On Thu, Mar 20, 2014 at 11:19 PM, John Kinsella <j...@stratosec.co> wrote: > 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.horn...@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> > > -- EOF
