PODB-603: Protect against client ID being null.

JanHolger · Jul 20, 2022 · de328e5 · de328e5
1 parent 1c7ac41
commit de328e5
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/LtiMessageLaunch.php b/src/LtiMessageLaunch.php
@@ -283,8 +283,13 @@ public function getLaunchId()
         return $this->launch_id;
     }
 
-    public static function getMissingRegistrationErrorMsg(string $issuerUrl, string $clientId): string
+    public static function getMissingRegistrationErrorMsg(string $issuerUrl, ?string $clientId = null): string
     {
+        // Guard against client ID being null
+        if (!isset($clientId)) {
+            $clientId = '(N/A)';
+        }
+
         $search = [':issuerUrl', ':clientId'];
         $replace = [$issuerUrl, $clientId];