fixed #12

src/BucketClient.php

@@ -155,7 +155,7 @@ public function getInventory(string $id): Http\Response
         return $this->get(\sprintf('/?inventory&id=%s', $id));
     }
 
-    public function getInventoryConfigurations(?string $nextContinuationToken = null): Http\Response
+    public function getInventoryConfigurations(string $nextContinuationToken = null): Http\Response
     {
         return $this->get(\sprintf('/?inventory&continuation-token=%s', $nextContinuationToken));
     }

src/Client.php

@@ -44,8 +44,6 @@ class Client
     protected array $requiredConfigKeys = [];
 
     /**
-     * @param  array|\Overtrue\CosClient\Config  $config
-     *
      * @throws \Overtrue\CosClient\Exceptions\InvalidConfigException
      */
     public function __construct(array|Config $config)

src/Http/Response.php

@@ -6,7 +6,7 @@
 use Overtrue\CosClient\Support\XML;
 use Psr\Http\Message\ResponseInterface;
 
-class Response extends \GuzzleHttp\Psr7\Response implements \JsonSerializable, \ArrayAccess
+class Response extends \GuzzleHttp\Psr7\Response implements \ArrayAccess, \JsonSerializable
 {
     protected ?array $arrayResult = null;
 
@@ -128,19 +128,19 @@ final public function isServerError(): bool
     #[Pure]
     final public function isOk(): bool
     {
-        return 200 === $this->getStatusCode();
+        return $this->getStatusCode() === 200;
     }
 
     #[Pure]
     final public function isForbidden(): bool
     {
-        return 403 === $this->getStatusCode();
+        return $this->getStatusCode() === 403;
     }
 
     #[Pure]
     final public function isNotFound(): bool
     {
-        return 404 === $this->getStatusCode();
+        return $this->getStatusCode() === 404;
     }
 
     #[Pure]

src/ServiceClient.php

@@ -4,7 +4,7 @@
 
 class ServiceClient extends Client
 {
-    public function getBuckets(?string $region = null): Http\Response
+    public function getBuckets(string $region = null): Http\Response
     {
         $uri = $region ? \sprintf('https://cos.%s.myqcloud.com/', $region) : 'https://service.cos.myqcloud.com/';
 

src/Signature.php

@@ -17,13 +17,13 @@ class Signature
         'range',
     ];
 
-    public function __construct(public string $accessKey, public  string $secretKey)
+    public function __construct(public string $accessKey, public string $secretKey)
     {
     }
 
     public function createAuthorizationHeader(RequestInterface $request, int|string|\DateTimeInterface $expires = null): string
     {
-        $signTime = self::getTimeSegments($expires);
+        $signTime = self::getTimeSegments($expires ?? '+60 minutes');
         $queryToBeSigned = self::getQueryToBeSigned($request);
         $headersToBeSigned = self::getHeadersToBeSigned($request);
 

tests/SignatureTest.php

@@ -41,4 +41,23 @@ public function test_getTimeSegments()
 
         date_default_timezone_set($timezone);
     }
+
+    public function test_getTimeSegments_with_null()
+    {
+        $signature = new Signature('mock-access-key', 'mock-secret-key');
+
+        $request = new Request('GET', 'https://example.com');
+
+        $timezone = \date_default_timezone_get();
+
+        date_default_timezone_set('PRC');
+
+        $output = strtotime('+60 minutes');
+        $expect = sprintf('q-sign-time=%d;%d&q-key-time=%d;%d', time() - 60, $output, time() - 60, $output);
+
+        $this->assertStringContainsString($expect, $signature->createAuthorizationHeader($request));
+        $this->assertStringContainsString($expect, $signature->createAuthorizationHeader($request, null));
+
+        date_default_timezone_set($timezone);
+    }
 }