Thanks for the review Ziv.
http://codereview.appspot.com/1855044/diff/45001/46004 File java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyBase.java (left): http://codereview.appspot.com/1855044/diff/45001/46004#oldcode60 java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyBase.java:60: "vary", "expires", "date", "pragma", "cache-control", "transfer-encoding", "www-authenticate" On 2010/07/26 18:40:22, zhoresh wrote:
Please make sure you don't loose this values as part of the change.
We made sure that we covered all of these headers. Anupama had added "etag" and "last-modified" headers to CACHING_DIRECTIVES. I think her latest patch got out of sync somehow. Resync would fix this. http://codereview.appspot.com/1855044/show
