LoginCidadao\OpenIDBundle\Storage\PublicKey

Complexity

Total Complexity

7

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getPublicKey()	0	5	1
A	getEncryptionAlgorithm()	0	3	1
A	getPrivateKey()	0	6	1
A	setFilesystem()	0	4	1
A	getPrivateKeyResource()	0	11	2
A	createKeys()	0	9	1

<?php
/**
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\OpenIDBundle\Storage;

use Gaufrette\Filesystem;
use Gaufrette\Exception\FileNotFound;
use OAuth2\Storage\PublicKeyInterface;

class PublicKey implements PublicKeyInterface
{
    /** @var Filesystem */
    private $filesystem;

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

    public function setFilesystem(Filesystem $filesystem, $fileName = 'private.pem')
    {
        $this->filesystem = $filesystem;
        $this->fileName = $fileName;
    }

    public function getEncryptionAlgorithm($client_id = null)
    {
        return 'RS256';
    }

    public function getPrivateKey($client_id = null)
    {
        $key = $this->getPrivateKeyResource();
        openssl_pkey_export($key, $priv);

        return $priv;
    }

    public function getPublicKey($client_id = null)
    {
        $details = openssl_pkey_get_details($this->getPrivateKeyResource());

        return $details['key'];
    }

    /**
     * @return \Gaufrette\File
     */
    private function createKeys()
    {
        $priv = null;
        $key = openssl_pkey_new();
        openssl_pkey_export($key, $priv);
        $file = $this->filesystem->get($this->fileName, true);
        $file->setContent($priv);

        return $file;
    }

    /**
     * @return resource
     */
    private function getPrivateKeyResource()
    {
        try {
            $file = $this->filesystem->get($this->fileName);
        } catch (FileNotFound $e) {
            $file = $this->createKeys();
        }

        $key = openssl_pkey_get_private($file->getContent());

        return $key;

The expression return $key could also return false which is incompatible with the documented return type resource. Did you maybe forget to handle an error condition?

    }
}