AbstractHashChecker::checkClaim() - Code Metrics - Inspection of "New implementation" - thomasvargiu/php-openid-client - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 898cc7...450add )

by Thomas Mauro

created 2019-07-08 11:59 UTC

AbstractHashChecker::checkClaim() A

↳ Parent: AbstractHashChecker

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	4
nc	2
nop	1
dl	0
loc	7
ccs	0
cts	6
cp	0
crap	6
rs	10
c	1
b	0
f	0

<?php

declare(strict_types=1);

namespace TMV\OpenIdClient\ClaimChecker;

use Jose\Component\Checker\ClaimChecker;
use Jose\Component\Checker\InvalidClaimException;
use function TMV\OpenIdClient\base64url_encode;

abstract class AbstractHashChecker implements ClaimChecker
{
    /** @var string */
    private $valueToCheck;

    /** @var string */
    private $alg;

    /**
     * SHashChecker constructor.
     *
     * @param string $valueToCheck
     * @param string $alg
     */
    public function __construct(string $valueToCheck, string $alg)
    {
        $this->valueToCheck = $valueToCheck;
        $this->alg = $alg;
    }

    private function getShaSize(string $alg): string
    {
        $size = \substr($alg, -3);

        switch ($size) {
            case '512':
                return 'sha512';
            case '384':
                return 'sha384';
            default:
                return 'sha256';
        }
    }

    /**
     * {@inheritdoc}
     */
    public function checkClaim($value): void
    {
        $hash = \hash($this->getShaSize($this->alg), $this->valueToCheck, true);
        $generated = base64url_encode(\substr($hash, 0, \strlen($hash) / 2));

        if ($value !== $generated) {
            throw new InvalidClaimException(\sprintf($this->supportedClaim() . ' mismatch, expected %s, got: %s', $generated, $value), $this->supportedClaim(), $value);
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace TMV\OpenIdClient\ClaimChecker;
6
7			use Jose\Component\Checker\ClaimChecker;
8			use Jose\Component\Checker\InvalidClaimException;
9			use function TMV\OpenIdClient\base64url_encode;
10
11			abstract class AbstractHashChecker implements ClaimChecker
12			{
13			/** @var string */
14			private $valueToCheck;
15
16			/** @var string */
17			private $alg;
18
19			/**
20			* SHashChecker constructor.
21			*
22			* @param string $valueToCheck
23			* @param string $alg
24			*/
25			public function __construct(string $valueToCheck, string $alg)
26			{
27			$this->valueToCheck = $valueToCheck;
28			$this->alg = $alg;
29			}
30
31			private function getShaSize(string $alg): string
32			{
33			$size = \substr($alg, -3);
34
35			switch ($size) {
36			case '512':
37			return 'sha512';
38			case '384':
39			return 'sha384';
40			default:
41			return 'sha256';
42			}
43			}
44
45			/**
46			* {@inheritdoc}
47			*/
48			public function checkClaim($value): void
49			{
50			$hash = \hash($this->getShaSize($this->alg), $this->valueToCheck, true);
51			$generated = base64url_encode(\substr($hash, 0, \strlen($hash) / 2));
52
53			if ($value !== $generated) {
54			throw new InvalidClaimException(\sprintf($this->supportedClaim() . ' mismatch, expected %s, got: %s', $generated, $value), $this->supportedClaim(), $value);
55			}
56			}
57			}
58

thomasvargiu / php-openid-client

Push — master ( 898cc7...450add )

AbstractHashChecker::checkClaim() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like