Merge pull request #144 from packbackbooks/mgmt-155-lti-error

MGMT-155 Update state not found error message

src/DeepLinkResources/Icon.php

@@ -12,8 +12,7 @@ public function __construct(
         private string $url,
         private int $width,
         private int $height
-    ) {
-    }
+    ) {}
 
     public static function new(string $url, int $width, int $height): self
     {

src/DeepLinkResources/Iframe.php

@@ -12,8 +12,7 @@ public function __construct(
         private ?string $src = null,
         private ?int $width = null,
         private ?int $height = null
-    ) {
-    }
+    ) {}
 
     public static function new(): self
     {

src/DeepLinkResources/Window.php

@@ -13,8 +13,7 @@ public function __construct(
         private ?int $width = null,
         private ?int $height = null,
         private ?string $window_features = null
-    ) {
-    }
+    ) {}
 
     public static function new(): self
     {

src/JwksEndpoint.php

@@ -8,9 +8,7 @@
 
 class JwksEndpoint
 {
-    public function __construct(private array $keys)
-    {
-    }
+    public function __construct(private array $keys) {}
 
     public static function new(array $keys): self
     {

src/LtiAbstractService.php

@@ -12,8 +12,7 @@ public function __construct(
         private ILtiServiceConnector $serviceConnector,
         private ILtiRegistration $registration,
         private array $serviceData
-    ) {
-    }
+    ) {}
 
     public function getServiceData(): array
     {

src/LtiDeepLink.php

@@ -11,8 +11,7 @@ public function __construct(
         private ILtiRegistration $registration,
         private string $deployment_id,
         private array $deep_link_settings
-    ) {
-    }
+    ) {}
 
     public function getResponseJwt(array $resources): string
     {

src/LtiDeployment.php

@@ -8,8 +8,7 @@ class LtiDeployment implements ILtiDeployment
 {
     public function __construct(
         private $deployment_id
-    ) {
-    }
+    ) {}
 
     public static function new($deployment_id): self
     {

src/LtiException.php

@@ -4,6 +4,4 @@
 
 use Exception;
 
-class LtiException extends Exception
-{
-}
+class LtiException extends Exception {}

src/LtiMessageLaunch.php

@@ -27,9 +27,9 @@ class LtiMessageLaunch
     public const ERR_FETCH_PUBLIC_KEY = 'Failed to fetch public key.';
     public const ERR_NO_PUBLIC_KEY = 'Unable to find public key.';
     public const ERR_NO_MATCHING_PUBLIC_KEY = 'Unable to find a public key which matches your JWT.';
-    public const ERR_STATE_NOT_FOUND = 'Please make sure you have cookies enabled in this browser and that you are not in private or incognito mode';
+    public const ERR_STATE_NOT_FOUND = 'Please make sure you have cookies and cross-site tracking enabled in the privacy and security settings of your browser.';
     public const ERR_MISSING_ID_TOKEN = 'Missing id_token.';
-    public const ERR_INVALID_ID_TOKEN = 'Invalid id_token, JWT must contain 3 parts';
+    public const ERR_INVALID_ID_TOKEN = 'Invalid id_token, JWT must contain 3 parts.';
     public const ERR_MISSING_NONCE = 'Missing Nonce.';
     public const ERR_INVALID_NONCE = 'Invalid Nonce.';
 

src/LtiOidcLogin.php

@@ -19,8 +19,7 @@ public function __construct(
         public IDatabase $db,
         public ICache $cache,
         public ICookie $cookie
-    ) {
-    }
+    ) {}
 
     /**
      * Static function to allow for method chaining without having to assign to a variable first.

src/LtiServiceConnector.php

@@ -20,8 +20,7 @@ class LtiServiceConnector implements ILtiServiceConnector
     public function __construct(
         private ICache $cache,
         private Client $client
-    ) {
-    }
+    ) {}
 
     public function setDebuggingMode(bool $enable): self
     {

src/OidcException.php

@@ -4,6 +4,4 @@
 
 use Exception;
 
-class OidcException extends Exception
-{
-}
+class OidcException extends Exception {}

src/ServiceRequest.php

@@ -47,8 +47,7 @@ public function __construct(
         private string $method,
         private string $url,
         private string $type = self::TYPE_UNSUPPORTED
-    ) {
-    }
+    ) {}
 
     public function getMethod(): string
     {