Merge pull request #3747 from backstraw/master

fix: close DataOutputStream in finally block for resource safety in postBody

Author: xuxueli

xxl-job-core/src/main/java/com/xxl/job/core/util/XxlJobRemotingUtil.java

@@ -68,6 +68,7 @@ public void checkServerTrusted(X509Certificate[] chain, String authType) throws
     public static ReturnT postBody(String url, String accessToken, int timeout, Object requestObj, Class returnTargClassOfT) {
         HttpURLConnection connection = null;
         BufferedReader bufferedReader = null;
+        DataOutputStream dataOutputStream = null;
         try {
             // connection
             URL realUrl = new URL(url);
@@ -102,7 +103,7 @@ public static ReturnT postBody(String url, String accessToken, int timeout, Obje
             if (requestObj != null) {
                 String requestBody = GsonTool.toJson(requestObj);
 
-                DataOutputStream dataOutputStream = new DataOutputStream(connection.getOutputStream());
+                dataOutputStream = new DataOutputStream(connection.getOutputStream());
                 dataOutputStream.write(requestBody.getBytes("UTF-8"));
                 dataOutputStream.flush();
                 dataOutputStream.close();
@@ -144,6 +145,9 @@ public static ReturnT postBody(String url, String accessToken, int timeout, Obje
             return new ReturnT<String>(ReturnT.FAIL_CODE, "xxl-job remoting error("+ e.getMessage() +"), for url : " + url);
         } finally {
             try {
+                if (dataOutputStream != null) {
+                    dataOutputStream.close();
+                }
                 if (bufferedReader != null) {
                     bufferedReader.close();
                 }