merge conflict fixed

Author: SumiteshRSrivastava

.travis.yml

@@ -1,7 +1,5 @@
 language: java
-"dist": "precise"
 jdk:
-  - oraclejdk8
   - oraclejdk7
   - openjdk7
-  - openjdk6
+  - openjdk6

README.md

@@ -110,7 +110,7 @@ Unix or Linux:	runSample.sh <service_name>
 ## JKS creation
 
 - To convert the p12 file to JKS follow the steps mentioned below.
-  - These commands will take out all the certs from the p12 file. 
+  - These commands will take out all the certs from the p12 file. 
   1. `openssl pkcs12 -in <Merchant_ID>.p12 -nocerts -out <Merchant_ID>.key`
   2. `openssl pkcs12 -in <Merchant_ID>.p12 -clcerts -nokeys -out  <Merchant_ID>.crt`
   3. `openssl pkcs12 -in <Merchant_ID>.p12 -cacerts -nokeys -out CyberSourceCertAuth.crt`
@@ -224,3 +224,4 @@ _______________________________
 - For more information about CyberSource services, see <http://www.cybersource.com/developers/documentation>.
 - For all other support needs, see <http://www.cybersource.com/support>.
 
+

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

@@ -34,7 +34,6 @@
 import java.io.FileInputStream;
 import java.io.IOException;
 import java.io.StringReader;
-import java.lang.reflect.InvocationTargetException;
 import java.security.KeyStore;
 import java.security.cert.PKIXParameters;
 import java.security.cert.TrustAnchor;
@@ -91,16 +90,14 @@ public static Map runTransaction(Map<String,String> request, Properties props)
      * @throws FaultException  if a fault occurs.
      * @throws ClientException if any other exception occurs.
      */
