NamshiAdapter::decode() - Code Metrics - thecsea/jwt-auth - Measure and Improve Code Quality continuously with Scrutinizer

NamshiAdapter::decode() A
last analyzed 2016-01-03 14:36 UTC

↳ Parent: NamshiAdapter

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3.7086

Metric	Value
dl	0
loc	14
ccs	4
cts	7
cp	0.5714
rs	9.4286
cc	3
eloc	8
nc	3
nop	1
crap	3.7086

<?php

namespace Tymon\JWTAuth\Providers\JWT;

use Exception;
use Namshi\JOSE\JWS;
use Tymon\JWTAuth\Exceptions\JWTException;
use Tymon\JWTAuth\Exceptions\TokenInvalidException;

class NamshiAdapter extends JWTProvider implements JWTInterface
{
    /**
     * @var \Namshi\JOSE\JWS
     */
    protected $jws;

    /**
     * @param string  $secret
     * @param string  $algo
     * @param null    $driver
     */
    public function __construct($secret, $algo, $driver = null)
    {
        parent::__construct($secret, $algo);

        $this->jws = $driver ?: new JWS(['typ' => 'JWT', 'alg' => $algo]);
    }

    /**
     * Create a JSON Web Token
     *
     * @return string
     * @throws \Tymon\JWTAuth\Exceptions\JWTException
     */
    public function encode(array $payload)
    {
        try {
            $this->jws->setPayload($payload)->sign($this->secret);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

            return $this->jws->getTokenString();
        } catch (Exception $e) {
            throw new JWTException('Could not create token: ' . $e->getMessage());
        }
    }

    /**
     * Decode a JSON Web Token
     *
     * @param  string  $token
     * @return array
     * @throws \Tymon\JWTAuth\Exceptions\JWTException
     */
    public function decode($token)
    {
        try {
            $jws = JWS::load($token);
        } catch (Exception $e) {
            throw new TokenInvalidException('Could not decode token: ' . $e->getMessage());
        }

        if (! $jws->verify($this->secret, $this->algo)) {
            throw new TokenInvalidException('Token Signature could not be verified.');
        }

        return $jws->getPayload();
    }
}


1		<?php
2
3		namespace Tymon\JWTAuth\Providers\JWT;
4
5		use Exception;
6		use Namshi\JOSE\JWS;
7		use Tymon\JWTAuth\Exceptions\JWTException;
8		use Tymon\JWTAuth\Exceptions\TokenInvalidException;
9
10		class NamshiAdapter extends JWTProvider implements JWTInterface
11		{
12		/**
13		* @var \Namshi\JOSE\JWS
14		*/
15		protected $jws;
16
17		/**
18		* @param string $secret
19		* @param string $algo
20		* @param null $driver
21		*/
22	9	public function __construct($secret, $algo, $driver = null)
23		{
24	9	parent::__construct($secret, $algo);
25
26	9	$this->jws = $driver ?: new JWS(['typ' => 'JWT', 'alg' => $algo]);
27	9	}
28
29		/**
30		* Create a JSON Web Token
31		*
32		* @return string
33		* @throws \Tymon\JWTAuth\Exceptions\JWTException
34		*/
35	6	public function encode(array $payload)
36		{
37		try {
38	6	$this->jws->setPayload($payload)->sign($this->secret);
		0 ignored issues – show Documentation introduced 2015-11-20 23:27 UTC by Report Bug Copy Issue Report `$this->secret` is of type `string`, but the function expects a `resource`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
39
40	3	return $this->jws->getTokenString();
41	3	} catch (Exception $e) {
42	3	throw new JWTException('Could not create token: ' . $e->getMessage());
43		}
44		}
45
46		/**
47		* Decode a JSON Web Token
48		*
49		* @param string $token
50		* @return array
51		* @throws \Tymon\JWTAuth\Exceptions\JWTException
52		*/
53	3	public function decode($token)
54		{
55		try {
56	3	$jws = JWS::load($token);
57	3	} catch (Exception $e) {
58	3	throw new TokenInvalidException('Could not decode token: ' . $e->getMessage());
59		}
60
61		if (! $jws->verify($this->secret, $this->algo)) {
62		throw new TokenInvalidException('Token Signature could not be verified.');
63		}
64
65		return $jws->getPayload();
66		}
67		}
68

thecsea / jwt-auth

NamshiAdapter::decode() A last analyzed 2016-01-03 14:36 UTC

Complexity

Size

Duplication

Code Coverage

Duplication Side-by-Side

Filter issues like

NamshiAdapter::decode() A
last analyzed 2016-01-03 14:36 UTC