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


Hey Laszlo, that's a nice feature please advise if you want to submit a tested 
patch that applies on master or you want to close it?
Thanks.

- Rohit Yadav


On April 15, 2014, 6:13 a.m., Laszlo Hornyak wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20343/
> -----------------------------------------------------------
> 
> (Updated April 15, 2014, 6:13 a.m.)
> 
> 
> Review request for cloudstack.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
>  - use the User-Agent header to get the client type
>  - only use the cache workaround if the browser is MSIE, use standard cache 
> control headers anyway
> 
> 
> Diffs
> -----
> 
>   ui/index.jsp 5840e71 
>   utils/pom.xml cc6de3b 
>   utils/src/com/cloud/utils/servlet/CacheControlUtil.java PRE-CREATION 
>   utils/test/com/cloud/utils/servlet/CacheControlUtilTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/20343/diff/
> 
> 
> Testing
> -------
> 
> -unit testing
> -tests with firefox
> - tests with IE
> - tests with modified files
> 
> 
> Thanks,
> 
> Laszlo Hornyak
> 
>

Reply via email to