platine-php / webauthn

src/Exception/WebauthnException.php

Braces +1 added lines, -2 removed lines

@@ -10,6 +10,5 @@
  * @class WebauthnException
  * @package Platine\Webauthn\Exception
  */
-class WebauthnException extends Exception
-{
+class WebauthnException extends Exception {
 }

src/Attestation/AttestationData.php

Indentation +3 added lines, -3 removed lines

@@ -169,9 +169,9 @@
     }
 
     /**
-    * {@inheritdoc}
-    * @return mixed
-    */
+     * {@inheritdoc}
+     * @return mixed
+     */
     public function jsonSerialize()
     {
         return get_object_vars($this);

Spacing +4 added lines, -4 removed lines

@@ -48,22 +48,22 @@
     {
         $enc = CborDecoder::decode($binary);
 
-        if (! is_array($enc) || ! array_key_exists('fmt', $enc) || ! is_string($enc['fmt'])) {
+        if (!is_array($enc) || !array_key_exists('fmt', $enc) || !is_string($enc['fmt'])) {
             throw new WebauthnException('Invalid attestation format provided');
         }
 
-        if (! array_key_exists('attStmt', $enc) || ! is_array($enc['attStmt'])) {
+        if (!array_key_exists('attStmt', $enc) || !is_array($enc['attStmt'])) {
             throw new WebauthnException('Invalid attestation format provided (attStmt not available)');
         }
 
-        if (! array_key_exists('authData', $enc) || ! $enc['authData'] instanceof ByteBuffer) {
+        if (!array_key_exists('authData', $enc) || !$enc['authData'] instanceof ByteBuffer) {
             throw new WebauthnException('Invalid attestation format provided (authData not available)');
         }
 
         $this->formatName = $enc['fmt'];
         $this->authenticatorData = new AuthenticatorData($enc['authData']->getBinaryString());
 
-        if (! in_array($this->formatName, $allowedFormats)) {
+        if (!in_array($this->formatName, $allowedFormats)) {
             throw new WebauthnException(sprintf(
                 'Invalid attestation format [%s], allowed [%s]',
                 $this->formatName,

Braces +4 added lines, -8 removed lines

@@ -19,8 +19,7 @@ 
  * @class AttestationData
  * @package Platine\Webauthn\Attestation
  */
-class AttestationData implements JsonSerializable
-{
+class AttestationData implements JsonSerializable {
     /**
      * The AuthenticatorData instance
      * @var AuthenticatorData
@@ -44,8 +43,7 @@ 
      * @param string $binary
      * @param array<string> $allowedFormats
      */
-    public function __construct(string $binary, array $allowedFormats)
-    {
+    public function __construct(string $binary, array $allowedFormats) {
         $enc = CborDecoder::decode($binary);
 
         if (! is_array($enc) || ! array_key_exists('fmt', $enc) || ! is_string($enc['fmt'])) {
@@ -88,8 +86,7 @@ 
      *
      * @return BaseFormat|FidoU2F|None|Packed
      */
-    public function getFormat()
-    {
+    public function getFormat() {
         return $this->format;
     }
 
@@ -172,8 +169,7 @@ 
     * {@inheritdoc}
     * @return mixed
     */
-    public function jsonSerialize()
-    {
+    public function jsonSerialize() {
         return get_object_vars($this);
     }
 

src/Attestation/AuthenticatorData.php

Spacing +1 added lines, -1 removed lines

@@ -284,7 +284,7 @@
 
         $extensionData = CborDecoder::decode($data);
 
-        if (! is_array($extensionData)) {
+        if (!is_array($extensionData)) {
             throw new WebauthnException('Invalid extension data');
         }
 

Braces +3 added lines, -6 removed lines

@@ -15,8 +15,7 @@ 
  * @class AuthenticatorData
  * @package Platine\Webauthn\Attestation
  */
-class AuthenticatorData implements JsonSerializable
-{
+class AuthenticatorData implements JsonSerializable {
     public const EC2_TYPE = 2;
     public const RSA_TYPE = 3;
 
@@ -60,8 +59,7 @@ 
      * Create new instance
      * @param string $binary
      */
-    public function __construct(string $binary)
-    {
+    public function __construct(string $binary) {
         if (strlen($binary) < 37) {
             throw new WebauthnException('Invalid authenticator data provided');
         }
@@ -225,8 +223,7 @@ 
     * {@inheritdoc}
     * @return mixed
     */
-    public function jsonSerialize()
-    {
+    public function jsonSerialize() {
         return get_object_vars($this);
     }
 

Indentation +3 added lines, -3 removed lines

@@ -270,9 +270,9 @@
 
 
     /**
-    * {@inheritdoc}
-    * @return mixed
-    */
+     * {@inheritdoc}
+     * @return mixed
+     */
     public function jsonSerialize()
     {
         return get_object_vars($this);

src/Attestation/Format/None.php

Indentation +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
 class None extends BaseFormat
 {
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateAttestation(string $clientData): bool
     {
         return true;

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
  * @class None
  * @package Platine\Webauthn\Attestation\Format
  */
-class None extends BaseFormat
-{
+class None extends BaseFormat {
     /**
     * {@inheritdoc}
     */

src/Attestation/Format/FidoU2F.php

Indentation +6 added lines, -6 removed lines

@@ -79,8 +79,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function getCertificatePem(): string
     {
         $pem = '-----BEGIN CERTIFICATE-----' . "\n";
@@ -91,8 +91,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateAttestation(string $clientData): bool
     {
         $publicKey = openssl_pkey_get_public($this->getCertificatePem());
@@ -129,8 +129,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateRootCertificate(array $rootCertificates): bool
     {
         $chain = $this->createX5cChainFile();

Spacing +5 added lines, -5 removed lines

@@ -56,21 +56,21 @@
         }
 
         if (
-            ! array_key_exists('sig', $attestationStatement) ||
-            ! $attestationStatement['sig'] instanceof ByteBuffer
+            !array_key_exists('sig', $attestationStatement) ||
+            !$attestationStatement['sig'] instanceof ByteBuffer
         ) {
             throw new WebauthnException('No signature found');
         }
 
         if (
-            ! array_key_exists('x5c', $attestationStatement) ||
-            ! is_array($attestationStatement['x5c']) ||
+            !array_key_exists('x5c', $attestationStatement) ||
+            !is_array($attestationStatement['x5c']) ||
             count($attestationStatement['x5c']) !== 1
         ) {
             throw new WebauthnException('Invalid X5C certificate');
         }
 
-        if (! $attestationStatement['x5c'][0] instanceof ByteBuffer) {
+        if (!$attestationStatement['x5c'][0] instanceof ByteBuffer) {
             throw new WebauthnException('Invalid X5C certificate');
         }
 

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
  * @class FidoU2F
  * @package Platine\Webauthn\Attestation\Format
  */
-class FidoU2F extends BaseFormat
-{
+class FidoU2F extends BaseFormat {
     /**
      * The algorithm used
      * @var int

src/Attestation/Format/Packed.php

Indentation +6 added lines, -6 removed lines

@@ -88,8 +88,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function getCertificatePem(): ?string
     {
         if (empty($this->x5c)) {
@@ -100,8 +100,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateAttestation(string $clientData): bool
     {
         if (! empty($this->x5c)) {
@@ -112,8 +112,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateRootCertificate(array $rootCertificates): bool
     {
         if (empty($this->x5c)) {

Spacing +5 added lines, -5 removed lines

@@ -46,7 +46,7 @@ 
         // check u2f data
         $attestationStatement = $this->attestationData['attStmt'];
         if (
-            ! array_key_exists('alg', $attestationStatement) ||
+            !array_key_exists('alg', $attestationStatement) ||
             $this->getCoseAlgorithm($attestationStatement['alg']) === null
         ) {
             throw new WebauthnException(sprintf(
@@ -56,8 +56,8 @@ 
         }
 
         if (
-            ! array_key_exists('sig', $attestationStatement) ||
-            ! $attestationStatement['sig'] instanceof ByteBuffer
+            !array_key_exists('sig', $attestationStatement) ||
+            !$attestationStatement['sig'] instanceof ByteBuffer
         ) {
             throw new WebauthnException('No signature found');
         }
@@ -72,7 +72,7 @@ 
         ) {
             // The attestation certificate attestnCert MUST be the first element in the array
             $attestCert = array_shift($attestationStatement['x5c']);
-            if (! $attestCert instanceof ByteBuffer) {
+            if (!$attestCert instanceof ByteBuffer) {
                 throw new WebauthnException('Invalid X5C certificate');
             }
 
@@ -104,7 +104,7 @@ 
     */
     public function validateAttestation(string $clientData): bool
     {
-        if (! empty($this->x5c)) {
+        if (!empty($this->x5c)) {
             return $this->validateOverX5C($clientData);
         }
 

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
  * @class Packed
  * @package Platine\Webauthn\Attestation\Format
  */
-class Packed extends BaseFormat
-{
+class Packed extends BaseFormat {
     /**
      * The algorithm used
      * @var int

src/Attestation/Format/BaseFormat.php

Spacing +10 added lines, -10 removed lines

@@ -188,7 +188,7 @@ 
                 'hash' => 'SHA1',
                 'openssl' => OPENSSL_ALGO_SHA1,
                 'cose' => [
-                    -65535,  // RS1
+                    -65535, // RS1
                 ],
             ],
             [
@@ -196,9 +196,9 @@ 
                 'openssl' => OPENSSL_ALGO_SHA256,
                 'cose' => [
                     -257, // RS256
-                    -37,  // PS256
-                    -7,   // ES256
-                    5,     // HMAC256
+                    -37, // PS256
+                    -7, // ES256
+                    5, // HMAC256
                 ],
             ],
             [
@@ -206,9 +206,9 @@ 
                 'openssl' => OPENSSL_ALGO_SHA384,
                 'cose' => [
                     -258, // RS384
-                    -38,  // PS384
-                    -35,  // ES384
-                    6,     // HMAC384
+                    -38, // PS384
+                    -35, // ES384
+                    6, // HMAC384
                 ],
             ],
             [
@@ -216,9 +216,9 @@ 
                 'openssl' => OPENSSL_ALGO_SHA512,
                 'cose' => [
                     -259, // RS512
-                    -39,  // PS512
-                    -36,  // ES512
-                    7,     // HMAC512
+                    -39, // PS512
+                    -36, // ES512
+                    7, // HMAC512
                 ],
             ],
         ];

Braces +3 added lines, -6 removed lines

@@ -11,8 +11,7 @@ 
  * @class BaseFormat
  * @package Platine\Webauthn\Attestation\Format
  */
-abstract class BaseFormat implements JsonSerializable
-{
+abstract class BaseFormat implements JsonSerializable {
     /**
      * The attestation object data
      * @var array<string, mixed>
@@ -53,8 +52,7 @@ 
     /**
      * Destructor
      */
-    public function __destruct()
-    {
+    public function __destruct() {
         // delete X.509 chain certificate file after use
         if ($this->x5cTempFile !== null && is_file($this->x5cTempFile)) {
             unlink($this->x5cTempFile);
@@ -112,8 +110,7 @@ 
     * {@inheritdoc}
     * @return mixed
     */
-    public function jsonSerialize()
-    {
+    public function jsonSerialize() {
         return get_object_vars($this);
     }
 

Indentation +3 added lines, -3 removed lines

@@ -169,9 +169,9 @@
     }
 
     /**
-    * {@inheritdoc}
-    * @return mixed
-    */
+     * {@inheritdoc}
+     * @return mixed
+     */
     public function jsonSerialize()
     {
         return get_object_vars($this);

src/Attestation/Format/Tpm.php

Indentation +7 added lines, -7 removed lines

@@ -132,8 +132,8 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function getCertificatePem(): ?string
     {
         if (empty($this->x5c)) {
@@ -144,16 +144,16 @@ 
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateAttestation(string $clientData): bool
     {
         return $this->validateOverX5C($clientData);
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     public function validateRootCertificate(array $rootCertificates): bool
     {
         if (empty($this->x5c)) {
@@ -206,7 +206,7 @@ 
         $dataToVerify = $this->authenticatorData->getBinary();
         $dataToVerify .= $clientData;
 
-         // Verify that magic is set to TPM_GENERATED_VALUE.
+            // Verify that magic is set to TPM_GENERATED_VALUE.
         if ($this->certInfo->getBytes(0, 4) !== self::TPM_GENERATED_VALUE) {
             throw new WebauthnException('TPM magic value not the same TPM_GENERATED_VALUE');
         }

Spacing +9 added lines, -9 removed lines

@@ -62,7 +62,7 @@ 
         $attestationStatement = $this->attestationData['attStmt'];
 
         if (
-            ! array_key_exists('ver', $attestationStatement) ||
+            !array_key_exists('ver', $attestationStatement) ||
             $attestationStatement['ver'] !== '2.0'
         ) {
             throw new WebauthnException(sprintf(
@@ -72,7 +72,7 @@ 
         }
 
         if (
-            ! array_key_exists('alg', $attestationStatement) ||
+            !array_key_exists('alg', $attestationStatement) ||
             $this->getCoseAlgorithm($attestationStatement['alg']) === null
         ) {
             throw new WebauthnException(sprintf(
@@ -82,22 +82,22 @@ 
         }
 
         if (
-            ! array_key_exists('sig', $attestationStatement) ||
-            ! $attestationStatement['sig'] instanceof ByteBuffer
+            !array_key_exists('sig', $attestationStatement) ||
+            !$attestationStatement['sig'] instanceof ByteBuffer
         ) {
             throw new WebauthnException('No signature found');
         }
 
         if (
-            ! array_key_exists('certInfo', $attestationStatement) ||
-            ! $attestationStatement['certInfo'] instanceof ByteBuffer
+            !array_key_exists('certInfo', $attestationStatement) ||
+            !$attestationStatement['certInfo'] instanceof ByteBuffer
         ) {
             throw new WebauthnException('No certificate information found');
         }
 
         if (
-            ! array_key_exists('pubArea', $attestationStatement) ||
-            ! $attestationStatement['pubArea'] instanceof ByteBuffer
+            !array_key_exists('pubArea', $attestationStatement) ||
+            !$attestationStatement['pubArea'] instanceof ByteBuffer
         ) {
             throw new WebauthnException('No public area information found');
         }
@@ -114,7 +114,7 @@ 
         ) {
             // The attestation certificate attestnCert MUST be the first element in the array
             $attestCert = array_shift($attestationStatement['x5c']);
-            if (! $attestCert instanceof ByteBuffer) {
+            if (!$attestCert instanceof ByteBuffer) {
                 throw new WebauthnException('Invalid X5C certificate');
             }
 

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
  * @class Tpm
  * @package Platine\Webauthn\Attestation\Format
  */
-class Tpm extends BaseFormat
-{
+class Tpm extends BaseFormat {
     public const TPM_GENERATED_VALUE = "\xFF\x54\x43\x47";
     public const TPM_ST_ATTEST_CERTIFY = "\x80\x17";
     /**

src/WebauthnConfiguration.php

Braces +1 added lines, -2 removed lines

@@ -10,8 +10,7 @@
  * @class WebauthnConfiguration
  * @package Platine\Webauthn
  */
-class WebauthnConfiguration extends AbstractConfiguration
-{
+class WebauthnConfiguration extends AbstractConfiguration {
     /**
      * {@inheritdoc}
      */