Make HTTP client timeout configurable

2 files changed, 4 insertions, 16 deletions

diff --git a/app/Core/Http/Client.php b/app/Core/Http/Client.php
index 3793fa33..84099a23 100644
--- a/app/Core/Http/Client.php
+++ b/app/Core/Http/Client.php
@@ -14,20 +14,6 @@ use Kanboard\Job\HttpAsyncJob;
 class Client extends Base
 {
     /**
-     * HTTP connection timeout in seconds
-     *
-     * @var integer
-     */
-    const HTTP_TIMEOUT = 10;
-
-    /**
-     * Number of maximum redirections for the HTTP client
-     *
-     * @var integer
-     */
-    const HTTP_MAX_REDIRECTS = 2;
-
-    /**
      * HTTP client user agent
      *
      * @var string
@@ -225,8 +211,8 @@ class Client extends Base
             'http' => [
                 'method' => $method,
                 'protocol_version' => 1.1,
-                'timeout' => self::HTTP_TIMEOUT,
-                'max_redirects' => self::HTTP_MAX_REDIRECTS,
+                'timeout' => HTTP_TIMEOUT,
+                'max_redirects' => HTTP_MAX_REDIRECTS,
                 'header' => implode("\r\n", $headers),
                 'content' => $content,
                 'ignore_errors' => $raiseForErrors,
diff --git a/app/constants.php b/app/constants.php
index 9d9d0a0f..af26856f 100644
--- a/app/constants.php
+++ b/app/constants.php
@@ -142,6 +142,8 @@ defined('BRUTEFORCE_LOCKDOWN_DURATION') or define('BRUTEFORCE_LOCKDOWN_DURATION'
 defined('SESSION_DURATION') or define('SESSION_DURATION', 0);
 
 // HTTP Client
+defined('HTTP_TIMEOUT') or define('HTTP_TIMEOUT', 10);
+defined('HTTP_MAX_REDIRECTS') or define('HTTP_MAX_REDIRECTS', 3);
 defined('HTTP_PROXY_HOSTNAME') or define('HTTP_PROXY_HOSTNAME', '');
 defined('HTTP_PROXY_PORT') or define('HTTP_PROXY_PORT', '3128');
 defined('HTTP_PROXY_USERNAME') or define('HTTP_PROXY_USERNAME', '');