TokenMask::remove() - Code Metrics - Inspection of "Additional renaming for #3" - yiisoft/security - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3e2247...bd6b53 )

by Alexander

created 2019-09-16 19:27 UTC

TokenMask::remove() A

↳ Parent: TokenMask

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes

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

<?php declare(strict_types=1);

namespace Yiisoft\Security;

use Yiisoft\Strings\StringHelper;

/**
 * TokenMask helps to mitigate BREACH attack by randomizing how token is outputted on each request.
 * A random mask is applied to the token making the string always unique.
 */
final class TokenMask
{
    /**
     * Masks a token to make it uncompressible.
     * Applies a random mask to the token and prepends the mask used to the result making the string always unique.
     * @param string $token An unmasked token.
     * @return string A masked token.
     * @throws \Exception if unable to securely generate random bytes
     */
    public static function apply(string $token): string
    {
        // The number of bytes in a mask is always equal to the number of bytes in a token.
        $mask = random_bytes(StringHelper::byteLength($token));
        return StringHelper::base64UrlEncode($mask . ($mask ^ $token));
    }

    /**
     * Unmasks a token previously masked by `mask`.
     * @param string $maskedToken A masked token.
     * @return string An unmasked token, or an empty string in case of token format is invalid.
     */
    public static function remove(string $maskedToken): string
    {
        $decoded = StringHelper::base64UrlDecode($maskedToken);
        $length = StringHelper::byteLength($decoded) / 2;
        // Check if the masked token has an even length.
        if (!is_int($length)) {

            return '';
        }

        return StringHelper::byteSubstr($decoded, $length, $length) ^ StringHelper::byteSubstr($decoded, 0, $length);
    }
}


1		<?php declare(strict_types=1);
2
3		namespace Yiisoft\Security;
4
5		use Yiisoft\Strings\StringHelper;
6
7		/**
8		* TokenMask helps to mitigate BREACH attack by randomizing how token is outputted on each request.
9		* A random mask is applied to the token making the string always unique.
10		*/
11		final class TokenMask
12		{
13		/**
14		* Masks a token to make it uncompressible.
15		* Applies a random mask to the token and prepends the mask used to the result making the string always unique.
16		* @param string $token An unmasked token.
17		* @return string A masked token.
18		* @throws \Exception if unable to securely generate random bytes
19		*/
20	5	public static function apply(string $token): string
21		{
22		// The number of bytes in a mask is always equal to the number of bytes in a token.
23	5	$mask = random_bytes(StringHelper::byteLength($token));
24	4	return StringHelper::base64UrlEncode($mask . ($mask ^ $token));
25		}
26
27		/**
28		* Unmasks a token previously masked by `mask`.
29		* @param string $maskedToken A masked token.
30		* @return string An unmasked token, or an empty string in case of token format is invalid.
31		*/
32	6	public static function remove(string $maskedToken): string
33		{
34	6	$decoded = StringHelper::base64UrlDecode($maskedToken);
35	6	$length = StringHelper::byteLength($decoded) / 2;
36		// Check if the masked token has an even length.
37	6	if (!is_int($length)) {
		0 ignored issues – show introduced 2019-06-27 22:30 UTC by Report Bug Copy Issue Report The condition `is_int($length)` is always `true`. Loading history...
38	1	return '';
39		}
40
41	5	return StringHelper::byteSubstr($decoded, $length, $length) ^ StringHelper::byteSubstr($decoded, 0, $length);
42		}
43		}
44

yiisoft / security

Push — master ( 3e2247...bd6b53 )

TokenMask::remove() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like