Skip to content

Commit

Permalink
refactor: inherit LtiException for backward compatibility
Browse files Browse the repository at this point in the history
  • Loading branch information
pnal committed Nov 21, 2024
1 parent 2a7a9a0 commit e8c4819
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions src/Exception/LtiBadRequestException.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@

namespace OAT\Library\Lti1p3Core\Exception;

use Exception;

class LtiBadRequestException extends Exception implements LtiExceptionInterface
class LtiBadRequestException extends LtiException implements LtiExceptionInterface
{
}
2 changes: 1 addition & 1 deletion src/Security/Oidc/OidcAuthenticator.php
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ public function authenticate(ServerRequestInterface $request): LtiMessageInterfa
throw new LtiBadRequestException('Missing LTI message hint in request');
}

$originalToken = $this->parser->parse($oidcRequest->getParameters()->get('lti_message_hint'));
$originalToken = $this->parser->parse($oidcRequest->getParameters()->getMandatory('lti_message_hint'));

$registration = $this->repository->find(
$originalToken->getClaims()->getMandatory(LtiMessagePayloadInterface::CLAIM_REGISTRATION_ID)
Expand Down

0 comments on commit e8c4819

Please sign in to comment.