rhertogh\Yii2Oauth2Server\controllers\console\client\Oauth2SetClientSecretAction

Complexity

Total Complexity

10

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	run()	0	59	10

<?php

namespace rhertogh\Yii2Oauth2Server\controllers\console\client;

use League\OAuth2\Server\Grant\GrantTypeInterface;
use rhertogh\Yii2Oauth2Server\controllers\console\Oauth2ClientController;
use rhertogh\Yii2Oauth2Server\helpers\DiHelper;
use rhertogh\Yii2Oauth2Server\interfaces\components\openidconnect\scope\Oauth2OidcScopeCollectionInterface;
use rhertogh\Yii2Oauth2Server\interfaces\models\Oauth2ClientInterface;
use rhertogh\Yii2Oauth2Server\interfaces\models\Oauth2ClientScopeInterface;
use rhertogh\Yii2Oauth2Server\Oauth2Module;
use Yii;
use yii\base\Action;
use yii\base\InvalidArgumentException;
use yii\base\InvalidCallException;
use yii\console\ExitCode;
use yii\helpers\Console;
use yii\helpers\StringHelper;

/**
 * @property Oauth2ClientController $controller
 */
class Oauth2SetClientSecretAction extends Action
{
    /**
     * @throws \yii\base\InvalidConfigException
     */
    public function run()
    {
        $module = $this->controller->module;

        $identifier = $this->controller->identifier;
        $secret = $this->controller->secret;
        $oldSecretValidUntilInput = $this->controller->oldSecretValidUntil;

        if (empty($identifier)) {
            throw new InvalidCallException('The `identifier` option must be specified.');
        }

        /** @var Oauth2ClientInterface $client */
        $client = $module->getClientRepository()->findModelByIdentifier($identifier);
        if (empty($client)) {
            throw new InvalidCallException('No client with identifier "' . $identifier . '" found.');
        }

        if (empty($secret)) {
            $secret = $this->controller->prompt('Client Secret?', [
                'required' => true,
                'validator' => [$client, 'validateNewSecret'],
            ]);
        }

        $oldSecretValidUntil = null;
        if ($oldSecretValidUntilInput) {
            if (StringHelper::startsWith($oldSecretValidUntilInput, 'P')) {
                try {
                    $oldSecretValidUntil = new \DateInterval($oldSecretValidUntilInput);
                } catch (\Exception $e) {
                    $oldSecretValidUntil = false;
                }
            } else {
                try {
                    $oldSecretValidUntil = new \DateTimeImmutable($oldSecretValidUntilInput);
                } catch (\Exception $e) {
                    $oldSecretValidUntil = false;
                }
            }
            if (empty($oldSecretValidUntil)) {
                throw new InvalidArgumentException('Unable to parse "' . $oldSecretValidUntilInput
                    . '" as a DateTime or DateInterval.');
            }
        }

        $client->setSecret($secret, $module->getEncryptor(), $oldSecretValidUntil);

It seems like $oldSecretValidUntil can also be of type false; however, parameter $oldSecretValidUntil of rhertogh\Yii2Oauth2Server\interfaces\models\Oauth2ClientInterface::setSecret() does only seem to accept DateInterval|DateTimeImmutable|null, maybe add an additional type check?

        $client->persist();

        if ($oldSecretValidUntil) {
            $oldSecretValidUntilInfo = ' The previous secret will be valid until '
                . Yii::$app->formatter->asDatetime($client->getOldSecretValidUntil(), 'long');
        } else {
            $oldSecretValidUntilInfo = ' Any previous secret is cleared.';
        }
        $this->controller->stdout('Successfully updated client secret.' . $oldSecretValidUntilInfo
            . PHP_EOL, Console::FG_GREEN);

        return ExitCode::OK;
    }
}