Reorder conn.disconnect, to support keepalives when bodyStream read

Fixes jhy#1232

CHANGES

@@ -8,6 +8,9 @@ jsoup changelog
   * Improvement: added Element chaining methods for various overridden methods on Node.
     <https://github.com/jhy/jsoup/issues/1193>
 
+  * Improvement: ensure HTTP keepalives work when fetching content via body() and bodyAsBytes().
+    <https://github.com/jhy/jsoup/issues/1232>
+
   * Bugfix: if duplicate attributes in an element exist, retain the first vs the last attribute with the same name. Case
     aware (HTML case-insensitive names, XML are case-sensitive).
     <https://github.com/jhy/jsoup/issues/1219>

src/main/java/org/jsoup/helper/HttpConnection.java

@@ -915,13 +915,10 @@ private static HttpURLConnection createConnection(Connection.Request req) throws
         }
 
         /**
-         * Call on completion of stream read, to close the body (or error) stream
+         * Call on completion of stream read, to close the body (or error) stream. The connection.disconnect allows
+         * keep-alives to work (as the underlying connection is actually held open, despite the name).
          */
         private void safeClose() {
-            if (conn != null) {
-                conn.disconnect();
-                conn = null;
-            }
             if (bodyStream != null) {
                 try {
                     bodyStream.close();
@@ -931,6 +928,10 @@ private void safeClose() {
                     bodyStream = null;
                 }
             }
+            if (conn != null) {
+                conn.disconnect();
+                conn = null;
+            }
         }
 
         // set up url, method, header, cookies