Merge pull request #85 from MichalSkoula/api_constructor_mb_strlen

fixes mb_strlen(): Passing null to parameter #1 ($string) of type str…
Salamek · Jun 29, 2024 · 7f93ae7 · 7f93ae7
2 parents 85c8b45 + 22901ee
commit 7f93ae7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Api.php b/src/Api.php
@@ -97,11 +97,11 @@ class Api
      */
     public function __construct($username = null, $password = null, $customerId = null, $storage = null, $trace = true)
     {
-        if (mb_strlen($username) > 32) {
+        if ($username !== null && mb_strlen($username) > 32) {
             throw new SecurityException('$username is longer than 32 characters');
         }
 
-        if (mb_strlen($password) > 32) {
+        if ($password !== null && mb_strlen($password) > 32) {
             throw new SecurityException('$password is longer than 32 characters');
         }