Reviewers: henry.saputra, hceylan_batoo.org, dev-remailer_shindig.apache.org,


Please review this at http://codereview.appspot.com/1802042/show

Affected files:
java/gadgets/src/main/java/org/apache/shindig/gadgets/http/BasicHttpFetcher.java


Index: java/gadgets/src/main/java/org/apache/shindig/gadgets/http/BasicHttpFetcher.java
===================================================================
--- java/gadgets/src/main/java/org/apache/shindig/gadgets/http/BasicHttpFetcher.java (revision 963805) +++ java/gadgets/src/main/java/org/apache/shindig/gadgets/http/BasicHttpFetcher.java (working copy)
@@ -43,6 +43,8 @@
 import org.apache.http.client.methods.HttpUriRequest;
 import org.apache.http.client.params.ClientPNames;
 import org.apache.http.client.params.HttpClientParams;
+import org.apache.http.client.protocol.RequestAddCookies;
+import org.apache.http.client.protocol.ResponseProcessCookies;
 import org.apache.http.conn.ClientConnectionManager;
 import org.apache.http.conn.ConnectionPoolTimeoutException;
 import org.apache.http.conn.HttpHostConnectException;
@@ -212,6 +214,10 @@
     });
client.setHttpRequestRetryHandler(new DefaultHttpRequestRetryHandler() );

+    // SHINDIG-1382 Disable automatic storage and sending of cookies
+    client.removeRequestInterceptorByClass(RequestAddCookies.class);
+    client.removeResponseInterceptorByClass(ResponseProcessCookies.class);
+
     // Use Java's built-in proxy logic in case no proxy set via guice.
     if (StringUtils.isEmpty(basicHttpFetcherProxy)) {
ProxySelectorRoutePlanner routePlanner = new ProxySelectorRoutePlanner(


Reply via email to