PageRequestAccount::saveWithoutEmailConfirmation() - Code Metrics - Inspection of "CSRF protection on the request form" - enwikipedia-acc/waca - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Pull Request — master (#1066)

by Simon
created 2025-10-27 23:47 UTC
PageRequestAccount::saveWithoutEmailConfirmation() A

↳ Parent: PageRequestAccount
Complexity

Conditions	2
Paths	2
Size

Total Lines	13
Code Lines	7
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	0
CRAP Score	6
Importance

Changes
Metric	Value
eloc	7
c	0
b	0
f	0
dl	0
loc	13
ccs	0
cts	6
cp	0
rs	10
cc	2
nc	2
nop	2
crap	6
<?php
/******************************************************************************
 * Wikipedia Account Creation Assistance tool                                 *
 * ACC Development Team. Please see team.json for a list of contributors.     *
 *                                                                            *
 * This is free and unencumbered software released into the public domain.    *
 * Please see LICENSE.md for the full licencing statement.                    *
 ******************************************************************************/

namespace Waca\Pages\Request;

use Exception;
use Waca\DataObjects\Comment;
use Waca\DataObjects\Domain;
use Waca\DataObjects\Request;
use Waca\DataObjects\RequestData;
use Waca\DataObjects\RequestForm;
use Waca\DataObjects\RequestQueue;
use Waca\Exceptions\ApplicationLogicException;
use Waca\Exceptions\OptimisticLockFailedException;
use Waca\Helpers\BanHelper;
use Waca\Helpers\MarkdownRenderingHelper;
use Waca\SessionAlert;
use Waca\Tasks\PublicInterfacePageBase;
use Waca\Validation\RequestValidationHelper;
use Waca\Validation\ValidationError;
use Waca\WebRequest;

class PageRequestAccount extends PublicInterfacePageBase
{
    const CSRF_TOKEN_CONTEXT = 'requestFormSubmission';
    /** @var RequestValidationHelper do not use directly. */
    private $validationHelper;

    /**
     * Main function for this page, when no specific actions are called.
     * @return void
     * @throws OptimisticLockFailedException
     * @throws Exception
     */
    protected function main()
    {
        $database = $this->getDatabase();

        /** @var RequestForm $defaultForm */
        $defaultForm = RequestForm::getById($this->getSiteConfiguration()->getDefaultRequestForm(), $database);

        $this->redirect('r', $defaultForm->getDomainObject()->getShortName() . '/' . $defaultForm->getPublicEndpoint());
    }

    /**
     * Handles dynamic request forms.
     * @return void
     * @throws OptimisticLockFailedException
     * @throws Exception
     */
    protected function dynamic()
    {
        $database = $this->getDatabase();

        $pathInfo = WebRequest::pathInfo();
        $domain = Domain::getByShortName($pathInfo[1], $database);
        if ($domain === false || !$domain->isEnabled()) {
            throw new ApplicationLogicException("This form is not available at this time.");
        }

        $form = RequestForm::getByPublicEndpoint($database, $pathInfo[2], $domain->getId());

        if ($form === false || !$form->isEnabled()) {
            throw new ApplicationLogicException("This form is not available at this time.");
        }

        // dual mode page
        if (WebRequest::wasPosted()) {
            // validate CSRF token
            $csrfTokenMinimumLifetime = $this->getSiteConfiguration()->getRequestMinimumTokenAge();
            try {
                $this->validateCSRFToken(self::CSRF_TOKEN_CONTEXT, $csrfTokenMinimumLifetime);
            } catch (ApplicationLogicException $e) {
                SessionAlert::error('Please try submitting the form again.', 'Oops! Something went wrong');
                $this->preserveSessionFormData();
                $this->redirect();
                return;
            }

            $request = $this->createNewRequest($form);
            $this->handleFormPost($request);
        }
        else {
            $this->assignCSRFToken(self::CSRF_TOKEN_CONTEXT);
            $this->requestClientHints();
            $this->handleFormRefilling();

            $renderer = new MarkdownRenderingHelper();
            $this->assign('formPreamble', $renderer->doRender($form->getFormContent()));
            $this->assign('formUsernameHelp', $renderer->doRenderInline($form->getUsernameHelp()));
            $this->assign('formEmailHelp', $renderer->doRenderInline($form->getEmailHelp()));
            $this->assign('formCommentsHelp', $renderer->doRenderInline($form->getCommentHelp()));

            $this->setTemplate('request/request-form-dynamic.tpl');
        }
    }

    /**
     * @param RequestForm|null $form
     *
     * @return Request
     * @throws ApplicationLogicException
     */
    protected function createNewRequest(?RequestForm $form): Request
    {
        $database = $this->getDatabase();

        $request = new Request();

        if ($form === null) {
            $domain = 1;
        }
        else {
            $domain = $form->getDomain();
            $request->setOriginForm($form->getId());
        }

        $request->setQueue(RequestQueue::getDefaultQueue($database, $domain)->getId());
        $request->setDatabase($database);
        $request->setDomain($domain);

        $request->setName(trim(WebRequest::postString('name')));
        $request->setName(trim(/** @scrutinizer ignore-type */ WebRequest::postString('name')));
        $request->setEmail(WebRequest::postEmail('email'));

        $request->setIp(WebRequest::remoteAddress());
        $request->setForwardedIp(WebRequest::forwardedAddress());

        $request->setUserAgent(WebRequest::userAgent());

        return $request;
    }

    /**
     * @return Comment|null
     */
    private function createComment()
    {
        $commentText = WebRequest::postString('comments');
        if ($commentText === null || trim($commentText) === '') {
            return null;
        }

        $comment = new Comment();
        $comment->setDatabase($this->getDatabase());

        $comment->setVisibility('requester');
        $comment->setUser(null);
        $comment->setComment($commentText);

        return $comment;
    }

    /**
     * @param Request $request
     *
     * @return ValidationError[]
     */
    protected function validateRequest($request)
    {
        $validationHelper = $this->getRequestValidationHelper();

        // These are arrays of ValidationError.
        $nameValidation = $validationHelper->validateName($request);
        $emailValidation = $validationHelper->validateEmail($request, WebRequest::postEmail('emailconfirm'));
        $otherValidation = $validationHelper->validateOther($request);

        $validationErrors = array_merge($nameValidation, $emailValidation, $otherValidation);

        return $validationErrors;
    }

    /**
     * @param Request      $request
     *
     * @param Comment|null $comment
     *
     * @throws OptimisticLockFailedException
     * @throws Exception
     */
    protected function saveAsEmailConfirmation(Request $request, $comment)
    {
        $request->generateEmailConfirmationHash();
        $request->save();

        if ($comment !== null) {
            $comment->setRequest($request->getId());
            $comment->save();
        }

        $trustedIp = $this->getXffTrustProvider()->getTrustedClientIp(
            $request->getIp(),
            $request->getForwardedIp());

        $this->assign("ip", $trustedIp);
        $this->assign("id", $request->getId());
        $this->assign("hash", $request->getEmailConfirm());

        // Sends the confirmation email to the user.
        // FIXME: domains
        /** @var Domain $domain */
        $domain = Domain::getById(1, $this->getDatabase());

        if ($this->getRequestValidationHelper()->preEmailConfirmationValidations($request)) {
            $this->getEmailHelper()->sendMail(
                $domain->getEmailReplyAddress(),
                $request->getEmail(),
                "[ACC #{$request->getId()}] English Wikipedia Account Request",
                $this->fetchTemplate('request/confirmation-mail.tpl'));
        }

        $this->redirect('emailConfirmationRequired');
    }

    /**
     * @param Request      $request
     *
     * @param Comment|null $comment
     *
     * @throws OptimisticLockFailedException
     * @throws Exception
     */
    protected function saveWithoutEmailConfirmation(Request $request, $comment)
    {
        $request->setEmailConfirm(0); // fixme Since it can't be null
        $request->save();

        if ($comment !== null) {
            $comment->setRequest($request->getId());
            $comment->save();
        }

        $this->getNotificationHelper()->requestReceived($request);

        $this->redirect('requestSubmitted');
    }

    /**
     * @return RequestValidationHelper
     */
    protected function getRequestValidationHelper(): RequestValidationHelper
    {
        $banHelper = new BanHelper($this->getDatabase(), $this->getXffTrustProvider(), null);

        if ($this->validationHelper === null) {
            $this->validationHelper = new RequestValidationHelper(
                $banHelper,
                $this->getDatabase(),
                $this->getAntiSpoofProvider(),
                $this->getXffTrustProvider(),
                $this->getHttpHelper(),
                $this->getTorExitProvider(),
                $this->getSiteConfiguration());
        }

        return $this->validationHelper;
}

    /**
     * @param Request $request
     *
     * @return void
     * @throws OptimisticLockFailedException
     */
    protected function handleFormPost(Request $request): void
    {
        $comment = $this->createComment();

        $validationErrors = $this->validateRequest($request);

        if (count($validationErrors) > 0) {
            foreach ($validationErrors as $validationError) {
                SessionAlert::error($validationError->getErrorMessage());
            }

            // Preserve the data after an error
            $this->preserveSessionFormData();

            // Validation error, bomb out early.
            $this->redirect();

            return;
        }

        // actually save the request to the database
        if ($this->getSiteConfiguration()->getEmailConfirmationEnabled()) {
            $this->saveAsEmailConfirmation($request, $comment);
            $this->savePrivateData($request);
        }
        else {
            $this->saveWithoutEmailConfirmation($request, $comment);
            $this->savePrivateData($request);
        }

        $this->getRequestValidationHelper()->postSaveValidations($request);
    }

    /**
     * @return void
     */
    protected function handleFormRefilling(): void
    {
        // set the form values from the session context
        $context = WebRequest::getSessionContext('accountReq');
        if ($context !== null && is_array($context)) {
            $this->assign('username', $context['username']);
            $this->assign('email', $context['email']);
            $this->assign('comments', $context['comments']);
        }

        // Clear it for a refresh
        WebRequest::setSessionContext('accountReq', null);
    }

    private function requestClientHints()
    {
        $hints = $this->getSiteConfiguration()->getAcceptClientHints();

        $this->headerQueue[] = "Accept-CH: " . implode(', ', $hints);
    }

    private function savePrivateData(Request $request)
    {
        RequestData::saveForRequest(
            $request, RequestData::TYPE_USERAGENT, WebRequest::userAgent());
            $request, RequestData::TYPE_USERAGENT, /** @scrutinizer ignore-type */ WebRequest::userAgent());

        foreach ($this->getSiteConfiguration()->getAcceptClientHints() as $header)
        {
            $value = WebRequest::httpHeader($header);

            if ($value === null) {
                continue;
            }

            RequestData::saveForRequest($request,
                RequestData::TYPE_CLIENTHINT, $value, $header);
        }
    }

    /**
     * @return void
     */
    protected function preserveSessionFormData(): void
    {
        WebRequest::setSessionContext(
            'accountReq',
            array(
                'username' => WebRequest::postString('name'),
                'email'    => WebRequest::postEmail('email'),
                'comments' => WebRequest::postString('comments'),
            )
        );
    }
}
enwikipedia-acc / waca

Pull Request — master (#1066)

PageRequestAccount::saveWithoutEmailConfirmation() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
