+ Fixing naming convention

+ Scenarios covered for proxy setting with auth

Author: gnongsie

src/main/java/net/authorize/util/Constants.java

@@ -6,7 +6,7 @@ public final class Constants {
 	public static final String HTTPS_USE_PROXY = "https.proxyUse";
 	public static final String HTTPS_PROXY_HOST = "https.proxyHost";
 	public static final String HTTPS_PROXY_PORT = "https.proxyPort";
-	public static final String HTTPS_PROXY_USERNAME = "https.proxyUser";
+	public static final String HTTPS_PROXY_USERNAME = "https.proxyUsername";
     public static final String HTTPS_PROXY_PASSWORD = "https.proxyPassword";
 
 	public static final String HTTP_USE_PROXY = "http.proxyUse";

src/main/java/net/authorize/util/HttpClient.java

@@ -344,39 +344,44 @@ static CloseableHttpClient getHttpsClient() throws Exception {
 
 			CloseableHttpClient httpClient;
 
-			if ( !UseProxy && ( ProxyHost == null || proxyUsername == null || proxyPassword == null ) ) {
-				LogHelper.warn(logger, "Defaulting to non-proxy environment");
-				
-				httpClient =  HttpClients.custom()
-						.setSSLSocketFactory(sslSocketFactory)
-						.setDefaultRequestConfig(requestConfig)
-						.setRedirectStrategy(new LaxRedirectStrategy())
-						.build();
-			} 
-			
-			else {
+			if ( UseProxy && ProxyHost != null) {
 
 				LogHelper.info(logger, "Setting up proxy to URL: '%s://%s:%d'", Constants.PROXY_PROTOCOL, ProxyHost, ProxyPort);
-				CredentialsProvider credsProvider = new BasicCredentialsProvider();
-
-				AuthScope proxyScope = new AuthScope(ProxyHost, ProxyPort);
-				Credentials proxyCreds = new UsernamePasswordCredentials(proxyUsername, proxyPassword);
-				credsProvider.setCredentials(proxyScope, proxyCreds);
-				
-				HttpClientBuilder hcBuilder = HttpClients.custom()
-						.setSSLSocketFactory(sslSocketFactory)
-						.setDefaultRequestConfig(requestConfig)
-						.setRedirectStrategy(new LaxRedirectStrategy())
-	                    .setDefaultCredentialsProvider(credsProvider);
+				HttpClientBuilder hcBuilder;
+				if (proxyUsername != null && proxyPassword != null) {
+					CredentialsProvider credsProvider = new BasicCredentialsProvider();
+					AuthScope proxyScope = new AuthScope(ProxyHost, ProxyPort);
+					Credentials proxyCreds = new UsernamePasswordCredentials(proxyUsername, proxyPassword);
+					credsProvider.setCredentials(proxyScope, proxyCreds);
+					hcBuilder = HttpClients.custom()
+								.setSSLSocketFactory(sslSocketFactory)
+								.setDefaultRequestConfig(requestConfig)
+								.setRedirectStrategy(new LaxRedirectStrategy())
+								.setDefaultCredentialsProvider(credsProvider);
+				}
+				else {
+					hcBuilder = HttpClients.custom()
+								.setSSLSocketFactory(sslSocketFactory)
+								.setDefaultRequestConfig(requestConfig)
+								.setRedirectStrategy(new LaxRedirectStrategy());
+				}
 
 				HttpHost httpProxy = new HttpHost(ProxyHost, ProxyPort, Constants.PROXY_PROTOCOL);
-	            DefaultProxyRoutePlanner routePlanner = new DefaultProxyRoutePlanner(httpProxy);
-	            hcBuilder.setRoutePlanner(routePlanner);
+	            hcBuilder.setProxy(httpProxy);
 
 	            httpClient = hcBuilder.build();
 
 				proxySet = true;
-			}			
+			}
+			else {
+				LogHelper.warn(logger, "Defaulting to non-proxy environment");
+				
+				httpClient =  HttpClients.custom()
+						.setSSLSocketFactory(sslSocketFactory)
+						.setDefaultRequestConfig(requestConfig)
+						.setRedirectStrategy(new LaxRedirectStrategy())
+						.build();
+			} 
 
 			return httpClient;
 		} catch (Exception e) {