fixed setting custom timeout value (JENKINS-29719)

fixed issue https://issues.jenkins-ci.org/browse/JENKINS-29719 it is possible now to set custom request timeout. code on the line 91 doesn't do it (I don't understand why)
jenkinsci · Jun 23, 2017 · 9433eae · 9433eae
1 parent 06bff90
commit 9433eae
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/graniteclient/GraniteClientExecutor.java b/src/main/java/org/jenkinsci/plugins/graniteclient/GraniteClientExecutor.java
@@ -38,6 +38,7 @@
 import com.cloudbees.plugins.credentials.common.UsernamePasswordCredentials;
 import com.ning.http.client.AsyncCompletionHandler;
 import com.ning.http.client.AsyncHttpClient;
+import com.ning.http.client.AsyncHttpClientConfig;
 import com.ning.http.client.Response;
 import hudson.model.TaskListener;
 import hudson.util.LogTaskListener;
@@ -73,7 +74,14 @@ public static <T> T execute(PackageManagerClientCallable<T> callable, GraniteCli
                                 TaskListener _listener) throws Exception {
         final TaskListener listener = _listener != null ? _listener : DEFAULT_LISTENER;
 
-        final AsyncHttpClient ahcClient = config.getGlobalConfig().getInstance();
+        final GraniteClientGlobalConfig globalConfig = config.getGlobalConfig();
+        final AsyncHttpClient ahcClient = new AsyncHttpClient(new AsyncHttpClientConfig.Builder().setProxyServer(globalConfig.getProxyServer())
+                .setConnectTimeout(globalConfig.getConnectionTimeoutInMs())
+                .setReadTimeout(config.getServiceTimeout() > 0 ? (int) config.getServiceTimeout() : globalConfig.getIdleConnectionTimeoutInMs())
+                .setRequestTimeout(config.getRequestTimeout() > 0 ? (int) config.getRequestTimeout() : globalConfig.getRequestTimeoutInMs())
+                .build()
+        );
+
         try {
             AsyncPackageManagerClient client = new AsyncPackageManagerClient(ahcClient);