Gandung\JWT\Algorithm\ECDSA

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	sign()	0	8	1
A	verify()	0	9	1
A	getPrivateKey()	0	4	1
A	getPublicKey()	0	4	1

<?php

namespace Gandung\JWT\Algorithm;

use Gandung\JWT\Adapter\ECDSAAdapterInterface;
use Gandung\JWT\SignerInterface;
use Gandung\JWT\Manager\KeyManagerInterface;
use Mdanter\Ecc\Crypto\Key\PrivateKeyInterface;
use Mdanter\Ecc\Crypto\Key\PublicKeyInterface;
use Mdanter\Ecc\Serializer\PrivateKey\DerPrivateKeySerializer;
use Mdanter\Ecc\Serializer\PrivateKey\PemPrivateKeySerializer;
use Mdanter\Ecc\Serializer\PublicKey\DerPublicKeySerializer;
use Mdanter\Ecc\Serializer\PublicKey\PemPublicKeySerializer;

/**
 * @author Paulus Gandung Prakosa <rvn.plvhx@gmail.com>
 */
abstract class ECDSA implements SignerInterface
{
    /**
     * @var ECDSAAdapterInterface
     */
    private $adapter;

    /**
     * @var $private
     */
    private $private;

    /**
     * @var $public
     */
    private $public;

    public function __construct(ECDSAAdapterInterface $adapter)
    {
        $this->adapter = $adapter;
        $publicDer = new DerPublicKeySerializer($this->adapter->getMathAdapter());
        $this->public = new PemPublicKeySerializer($publicDer);
        $this->private = new PemPrivateKeySerializer(
            new DerPrivateKeySerializer($this->adapter->getMathAdapter(), $publicDer)
        );
    }

    /**
     * {@inheritdoc}
     */
    public function sign($payload, KeyManagerInterface $key)
    {
        return $this->adapter->sign(
            $this->getPrivateKey($key),
            $this->adapter->createSigningHash($payload, $this->getAlgorithm()),

It seems like $payload defined by parameter $payload on line 48 can also be of type array; however, Gandung\JWT\Adapter\ECDSAAdapterInterface::createSigningHash() does only seem to accept string, maybe add an additional type check?

            $this->getAlgorithm()
        );
    }

    /**
     * {@inheritdoc}
     */
    public function verify($expected, $payload, KeyManagerInterface $key)
    {
        return $this->adapter->verify(
            $expected,
            $this->getPublicKey($key),
            $this->adapter->createSigningHash($payload, $this->getAlgorithm()),

It seems like $payload defined by parameter $payload on line 60 can also be of type array; however, Gandung\JWT\Adapter\ECDSAAdapterInterface::createSigningHash() does only seem to accept string, maybe add an additional type check?

            $this->getAlgorithm()
        );
    }

    /**
     * Get private key object from given certificate.
     *
     * @param string $key
     * @return PrivateKeyInterface
     */
    public function getPrivateKey(KeyManagerInterface $key)
    {
        return $this->private->parse($key->getContent());
    }

    /**
     * Get public key object from given certificate.
     *
     * @param string $key
     * @return PublicKeyInterface
     */
    public function getPublicKey(KeyManagerInterface $key)
    {
        return $this->public->parse($key->getContent());
    }
}