-    @SuppressWarnings("unchecked")
-	public static Map runTransaction(
+    public static Map runTransaction(
             Map<String, String> request, Properties props,
             Logger _logger, boolean prepare, boolean logTranStart)
             throws FaultException, ClientException {
         MerchantConfig mc;
         LoggerWrapper logger = null;
         Connection con = null;
 
-
         try {
             setVersionInformation(request);
 
@@ -125,39 +122,8 @@ public static Map runTransaction(
 //          FileWriter writer = new FileWriter(new File("signedDoc.xml"));
 //          writer.write(XMLUtils.PrettyDocumentToString(signedDoc));
 //          writer.close();
-            if(mc.isCustomHttpClassEnabled()){
-				Class<Connection> customConnectionClass;
-				try {
-					customConnectionClass = (Class<Connection>) Class.forName(mc.getcustomHttpClass());
-					Class[] constructor_Args = new Class[] {com.cybersource.ws.client.MerchantConfig.class, javax.xml.parsers.DocumentBuilder.class, com.cybersource.ws.client.LoggerWrapper.class}; 
-					con=customConnectionClass.getDeclaredConstructor(constructor_Args).newInstance(mc, builder, logger);
-
-				} catch (InstantiationException e) {
-					logger.log(Logger.LT_INFO, "Failed to Instantiate the class "+e);
-					throw new ClientException(e, false, null);
-				} catch (IllegalAccessException e) {
-					logger.log(Logger.LT_INFO, "Could not Access the method invoked "+e);
-					throw new ClientException(e, false, null);
-				} catch (ClassNotFoundException e) {
-					logger.log(Logger.LT_INFO, "Could not load the custom HTTP class ");
-					throw new ClientException(e, false, null);
-				} catch (IllegalArgumentException e) {
-					logger.log(Logger.LT_INFO, "Method invoked with Illegal Argument list  "+e);
-					throw new ClientException(e, false, null);
-				} catch (SecurityException e) {
-					logger.log(Logger.LT_INFO, "Security Exception "+e);
-					throw new ClientException(e, false, null);
-				} catch (InvocationTargetException e) {
-					logger.log(Logger.LT_INFO, "Exception occured while calling the method "+e);
-					throw new ClientException(e, false, null);
-				} catch (NoSuchMethodException e) {
-					logger.log(Logger.LT_INFO, "Method not found ");
-					throw new ClientException(e, false, null);
-				}  	
-            }
-            else{
-            	con = Connection.getInstance(mc, builder, logger);
-            }
+            
+            con = Connection.getInstance(mc, builder, logger);
             Document wrappedReply = con.post(signedDoc);
             Map<String, String> replyMap = soapUnwrap(wrappedReply, mc, logger);
             logger.log(Logger.LT_INFO, "Client, End of runTransaction Call   ",false);

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

@@ -47,15 +47,8 @@ abstract class Connection {
     final MerchantConfig mc;
     private final DocumentBuilder builder;
     final LoggerWrapper logger;
-    
-   /**
-    * It initializes three arguments MerchantConfig, DocumentBuilder and Logger
-    * Any class extending this class must implement three argument constructor
-    * @param mc
-    * @param builder
-    * @param logger
- */
-protected Connection(MerchantConfig mc, DocumentBuilder builder,
+
+    Connection(MerchantConfig mc, DocumentBuilder builder,
                LoggerWrapper logger) {
         this.mc = mc;
         this.builder = builder;

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

@@ -60,13 +60,7 @@ void postDocument(Document request)
             throws IOException, TransformerConfigurationException,
             TransformerException, MalformedURLException,
             ProtocolException {
-    	
-    	/*
-    	 * SimpleHttpConnectionManager(boolean alwaysClose) : 
-    	 * alwaysClose - if set true, the connection manager will always close connections upon release.
-    	 */
-    	
-        HttpClient httpClient = new HttpClient(new SimpleHttpConnectionManager(true));
+        HttpClient httpClient = new HttpClient();
         setTimeout(httpClient, mc.getTimeout() * 1000);
         setProxy(httpClient);
 

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

@@ -64,14 +64,14 @@ public class MerchantConfig {
     private String proxyUser;
     private String proxyPassword;
     private String cacertPassword;
-    private String customHttpClass;
-    private boolean customHttpClassEnabled;
+    private String useClientHttpFactory;
+    private boolean useClientHttpFactoryFlag;
 
-    public String getcustomHttpClass() {
-		return customHttpClass;
+    public String getUseClientHttpFactory() {
+		return useClientHttpFactory;
 	}
-    public boolean isCustomHttpClassEnabled() {
-		return customHttpClassEnabled;
+    public boolean getUseClientHttpFactoryFlag() {
+		return useClientHttpFactoryFlag;
 	}
 	// computed values
     private String effectiveServerURL;
@@ -261,7 +261,7 @@ public MerchantConfig(Properties _props, String _merchantID)
         logFilename = getProperty(merchantID, "logFilename");
         logMaximumSize = getIntegerProperty(merchantID, "logMaximumSize", 10);
         useHttpClient = getBooleanProperty(merchantID, "useHttpClient", ConnectionHelper.getDefaultUseHttpClient());
-        customHttpClass = getProperty(merchantID, "customHttpClass");
+        useClientHttpFactory = getProperty(merchantID, "useClientHttpFactory");
         timeout = getIntegerProperty(merchantID, "timeout", DEFAULT_TIMEOUT);
         proxyHost = getProperty(merchantID, "proxyHost");
         proxyPort = getIntegerProperty(merchantID, "proxyPort", DEFAULT_PROXY_PORT);
@@ -270,7 +270,7 @@ public MerchantConfig(Properties _props, String _merchantID)
         enableJdkCert = getBooleanProperty(merchantID, "enableJdkCert", false);
         enableCacert=getBooleanProperty(merchantID, "enableCacert", false);
         cacertPassword=getProperty(merchantID,"cacertPassword","changeit");
-        customHttpClassEnabled=getBooleanProperty(merchantID,"customHttpClassEnabled",false);
+        useClientHttpFactoryFlag=getBooleanProperty(merchantID,"useClientHttpFactoryFlag",false);
         // compute and store effective namespace URI
 
         if (namespaceURI == null && targetAPIVersion == null) {
@@ -473,8 +473,8 @@ public String getLogString() {
         appendPair(sb, "logDirectory", logDirectory);
         appendPair(sb, "logFilename", logFilename);
         appendPair(sb, "logMaximumSize", logMaximumSize);
-        appendPair(sb, "customHttpClass", customHttpClass);
-        appendPair(sb, "customHttpClassEnabled", customHttpClassEnabled);
+        appendPair(sb, "useClientHttpFactory", useClientHttpFactory);
+        appendPair(sb, "useClientHttpFactoryFlag", useClientHttpFactoryFlag);
         appendPair(sb, "useHttpClient", useHttpClient);
         appendPair(sb, "enableJdkCert", enableJdkCert);
         appendPair(sb, "enableCacert", enableCacert);

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

@@ -48,7 +48,7 @@ private Utility() {
     /**
      * Version number of this release.
      */
-    public static final String VERSION = "6.2.5";
+    public static final String VERSION = "6.2.3";
 
     /**
      * If in the Request map, a key called "_has_escapes" is present and is set

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

@@ -35,7 +35,6 @@
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.StringReader;
-import java.lang.reflect.InvocationTargetException;
 import java.util.Properties;
 
 /**
@@ -65,8 +64,7 @@ public class XMLClient {
 
     private static Document soapEnvelope;
     private static Exception initException = null;
-
-
+    
     static {
         try {
             // load the SOAP envelope document.
@@ -139,8 +137,7 @@ public static Document runTransaction(Document request, Properties props)
      * @throws FaultException  if a fault occurs.
      * @throws ClientException if any other exception occurs.
      */
-    @SuppressWarnings("unchecked")
-	public static Document runTransaction(
+    public static Document runTransaction(
             Document request, Properties props,
             Logger _logger, boolean prepare, boolean logTranStart)
             throws FaultException, ClientException {
@@ -180,39 +177,8 @@ public static Document runTransaction(
 
             Document signedDoc
                     = soapWrapAndSign(request, mc, builder, logger);
-            if(mc.isCustomHttpClassEnabled()){
-				Class<Connection> customConnectionClass;
-				try {
-					customConnectionClass = (Class<Connection>) Class.forName(mc.getcustomHttpClass());
-					Class[] constructor_Args = new Class[] {com.cybersource.ws.client.MerchantConfig.class, javax.xml.parsers.DocumentBuilder.class, com.cybersource.ws.client.LoggerWrapper.class}; 
-					con=customConnectionClass.getDeclaredConstructor(constructor_Args).newInstance(mc, builder, logger);
-
-				} catch (InstantiationException e) {
-					logger.log(Logger.LT_INFO, "Failed to Instantiate the class "+e);
-					throw new ClientException(e, false, null);
-				} catch (IllegalAccessException e) {
-					logger.log(Logger.LT_INFO, "Could not Access the method invoked "+e);
-					throw new ClientException(e, false, null);
-				} catch (ClassNotFoundException e) {
-					logger.log(Logger.LT_INFO, "Could not load the custom HTTP class ");
-					throw new ClientException(e, false, null);
-				} catch (IllegalArgumentException e) {
-					logger.log(Logger.LT_INFO, "Method invoked with Illegal Argument list  "+e);
-					throw new ClientException(e, false, null);
-				} catch (SecurityException e) {
-					logger.log(Logger.LT_INFO, "Security Exception "+e);
-					throw new ClientException(e, false, null);
-				} catch (InvocationTargetException e) {
-					logger.log(Logger.LT_INFO, "Exception occured while calling the method "+e);
-					throw new ClientException(e, false, null);
-				} catch (NoSuchMethodException e) {
-					logger.log(Logger.LT_INFO, "Method not found ");
-					throw new ClientException(e, false, null);
-				}    	
-            }
-            else{
-            	con = Connection.getInstance(mc, builder, logger);
-            }
+
+            con = Connection.getInstance(mc, builder, logger);
             Document wrappedReply = con.post(signedDoc);
 
             Document doc = soapUnwrap(wrappedReply, mc, builder, logger);

java/src/main/resources/cybs.properties

@@ -17,8 +17,8 @@ sendToAkamai=true
 
 #To use your own Http library set useClientHttpFactoryFlag to true
 # set useClientHttpFactory field with the class name which has Http implementation
-#useClientHttpFactory=
-#useClientHttpFactoryFlag=false
+#useClientHttpFactory=AmazonHttpConnection
+#useClientHttpFactoryFlag=true
 
 #useHttpClient=false
 
@@ -32,12 +32,6 @@ allowRetry=true
 numberOfRetries=5
 retryInterval=5
 
-
-#To use your own Http library set customHttpClassEnabled to true
-# set customHttpClass field with the class name which has Http implementation. Provide the full class with package name.
-#customHttpClassEnabled=
-#customHttpClass=
-
 # If This property is set to true then the p12 certificate must be stored in JKS format
 # program will read it from there. If it is set to false then the certificate will be read from 
 # the location specified above from the key directory location