You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by or...@apache.org on 2018/03/16 14:47:27 UTC

qpid-broker-j git commit: NO-JIRA: [Broker-J] [System tests] Add extra debug logging into HttpTestHelper

Repository: qpid-broker-j
Updated Branches:
  refs/heads/master a15892b9f -> c05dfbd52


NO-JIRA: [Broker-J] [System tests] Add extra debug logging into HttpTestHelper


Project: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/repo
Commit: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/commit/c05dfbd5
Tree: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/tree/c05dfbd5
Diff: http://git-wip-us.apache.org/repos/asf/qpid-broker-j/diff/c05dfbd5

Branch: refs/heads/master
Commit: c05dfbd52132e2969966bdfe478b2b58f0108b9a
Parents: a15892b
Author: Alex Rudyy <or...@apache.org>
Authored: Fri Mar 16 14:46:52 2018 +0000
Committer: Alex Rudyy <or...@apache.org>
Committed: Fri Mar 16 14:47:13 2018 +0000

----------------------------------------------------------------------
 .../apache/qpid/tests/http/HttpTestHelper.java  | 31 +++++++++++++++-----
 1 file changed, 23 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/qpid-broker-j/blob/c05dfbd5/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestHelper.java
----------------------------------------------------------------------
diff --git a/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestHelper.java b/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestHelper.java
index 00eb60d..d6eeba4 100644
--- a/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestHelper.java
+++ b/systests/qpid-systests-http-management/src/main/java/org/apache/qpid/tests/http/HttpTestHelper.java
@@ -356,23 +356,38 @@ public class HttpTestHelper
 
     public int submitRequest(String url, String method, Object data, Map<String, List<String>> responseHeadersToCapture) throws IOException
     {
-        HttpURLConnection connection = openManagementConnection(url, method);
+        LOGGER.debug("HttpURLConnection : open  : {} : {}", method, url);
+        RuntimeException ex = null;
+        int responseCode = -1;
         try
         {
-            if (data != null)
+            HttpURLConnection connection = openManagementConnection(url, method);
+            try
             {
-                writeJsonRequest(connection, data);
+                if (data != null)
+                {
+                    writeJsonRequest(connection, data);
+                }
+                responseCode = connection.getResponseCode();
+                if (responseHeadersToCapture != null)
+                {
+                    responseHeadersToCapture.putAll(connection.getHeaderFields());
+                }
+                return responseCode;
+            }
+            catch (RuntimeException e)
+            {
+                ex = e;
+                throw e;
             }
-            int responseCode = connection.getResponseCode();
-            if (responseHeadersToCapture != null)
+            finally
             {
-                responseHeadersToCapture.putAll(connection.getHeaderFields());
+                connection.disconnect();
             }
-            return responseCode;
         }
         finally
         {
-            connection.disconnect();
+            LOGGER.debug("HttpURLConnection : close : {} : {} : {}", method, responseCode, url, ex);
         }
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org