Skip to content

Commit 8684c71

Browse files
committed
simplify code
1 parent 818579c commit 8684c71

File tree

1 file changed

+13
-23
lines changed
  • imagetool/src/main/java/com/oracle/weblogic/imagetool/util

1 file changed

+13
-23
lines changed

imagetool/src/main/java/com/oracle/weblogic/imagetool/util/HttpUtil.java

Lines changed: 13 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -124,32 +124,22 @@ public static HttpClient getOraClient(String userId, String password) {
124124
String proxyHost = System.getProperty("https.proxyHost");
125125
String proxyPort = System.getProperty("https.proxyPort");
126126
HttpClient result;
127+
CredentialsProvider credentialsProvider = new BasicCredentialsProvider();
127128

128-
if (proxyHost != null) {
129-
result = HttpClientBuilder.create()
130-
.setDefaultRequestConfig(config.build())
131-
.setRetryHandler(retryHandler())
132-
.setProxy(new HttpHost(proxyHost, Integer.parseInt(proxyPort)))
133-
.setUserAgent("Wget/1.10")
134-
.setDefaultCookieStore(cookieStore).useSystemProperties()
135-
.build();
136-
137-
} else {
138-
CredentialsProvider credentialsProvider = new BasicCredentialsProvider();
139-
140-
if (userId != null && password != null) {
141-
credentialsProvider.setCredentials(AuthScope.ANY, new UsernamePasswordCredentials(
142-
userId, password));
143-
}
144-
145-
result = HttpClientBuilder.create()
146-
.setDefaultRequestConfig(config.build())
147-
.setRetryHandler(retryHandler())
148-
.setUserAgent("Wget/1.10")
149-
.setDefaultCookieStore(cookieStore).useSystemProperties()
150-
.setDefaultCredentialsProvider(credentialsProvider).build();
129+
if (userId != null && password != null) {
130+
credentialsProvider.setCredentials(AuthScope.ANY, new UsernamePasswordCredentials(
131+
userId, password));
151132
}
152133

134+
result = HttpClientBuilder.create()
135+
.setDefaultRequestConfig(config.build())
136+
.setRetryHandler(retryHandler())
137+
.setProxy(proxyHost == null ? new HttpHost(proxyHost, Integer.parseInt(proxyPort)) : null)
138+
.setUserAgent("Wget/1.10")
139+
.setDefaultCookieStore(cookieStore).useSystemProperties()
140+
.setDefaultCredentialsProvider(credentialsProvider)
141+
.build();
142+
153143
logger.exiting();
154144
return result;
155145
}

0 commit comments

Comments
 (0)