Merge pull request #48 from johmanx10/issue-47

Fix passing null to parameter of type string

Author: glaubinix

.github/workflows/php.yml

@@ -18,6 +18,7 @@ jobs:
           - "7.3"
           - "7.4"
           - "8.0"
+          - "8.1"
 
     steps:
       - uses: actions/checkout@v2

src/HttpClient/Plugin/RequestSignature.php

@@ -84,6 +84,6 @@ private function normalizeParameters(array $params)
     {
         uksort($params, 'strcmp');
 
-        return http_build_query($params, null, '&', PHP_QUERY_RFC3986);
+        return http_build_query($params, '', '&', PHP_QUERY_RFC3986);
     }
 }

src/WebhookSignature.php

@@ -14,12 +14,12 @@ public function __construct($secret)
 
     /**
      * @param string $signature
-     * @param string $payload
+     * @param ?string $payload
      * @return bool
      */
     public function validate($signature, $payload)
     {
-        $payloadSignature = 'sha1='.hash_hmac('sha1', $payload, $this->secret);
+        $payloadSignature = 'sha1='.hash_hmac('sha1', $payload ?? '', $this->secret);
 
         return hash_equals($payloadSignature, (string) $signature);
     }