WebTheory\Saveyour\Auth\Policy\ReCaptchaPolicy

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	approvesRequest()	0	3	1
A	getGeneratedResponse()	0	3	1
A	getApiResponse()	0	3	1
A	__construct()	0	4	1

<?php

namespace WebTheory\Saveyour\Auth\Policy;

use Psr\Http\Message\ServerRequestInterface;
use ReCaptcha\ReCaptcha;
use ReCaptcha\Response;
use WebTheory\HttpPolicy\ServerRequestPolicyInterface;
use WebTheory\Saveyour\Http\Request;

class ReCaptchaPolicy implements ServerRequestPolicyInterface
{
    protected string $response;

    protected ReCaptcha $reCaptcha;

    public function __construct(string $response, ReCaptcha $reCaptcha)
    {
        $this->response = $response;
        $this->reCaptcha = $reCaptcha;
    }

    public function approvesRequest(ServerRequestInterface $request): bool
    {
        return $this->getApiResponse($request)->isSuccess();
    }

    protected function getApiResponse(ServerRequestInterface $request): Response
    {
        return $this->reCaptcha->verify($this->getGeneratedResponse($request));
    }

    protected function getGeneratedResponse(ServerRequestInterface $request): string
    {
        return Request::var($request, $this->response);

The expression return WebTheory\Saveyour\Http\Request::var($request, $this->response) could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}