Migrate code to PHP 8.3

Author: tvdijen

src/Certificate/Key.php

@@ -19,9 +19,9 @@
 class Key implements ArrayAccess
 {
     // Possible key usages
-    public const USAGE_SIGNING = 'signing';
+    public const string USAGE_SIGNING = 'signing';
 
-    public const USAGE_ENCRYPTION = 'encryption';
+    public const string USAGE_ENCRYPTION = 'encryption';
 
 
     /** @var array */
@@ -43,8 +43,6 @@ public function __construct(array $keyData)
     /**
      * Whether or not the key is configured to be used for usage given
      *
-     * @param string $usage
-     * @return bool
      * @throws \SimpleSAML\SAML2\Exception\InvalidArgumentException
      */
     public function canBeUsedFor(string $usage): bool
@@ -62,8 +60,8 @@ public function canBeUsedFor(string $usage): bool
 
     /**
      * @param mixed $offset
+     *
      * @throws \SimpleSAML\SAML2\Exception\InvalidArgumentException
-     * @return bool
      */
     public function offsetExists(mixed $offset): bool
     {
@@ -75,9 +73,7 @@ public function offsetExists(mixed $offset): bool
 
 
     /**
-     * @param mixed $offset
      * @throws \SimpleSAML\SAML2\Exception\InvalidArgumentException
-     * @return mixed
      */
     public function offsetGet($offset): mixed
     {
@@ -89,8 +85,6 @@ public function offsetGet($offset): mixed
 
 
     /**
-     * @param mixed $offset
-     * @param mixed $value
      * @throws \SimpleSAML\SAML2\Exception\InvalidArgumentException
      */
     public function offsetSet(mixed $offset, mixed $value): void
@@ -103,7 +97,6 @@ public function offsetSet(mixed $offset, mixed $value): void
 
 
     /**
-     * @param mixed $offset
      * @throws \SimpleSAML\SAML2\Exception\InvalidArgumentException
      */
     public function offsetUnset(mixed $offset): void

src/Configuration/PrivateKey.php

@@ -11,20 +11,13 @@
  */
 class PrivateKey extends ArrayAdapter
 {
-    /** @var string */
-    public const NAME_NEW = 'new';
+    public const string NAME_NEW = 'new';
 
-    /** @var string */
-    public const NAME_DEFAULT = 'default';
+    public const string NAME_DEFAULT = 'default';
 
 
     /**
      * Constructor for PrivateKey.
-     *
-     * @param string $filePathOrContents
-     * @param string $name
-     * @param string $passphrase
-     * @param bool $isFile
      */
     public function __construct(
         private string $filePathOrContents,
@@ -36,8 +29,6 @@ public function __construct(
 
 
     /**
-     * @return string
-     *
      * @throws \SimpleSAML\Assert\AssertionFailedException if assertions are false
      */
     public function getFilePath(): string
@@ -49,7 +40,6 @@ public function getFilePath(): string
 
 
     /**
-     * @return bool
      */
     public function hasPassPhrase(): bool
     {
@@ -58,7 +48,6 @@ public function hasPassPhrase(): bool
 
 
     /**
-     * @return string
      */
     public function getPassPhrase(): string
     {
@@ -67,7 +56,6 @@ public function getPassPhrase(): string
 
 
     /**
-     * @return string
      */
     public function getName(): string
     {
@@ -76,8 +64,6 @@ public function getName(): string
 
 
     /**
-     * @return string
-     *
      * @throws \SimpleSAML\Assert\AssertionFailedException if assertions are false
      */
     public function getContents(): string
@@ -89,7 +75,6 @@ public function getContents(): string
 
 
     /**
-     * @return bool
      */
     public function isFile(): bool
     {