MagicLink\Middlewares\AskForAccessCode::handle()

Complexity

Conditions	6
Paths	7

Size

Total Lines	32
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function handle(Request $request, Closure $next)
    {
        $magicLink = MagicLink::getValidMagicLinkByToken($request->route('token'));

It seems like $request->route('token') can also be of type Illuminate\Routing\Route and object; however, parameter $token of MagicLink\MagicLink::getValidMagicLinkByToken() does only seem to accept string, maybe add an additional type check?

        if (! $magicLink || is_null($magicLink->access_code ?? null)) {
            return $next($request);
        }

        if ($this->isAccessCodeValid($request->route('token'), $request->get('access-code'))) {

It seems like $request->route('token') can also be of type Illuminate\Routing\Route and null and object; however, parameter $token of MagicLink\Middlewares\AskForAccessCode::isAccessCodeValid() does only seem to accept string, maybe add an additional type check?

            // access code is valid
            return redirect($request->url())->withCookie(
                cookie(
                    'magic-link-access-code',
                    encrypt($request->get('access-code')),
                    0,
                    '/'
                )
            );
        }

        try {
            $accessCode = decrypt($request->cookie('magic-link-access-code'));

It seems like $request->cookie('magic-link-access-code') can also be of type array; however, parameter $value of decrypt() does only seem to accept string, maybe add an additional type check?

            // Validate access_code
            if ($this->isAccessCodeValid($request->route('token'), $accessCode)) {
                return $next($request);
            }
        } catch (DecryptException $e) {
            // empty value in cookie
        }

        return response(view('magiclink::ask-for-access-code-form'), 403);
    }