Fix JENKINS-76283: httpRequest ignores default timeout 0 and enforces 5 minutes

When the timeout parameter is not provided or set to 0 (documented as
'no timeout'), httpclient5 was using its default 5-minute timeout instead.

This fixes the regression introduced in v1.22 during the httpclient5
migration (PR jenkinsci#197) by explicitly setting timeout to DISABLED when
timeout <= 0, restoring the documented behavior.

Author: sgscheffler

src/main/java/jenkins/plugins/http_request/HttpRequestExecution.java

@@ -411,20 +411,33 @@ private void configureTimeoutAndSsl(HttpClientBuilder clientBuilder) throws NoSu
             connectionManager = new PoolingHttpClientConnectionManager(socketFactoryRegistry);
         }
 
-        //timeout
-        if (timeout > 0) {
-            int t = timeout * 1000;
-            RequestConfig config = RequestConfig.custom()
-                    .setResponseTimeout(t, TimeUnit.MILLISECONDS)
-                    .setConnectionRequestTimeout(t, TimeUnit.MILLISECONDS)
-                    .build();
-            clientBuilder.setDefaultRequestConfig(config);
-
-            ConnectionConfig connectionConfig = ConnectionConfig.custom()
-                    .setConnectTimeout(t, TimeUnit.MILLISECONDS)
-                    .build();
-            connectionManager.setDefaultConnectionConfig(connectionConfig);
-        }
+    //timeout
+    if (timeout > 0) {
+        int t = timeout * 1000;
+        RequestConfig config = RequestConfig.custom()
+                .setResponseTimeout(t, TimeUnit.MILLISECONDS)
+                .setConnectionRequestTimeout(t, TimeUnit.MILLISECONDS)
+                .build();
+        clientBuilder.setDefaultRequestConfig(config);
+
+        ConnectionConfig connectionConfig = ConnectionConfig.custom()
+                .setConnectTimeout(t, TimeUnit.MILLISECONDS)
+                .build();
+        connectionManager.setDefaultConnectionConfig(connectionConfig);
+    } else {
+        // timeout is 0 or not provided, meaning "no timeout" per documentation
+        // Explicitly set to DISABLED to prevent httpclient5 from using its 5-minute default
+        RequestConfig config = RequestConfig.custom()
+                .setResponseTimeout(org.apache.hc.core5.util.Timeout.DISABLED)
+                .setConnectionRequestTimeout(org.apache.hc.core5.util.Timeout.DISABLED)
+                .build();
+        clientBuilder.setDefaultRequestConfig(config);
+
+        ConnectionConfig connectionConfig = ConnectionConfig.custom()
+                .setConnectTimeout(org.apache.hc.core5.util.Timeout.DISABLED)
+                .build();
+        connectionManager.setDefaultConnectionConfig(connectionConfig);
+    }
 
         clientBuilder.setConnectionManager(connectionManager);
     }