Lower curl default timeout time

src/Curl/CurlOptions.php

@@ -4,17 +4,20 @@
 
 class CurlOptions
 {
+    const DEFAULT_TIMEOUT = 5;
+    const DEFAULT_CONNECT_TIMEOUT = 3;
+
     /** @var array<string> */
     protected $headers = [];
 
     /** @var int */
     private $verifyHost = 0;
 
     /** @var int */
-    private $timeout = 30;
+    private $timeout = self::DEFAULT_TIMEOUT;
 
     /** @var int */
-    private $connectTimeout = 15;
+    private $connectTimeout = self::DEFAULT_CONNECT_TIMEOUT;
 
     /** @var bool */
     private $verifyPeer = false;

tests/Curl/CurlOptionsTest.php

@@ -0,0 +1,31 @@
+<?php
+
+namespace Tpay\Tests\OpenApi\Curl;
+
+use PHPUnit\Framework\TestCase;
+use Tpay\OpenApi\Curl\CurlOptions;
+
+/**
+ * @covers \Tpay\OpenApi\Curl\CurlOptions
+ */
+class CurlOptionsTest extends TestCase
+{
+    public function testDefaultTimeoutsAreShort()
+    {
+        $options = (new CurlOptions())->getOptionsArray();
+
+        self::assertSame(CurlOptions::DEFAULT_CONNECT_TIMEOUT, $options[CURLOPT_CONNECTTIMEOUT]);
+        self::assertSame(CurlOptions::DEFAULT_TIMEOUT, $options[CURLOPT_TIMEOUT]);
+    }
+
+    public function testTimeoutsCanBeOverridden()
+    {
+        $options = (new CurlOptions())
+            ->setConnectTimeout(15)
+            ->setTimeout(30)
+            ->getOptionsArray();
+
+        self::assertSame(15, $options[CURLOPT_CONNECTTIMEOUT]);
+        self::assertSame(30, $options[CURLOPT_TIMEOUT]);
+    }
+}