Graviton\SecurityBundle\Authentication\Strategies\SameSubnetStrategy

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

25%

Importance

Changes	4
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	apply()	0	8	2
A	stopPropagation()	0	4	1
A	getRoles()	0	4	1
A	determineName()	0	8	2

<?php
/**
 * strategy for validating auth through the ip address.
 */

namespace Graviton\SecurityBundle\Authentication\Strategies;

use Graviton\SecurityBundle\Entities\SecurityUser;
use Symfony\Component\HttpFoundation\IpUtils;
use Symfony\Component\HttpFoundation\Request;

/**
 * Class SameSubnetStrategy
 *
 * @author   List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
 * @link     http://swisscom.ch
 */
class SameSubnetStrategy extends AbstractHttpStrategy
{
    /** @var String */
    protected $subnet;

    /**
     * @param String $subnet      Subnet to be checked (e.g. 10.2.0.0/24)
     * @param String $headerField Http header field to be searched for the 'username'
     */
    public function __construct($subnet, $headerField = 'x-graviton-authentication')
    {
        $this->subnet= $subnet;
        $this->headerField = $headerField;
    }

    /**
     * Applies the defined strategy on the provided request.
     *
     * @param Request $request request to handle
     *
     * @return string

Should the return type not be string|array? Also, consider making the array more specific, something like array<String>, or String[].

     */
    public function apply(Request $request)
    {
        if (IpUtils::checkIp($request->getClientIp(), $this->subnet)) {
            return $this->determineName($request);

The expression $this->determineName($request); of type string|array adds the type array to the return on line 44 which is incompatible with the return type declared by the interface Graviton\SecurityBundle\Authentication\Strategies\StrategyInterface::apply of type string.

        }

        throw new \InvalidArgumentException('Provided request information are not valid.');
    }

    /**
     * Decider to stop other strategies running after from being considered.
     *
     * @return boolean
     */
    public function stopPropagation()
    {
        return false;
    }

    /**
     * Provides the list of registered roles.
     *
     * @return Role[]
     */
    public function getRoles()
    {
        return [SecurityUser::ROLE_SUBNET];
    }

    /**
     * Finds the username either from a http header filed or returns a default.
     *
     * @param Request $request Current http request
     *
     * @return string

Should the return type not be string|array? Also, consider making the array more specific, something like array<String>, or String[].

     */
    private function determineName(Request $request)
    {
        if ($request->headers->has($this->headerField)) {
            return $request->headers->get($this->headerField);
        }

        return 'graviton_subnet_user';
    }
}