Gewaer\Middleware\AuthenticationMiddleware

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	call()	0	28	4

<?php

End of line character is invalid; expected "\n" but found "\r\n"

declare(strict_types=1);


namespace Gewaer\Middleware;


use Phalcon\Mvc\Micro;

use Phalcon\Mvc\Micro\MiddlewareInterface;

use Baka\Auth\Models\Sessions;

use Gewaer\Models\Users;

use Phalcon\Http\Request;

use Gewaer\Exception\UnauthorizedHttpException;


/**

 * Class AuthenticationMiddleware

 *

 * @package Niden\Middleware

 */

class AuthenticationMiddleware implements MiddlewareInterface

{

    /**

     * Call me

     *

     * @param Micro $api

     * @todo need to check section for auth here

     * @return bool

     */

    public function call(Micro $api)

    {

        $auth = $api->getService('auth');

        $config = $api->getService('config');


        // to get the payload

        $data = $auth->data();


        $api->getDI()->setShared(

            'userData',

            function () use ($config, $data) {

The import $config is not used and could be removed.

                $session = new Sessions();

                $request = new Request();


                if (!empty($data) && !empty($data['sessionId'])) {

                    //user

                    if (!$user = Users::getByEmail($data['email'])) {

                        throw new UnauthorizedHttpException('User not found');

                    }


                    return $session->check($user, $data['sessionId'], $request->getClientAddress(), 1);

It seems like $request->getClientAddress() can also be of type boolean; however, parameter $userIp of Baka\Auth\Models\Sessions::check() does only seem to accept string, maybe add an additional type check?

                } else {

                    throw new UnauthorizedHttpException('User not found');

                }

            }

        );


        return true;

    }

}