rhertogh\Yii2Oauth2Server\controllers\web\openidconnect\Oauth2OidcEndSessionAction

Complexity

Total Complexity

10

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Test Coverage

Coverage

75.76%

Importance

Changes	1
Bugs	0	Features	0

1 Method

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

<?php

namespace rhertogh\Yii2Oauth2Server\controllers\web\openidconnect;

// phpcs:disable Generic.Files.LineLength.TooLong
use rhertogh\Yii2Oauth2Server\controllers\web\base\Oauth2BaseWebAction;
use rhertogh\Yii2Oauth2Server\controllers\web\Oauth2OidcController;
use rhertogh\Yii2Oauth2Server\interfaces\components\authorization\EndSession\Oauth2EndSessionAuthorizationRequestInterface;
use rhertogh\Yii2Oauth2Server\interfaces\controllers\web\openidconnect\Oauth2OidcEndSessionActionInterface;
use rhertogh\Yii2Oauth2Server\interfaces\models\external\user\Oauth2OidcUserInterface;
use Yii;
use yii\base\InvalidConfigException;
use yii\web\BadRequestHttpException;
use yii\web\ForbiddenHttpException;
use yii\web\NotFoundHttpException;
use yii\web\Response;
use yii\web\UnauthorizedHttpException;

Header blocks must be separated by a single blank line

// phpcs:enable Generic.Files.LineLength.TooLong

/**
 * @property Oauth2OidcController $controller
 */
class Oauth2OidcEndSessionAction extends Oauth2BaseWebAction implements Oauth2OidcEndSessionActionInterface
{
    /**
     * @see https://openid.net/specs/openid-connect-rpinitiated-1_0.html
     * @return Response
     * @throws InvalidConfigException
     */
    public function run($endSessionAuthorizationRequestId = null)
    {
        $module = $this->controller->module;
        $request = Yii::$app->request;

It seems like Yii::app->request can also be of type yii\web\Request. However, the property $request is declared as type yii\console\Request. Maybe add an additional type check?

        $identity = $module->getUserIdentity();

Are you sure the assignment to $identity is correct as $module->getUserIdentity() targeting rhertogh\Yii2Oauth2Server\Oauth2Module::getUserIdentity() seems to always return null.

        if (!$module->enableOpenIdConnect) {
            throw new ForbiddenHttpException('OpenID Connect is disabled.');
        }

        if ($identity && !($identity instanceof Oauth2OidcUserInterface)) {

$identity is of type null, thus it always evaluated to false.

            throw new InvalidConfigException('In order to support OpenID Connect '
                . get_class($identity) . ' must implement ' . Oauth2OidcUserInterface::class);
        }

        if (empty($endSessionAuthorizationRequestId)) {
            /** @var Oauth2EndSessionAuthorizationRequestInterface $endSessionAuthorizationRequest */
            $endSessionAuthorizationRequest = Yii::createObject([
                'class' => Oauth2EndSessionAuthorizationRequestInterface::class,
                'module' => $module,
                'idTokenHint' => $this->getRequestParam($request, 'id_token_hint'),

It seems like $request can also be of type yii\console\Request; however, parameter $request of rhertogh\Yii2Oauth2Server\controllers\web\base\Oauth2BaseWebAction::getRequestParam() does only seem to accept yii\web\Request, maybe add an additional type check?

                'clientIdentifier' => $this->getRequestParam($request, 'client_id'),
                'endSessionUrl' => $request->absoluteUrl,
                'redirectUri' => $this->getRequestParam($request, 'post_logout_redirect_uri'),
                'state' => $this->getRequestParam($request, 'state'),
            ]);

            $endSessionAuthorizationRequest->validateRequest();

            if (!$identity) {

$identity is of type null, thus it always evaluated to false.

                /**
                 * Specified in https://openid.net/specs/openid-connect-rpinitiated-1_0.html#ValidationAndErrorHandling
                 * "Note that because RP-Initiated Logout Requests are intended to be idempotent,
                 *  it is explicitly not an error for an RP to request that a logout be performed when the OP does not
                 *  consider that the End-User is logged in with the OP at the requesting RP."
                 */
                return $this->controller->redirect($endSessionAuthorizationRequest->getRequestCompletedRedirectUrl(true));
            }

            if ($endSessionAuthorizationRequest->getEndUserAuthorizationRequired()) {
                if ($endSessionAuthorizationRequest->isAuthorizationAllowed()) {
                    return $module->generateEndSessionAuthReqRedirectResponse($endSessionAuthorizationRequest);
                } else {
                    throw new UnauthorizedHttpException(Yii::t('oauth2', 'You are not allowed to authorize logging out.'));
                }
            } else {
                $endSessionAuthorizationRequest->autoApproveAndProcess();
            }
        } else {
            $endSessionAuthorizationRequest = $module->getEndSessionAuthReqSession($endSessionAuthorizationRequestId);
            if (empty($endSessionAuthorizationRequest)) {
                throw new NotFoundHttpException('End Session authorization request not found.');
            }
        }

        if (!$endSessionAuthorizationRequest->isCompleted()) {
            throw new BadRequestHttpException('End Session authorization is not completed.');
        }

        return $this->controller->redirect($endSessionAuthorizationRequest->getRequestCompletedRedirectUrl());
    }
}