http issue 169: EOFException using NetHttpTransport on HEAD request with gzip encoding

https://codereview.appspot.com/6846109/

Author: Yaniv Inbar

google-http-client/src/main/java/com/google/api/client/http/HttpResponse.java

@@ -21,6 +21,7 @@
 import com.google.common.base.Preconditions;
 
 import java.io.ByteArrayOutputStream;
+import java.io.EOFException;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
@@ -377,6 +378,9 @@ public InputStream getContent() throws IOException {
           }
           content = lowLevelResponseContent;
           contentProcessed = true;
+        } catch (EOFException e) {
+          // this may happen for example on a HEAD request since there no actual response data read
+          // in GZIPInputStream
         } finally {
           if (!contentProcessed) {
             lowLevelResponseContent.close();

google-http-client/src/test/java/com/google/api/client/http/HttpResponseTest.java

@@ -390,4 +390,25 @@ public LowLevelHttpResponse execute() throws IOException {
     response.parseAsString();
     recorder.assertMessages(expectedMessages);
   }
+
+  public void testGetContent_gzipNoContent() throws IOException {
+    HttpTransport transport = new MockHttpTransport() {
+      @Override
+      public LowLevelHttpRequest buildRequest(String method, final String url) throws IOException {
+        return new MockLowLevelHttpRequest() {
+          @Override
+          public LowLevelHttpResponse execute() throws IOException {
+            MockLowLevelHttpResponse result = new MockLowLevelHttpResponse();
+            result.setContent("");
+            result.setContentEncoding("gzip");
+            result.setContentType("text/plain");
+            return result;
+          }
+        };
+      }
+    };
+    HttpRequest request =
+        transport.createRequestFactory().buildHeadRequest(HttpTesting.SIMPLE_GENERIC_URL);
+    request.execute().getContent();
+  }
 }