Placed the Request headers before execute method

Author: DivyaDosibhatla

java/src/main/java/com/cybersource/ws/client/Connection.java

@@ -124,11 +124,8 @@ private void checkForFault()
             throws FaultException, ClientException {
         try {
             logger.log(Logger.LT_INFO, "Reading response...");
-
             int responseCode = getHttpResponseCode();
-            
             logResponseHeaders();
-
             // if successful, there's nothing left to do here.
             // we'll process the response in a later method.
             if (responseCode == HttpURLConnection.HTTP_OK) return;

java/src/main/java/com/cybersource/ws/client/HttpClientConnection.java

@@ -26,7 +26,6 @@
 import org.apache.commons.httpclient.params.HttpConnectionManagerParams;
 import org.apache.commons.httpclient.params.HttpMethodParams;
 import org.w3c.dom.Document;
-
 import javax.xml.parsers.DocumentBuilder;
 import javax.xml.transform.TransformerConfigurationException;
 import javax.xml.transform.TransformerException;
@@ -37,12 +36,10 @@
 import java.net.ProtocolException;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.HashMap;
 import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
 import org.apache.commons.httpclient.Header;
-
+import java.util.ArrayList;
+import java.util.Arrays;
 /**
  * Class helps in posting the Request document for the Transaction using HttpClient.
  * Converts the document to String format and also helps in setting up the Proxy connections.
@@ -86,9 +83,9 @@ void postDocument(Document request)
 
         postMethod.setRequestEntity(
                 new StringRequestEntity(requestString, null, "UTF-8"));
-        httpClient.executeMethod(postMethod);
-		postMethod.setRequestHeader("v-c-origin-iat", String.valueOf(System.currentTimeMillis()));
+        postMethod.setRequestHeader("v-c-origin-iat", String.valueOf(System.currentTimeMillis()));
 		logRequestHeaders();
+        httpClient.executeMethod(postMethod);
     }
 
     /* (non-Javadoc)
@@ -238,26 +235,17 @@ public boolean retryMethod(
         }
     }
 
-	@Override
-	void logRequestHeaders() {
-		Header[] headers=postMethod.getRequestHeaders();
-		Map<String, String> headerMap = new HashMap<String, String>();
-		for(Header header:headers){
-			headerMap.put(header.getName(), header.getValue());
-		 }
-        	logger.log(Logger.LT_INFO, "Request Headers: " + headerMap);
-	    }
+    @Override
+	void logRequestHeaders() {		
+		List<Header> reqheaders=Arrays.asList(postMethod.getRequestHeaders());
+        logger.log(Logger.LT_INFO, "Request Headers: " +reqheaders);
+	}
 
 	@Override
 	public void logResponseHeaders() {
-		Header[] headers = postMethod.getResponseHeaders();
-		 Map<String, String> headerMap = new HashMap<String, String>();
-		 for(Header header: headers){
-			 headerMap.put(header.getName(), header.getValue());
-		 }
-	    
-		 logger.log(Logger.LT_INFO, "Response Headers:" +headerMap);
-	     }
+		List<Header> respheaders=Arrays.asList(postMethod.getResponseHeaders());
+		 logger.log(Logger.LT_INFO, "Response Headers"+ respheaders);
+	}
 
 }