Throw a more specific ProtocolViolationException

simplesamlphp · Jan 23, 2025 · 7177b3f · 7177b3f
1 parent a0727a2
commit 7177b3f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/Binding/HTTPPost.php b/src/Binding/HTTPPost.php
@@ -10,6 +10,7 @@
 use Psr\Http\Message\ServerRequestInterface;
 use SimpleSAML\Assert\Assert;
 use SimpleSAML\SAML2\Binding;
+use SimpleSAML\SAML2\Exception\ProtocolViolationException;
 use SimpleSAML\SAML2\Utils;
 use SimpleSAML\SAML2\XML\samlp\AbstractMessage;
 use SimpleSAML\SAML2\XML\samlp\AbstractRequest;
@@ -106,7 +107,8 @@ public function receive(ServerRequestInterface $request): AbstractMessage
          * message.
          */
         if ($msg->isSigned()) {
-            Assert::notNull($msg->getDestination()); // Validation of the value must be done upstream
+            Assert::notNull($msg->getDestination(), ProtocolViolationException::class);
+            // Validation of the Destination must be done upstream
         }
 
         if (array_key_exists('RelayState', $query)) {