SimpleSAML\Module\webauthn\Controller\RegProcess::main()

Complexity

Conditions	15
Paths	78

Size

Total Lines	151
Code Lines	88

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function main(Request $request): Response
    {
        $this->logger::info('FIDO2 - Accessing WebAuthn enrollment validation');

        $stateId = $request->query->get('StateId');
        if ($stateId === null) {
            throw new Error\BadRequest('Missing required StateId query parameter.');
        }

        $moduleConfig = Configuration::getOptionalConfig('module_webauthn.php');
        $debugEnabled = $moduleConfig->getOptionalBoolean('debug', false);

        $state = $this->authState::loadState($stateId, 'webauthn:request');

        // registering a credential is only allowed for new users or after being authenticated
        if (WebAuthn::workflowStateMachine($state) !== WebAuthn::STATE_MGMT) {

It seems like $state can also be of type null; however, parameter $state of SimpleSAML\Module\webauthn\Controller\WebAuthn::workflowStateMachine() does only seem to accept array, maybe add an additional type check?

            throw new Exception("Attempt to register new token in unacceptable context.");
        }

        $fido2Scope = ($state['FIDO2Scope'] === null ? $state['FIDO2DerivedScope'] : $state['FIDO2Scope']);
        if ($fido2Scope === null) {
            throw new Exception("FIDO2Scope cannot be null!");
        }

        $regObject = new WebAuthnRegistrationEvent(
            $request->request->get('type'),
            $fido2Scope,
            $state['FIDO2SignupChallenge'],
            base64_decode($request->request->get('attestation_object')),
            $request->request->get('response_id'),
            $request->request->get('attestation_client_data_json'),
            ($request->request->get('passwordless') == "on" ?
                    $state['authenticatorAcceptabilityPasswordless'] : $state['authenticatorAcceptability2FA']),
            $debugEnabled,

It seems like $debugEnabled can also be of type null; however, parameter $debugMode of SimpleSAML\Module\webauthn\WebAuthn\WebAuthnRegistrationEvent::__construct() does only seem to accept boolean, maybe add an additional type check?

        );
        // at this point, we need to talk to the DB
        /**
         * STEP 19 of the validation procedure in § 7.1 of the spec: see if this credential is already registered
         */
        $store = $state['webauthn:store'];
        if ($store->doesCredentialExist(bin2hex($regObject->getCredentialId())) === false) {
            // credential does not exist yet in database, good.
        } else {
            throw new Exception("The credential with ID " . $regObject->getCredentialId() . " already exists.");
        }

        // THAT'S IT. This is a valid credential and can be enrolled to the user.
        $friendlyName = $request->request->get('tokenname');

        // if we know the token model, add it to the name
        $model = Translate::noop('unknown model');
        $aaguiddict = AAGUID::getInstance();
        if ($aaguiddict->hasToken($regObject->getAAGUID())) {
            $token = $aaguiddict->get($regObject->getAAGUID());
            $model = $token['metadataStatement']['description'];
        }
        $friendlyName .= " ($model)";

        /**
         * STEP 20 of the validation procedure in § 7.1 of the spec: store credentialId, credential,
         * signCount and associate with user
         */

        /*
         * Observed with YubiKey 5: the transaction counter is 0 if the key has NEVER been used, but
         * the first transaction is also transaction #0.
         * i.e. 0 means "before first transaction OR the very first transaction has already taken place"
         *
         * The very first use of a key should not trigger the physical object cloning alert, so a
         * transaction counter == 0 should be allowed for the first authentication of a new key.
         * The best way to do this is to set the current counter value to -1 when registering a key
         * with a transaction counter of 0.
         */
        $currentCounterValue = -1;
        if ($regObject->getCounter() > 0) {
            $currentCounterValue = $regObject->getCounter();
        }

        // did we get any client extensions?
        $isResidentKey = 0;
        if (
            strlen($request->request->get('clientext')) > 0 &&
            count(json_decode($request->request->get('clientext'), true)) > 0
        ) {
            $extensions = json_decode($request->request->get('clientext'), true);
            if ($extensions['credProps']['rk'] === true) {
                $isResidentKey = 1;
            }
        }

        // we also need to store the hased user_id in case we need to retrieve
        // tokens in passwordless mode
        // use identical hashing as in JS generation step
        $configUtils = new Utils\Config();
        $username = hash('sha512', $state['FIDO2Username'] . '|' . $configUtils->getSecretSalt());
        $store->storeTokenData(
            $state['FIDO2Username'],
            $regObject->getCredentialId(),
            $regObject->getCredential(),
            $regObject->getAlgo(),
            $regObject->getPresenceLevel(),
            // we deny saving resident key properties if Passwordless mode
            // was not requested
            ($request->request->get('passwordless') == "on" ? $isResidentKey : 0),
            $currentCounterValue,
            $friendlyName,
            $username,
            $regObject->getAAGUID(),
            $regObject->getAttestationLevel(),
        );

        // make sure $state gets the news, the token is to be displayed to the user on the next page
        $state['FIDO2Tokens'][] = [
            0 => $regObject->getCredentialId(),
            1 => $regObject->getCredential(),
            2 => $currentCounterValue,
            3 => $friendlyName,
            4 => $regObject->getAlgo(),
            5 => $regObject->getPresenceLevel(),
            6 => $isResidentKey,
        ];

        $id = $this->authState::saveState($state, 'webauthn:request');

It seems like $state can also be of type null; however, parameter $state of SimpleSAML\Auth\State::saveState() does only seem to accept array, maybe add an additional type check?

        if ($debugEnabled === true) {
            $response = new RunnableResponse(
                function (WebAuthnRegistrationEvent $regObject, string $id) {
                    echo $regObject->getDebugBuffer();
                    echo $regObject->getValidateBuffer();
                    echo "<form id='regform' method='POST' action='" .
                        Module::getModuleURL('webauthn/webauthn?StateId=' . urlencode($id)) . "'>";
                    echo "<button type='submit'>Return to previous page.</button>";
                },
                [$regObject, $id],
            );
        } elseif (array_key_exists('Registration', $state)) {
            $response = new RedirectResponse(Module::getModuleURL('webauthn/webauthn?StateId=' . urlencode($id)));
        } else {
            $response = new RunnableResponse([Auth\ProcessingChain::class, 'resumeProcessing'], [$state]);
        }

        $response->setExpires(new DateTime('Thu, 19 Nov 1981 08:52:00 GMT'));
        $response->setCache([
            'must_revalidate'  => true,
            'no_cache'         => true,
            'no_store'         => true,
            'no_transform'     => false,
            'public'           => false,
            'private'          => false,
        ]);

        return $response;
    }