Swallow exception from close call when there is an underlying IOException

chajath · ejona86 · commit 75c2b202b8de · 2016-11-07T13:08:55.000-08:00
diff --git a/google-http-client/src/main/java/com/google/api/client/http/javanet/NetHttpRequest.java b/google-http-client/src/main/java/com/google/api/client/http/javanet/NetHttpRequest.java
@@ -75,10 +75,21 @@ public LowLevelHttpResponse execute() throws IOException {
           connection.setChunkedStreamingMode(0);
         }
         OutputStream out = connection.getOutputStream();
+        boolean threw = true;
         try {
           getStreamingContent().writeTo(out);
+          threw = false;
         } finally {
-          out.close();
+          try {
+            out.close();
+          } catch (IOException exception) {
+            // When writeTo() throws an exception, chances are that the close call will also fail.
+            // In such case, swallow exception from close call so that the underlying cause can
+            // propagate.
+            if (!threw) {
+              throw exception;
+            }
+          }
         }
       } else {
         // cannot call setDoOutput(true) because it would change a GET method to POST
