ByTIC\Hello\Oauth\ServiceProvider\Traits\CryptKeysTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

57.14%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	makeCryptKey()	0	3	1
A	appendCryptKeysToProvide()	0	5	1
A	registerCryptKeys()	0	7	1

<?php

namespace ByTIC\Hello\Oauth\ServiceProvider\Traits;

use ByTIC\Hello\Utility\ConfigHelper;
use ByTIC\Hello\Utility\CryptHelper;
use League\OAuth2\Server\CryptKey;

/**
 * Trait CryptKeysTrait
 * @package ByTIC\Hello\Oauth\ServiceProvider\Traits
 */
trait CryptKeysTrait
{
    public function registerCryptKeys()
    {
        $this->getContainer()->share('hello.keys.private', function () {

It seems like getContainer() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return $this->makeCryptKey('private');
        });
        $this->getContainer()->share('hello.keys.public', function () {
            return $this->makeCryptKey('public');
        });
    }

    /**
     * @param $return
     * @return array
     */
    protected function appendCryptKeysToProvide($return)
    {
        $return[] = 'hello.keys.private';
        $return[] = 'hello.keys.public';
        return $return;
    }

    /**
     * Create a CryptKey instance without permissions check
     *
     * @param  $type
     * @return \League\OAuth2\Server\CryptKey
     */
    protected function makeCryptKey($type)
    {
        return CryptHelper::makeCryptKey($type);
    }
}