SecurityManager::findResult() - Code Metrics - Inspection of "Fix two XSS vulns" - enwikipedia-acc/waca - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — newinternal ( 65a0f5...5b021c )

by Simon

created 2020-04-26 13:09 UTC

SecurityManager::findResult() B

↳ Parent: SecurityManager

Complexity

Conditions	9
Paths	18

Size

Total Lines

Duplication

Lines	10
Ratio	31.25 %

Code Coverage

Tests	0
CRAP Score	90

Importance

Changes

Metric	Value
cc	9
nc	18
nop	3
dl	10
loc	32
ccs	0
cts	8
cp	0
crap	90
rs	8.0555
c	0
b	0
f	0

<?php
/******************************************************************************
 * Wikipedia Account Creation Assistance tool                                 *
 *                                                                            *
 * All code in this file is released into the public domain by the ACC        *
 * Development Team. Please see team.json for a list of contributors.         *
 ******************************************************************************/

namespace Waca\Security;

use Waca\DataObjects\User;
use Waca\DataObjects\UserRole;
use Waca\IdentificationVerifier;

final class SecurityManager
{
    const ALLOWED = 1;
    const ERROR_NOT_IDENTIFIED = 2;
    const ERROR_DENIED = 3;
    /** @var IdentificationVerifier */
    private $identificationVerifier;
    /**
     * @var RoleConfiguration
     */
    private $roleConfiguration;

    /**
     * SecurityManager constructor.
     *
     * @param IdentificationVerifier $identificationVerifier
     * @param RoleConfiguration      $roleConfiguration
     */
    public function __construct(
        IdentificationVerifier $identificationVerifier,
        RoleConfiguration $roleConfiguration
    ) {
        $this->identificationVerifier = $identificationVerifier;
        $this->roleConfiguration = $roleConfiguration;
    }

    /**
     * Tests if a user is allowed to perform an action.
     *
     * This method should form a hard, deterministic security barrier, and only return true if it is absolutely sure
     * that a user should have access to something.
     *
     * @param string $page
     * @param string $route
     * @param User   $user
     *
     * @return int
     *
     * @category Security-Critical
     */
    public function allows($page, $route, User $user)
    {
        $this->getActiveRoles($user, $activeRoles, $inactiveRoles);

        $availableRights = $this->flattenRoles($activeRoles);
        $testResult = $this->findResult($availableRights, $page, $route);

        if ($testResult !== null) {
            // We got a firm result here, so just return it.
            return $testResult;
        }

        // No firm result yet, so continue testing the inactive roles so we can give a better error.
        $inactiveRights = $this->flattenRoles($inactiveRoles);
        $testResult = $this->findResult($inactiveRights, $page, $route);

        if ($testResult === self::ALLOWED) {
            // The user is allowed to access this, but their role is inactive.
            return self::ERROR_NOT_IDENTIFIED;
        }

        // Other options from the secondary test are denied and inconclusive, which at this point defaults to denied.
        return self::ERROR_DENIED;
    }

    /**
     * @param array  $pseudoRole The role (flattened) to check
     * @param string $page       The page class to check
     * @param string $route      The page route to check
     *
     * @return int|null
     */
    private function findResult($pseudoRole, $page, $route)
    {
        if (isset($pseudoRole[$page])) {
            // check for deny on catch-all route
            if (isset($pseudoRole[$page][RoleConfiguration::ALL])) {

                if ($pseudoRole[$page][RoleConfiguration::ALL] === RoleConfiguration::ACCESS_DENY) {
                    return self::ERROR_DENIED;
                }
            }

            // check normal route
            if (isset($pseudoRole[$page][$route])) {
                if ($pseudoRole[$page][$route] === RoleConfiguration::ACCESS_DENY) {
                    return self::ERROR_DENIED;
                }

                if ($pseudoRole[$page][$route] === RoleConfiguration::ACCESS_ALLOW) {
                    return self::ALLOWED;
                }
            }

            // check for allowed on catch-all route
            if (isset($pseudoRole[$page][RoleConfiguration::ALL])) {

                if ($pseudoRole[$page][RoleConfiguration::ALL] === RoleConfiguration::ACCESS_ALLOW) {
                    return self::ALLOWED;
                }
            }
        }

        // return indeterminate result
        return null;
    }

    /**
     * Takes an array of roles and flattens the values to a single set.
     *
     * @param array $activeRoles
     *
     * @return array
     */
    private function flattenRoles($activeRoles)
    {
        $result = array();

        $roleConfig = $this->roleConfiguration->getApplicableRoles($activeRoles);

        // Iterate over every page in every role
        foreach ($roleConfig as $role) {
            foreach ($role as $page => $pageRights) {
                // Create holder in result for this page
                if (!isset($result[$page])) {
                    $result[$page] = array();
                }

                foreach ($pageRights as $action => $permission) {
                    // Deny takes precedence, so if it's set, don't change it.
                    if (isset($result[$page][$action])) {
                        if ($result[$page][$action] === RoleConfiguration::ACCESS_DENY) {
                            continue;
                        }
                    }

                    if ($permission === RoleConfiguration::ACCESS_DEFAULT) {
                        // Configured to do precisely nothing.
                        continue;
                    }

                    $result[$page][$action] = $permission;
                }
            }
        }

        return $result;
    }

    /**
     * @param User  $user
     * @param array $activeRoles
     * @param array $inactiveRoles
     */
    public function getActiveRoles(User $user, &$activeRoles, &$inactiveRoles)
    {
        // Default to the community user here, because the main user is logged out
        $identified = false;
        $userRoles = array('public');

        // if we're not the community user, get our real rights.
        if (!$user->isCommunityUser()) {
            // Check the user's status - only active users are allowed the effects of roles

            $userRoles[] = 'loggedIn';

            if ($user->isActive()) {
                $ur = UserRole::getForUser($user->getId(), $user->getDatabase());

                // NOTE: public is still in this array.
                foreach ($ur as $r) {
                    $userRoles[] = $r->getRole();
                }

                $identified = $user->isIdentified($this->identificationVerifier);
            }
        }

        $activeRoles = array();
        $inactiveRoles = array();

        /** @var string $v */
        foreach ($userRoles as $v) {
            if ($this->roleConfiguration->roleNeedsIdentification($v)) {
                if ($identified) {
                    $activeRoles[] = $v;
                }
                else {
                    $inactiveRoles[] = $v;
                }
            }
            else {
                $activeRoles[] = $v;
            }
        }
    }

    public function getRoleConfiguration(){
        return $this->roleConfiguration;
    }
}


1		<?php
2		/******************************************************************************
3		* Wikipedia Account Creation Assistance tool *
4		* *
5		* All code in this file is released into the public domain by the ACC *
6		* Development Team. Please see team.json for a list of contributors. *
7		******************************************************************************/
8
9		namespace Waca\Security;
10
11		use Waca\DataObjects\User;
12		use Waca\DataObjects\UserRole;
13		use Waca\IdentificationVerifier;
14
15		final class SecurityManager
16		{
17		const ALLOWED = 1;
18		const ERROR_NOT_IDENTIFIED = 2;
19		const ERROR_DENIED = 3;
20		/** @var IdentificationVerifier */
21		private $identificationVerifier;
22		/**
23		* @var RoleConfiguration
24		*/
25		private $roleConfiguration;
26
27		/**
28		* SecurityManager constructor.
29		*
30		* @param IdentificationVerifier $identificationVerifier
31		* @param RoleConfiguration $roleConfiguration
32		*/
33		public function __construct(
34		IdentificationVerifier $identificationVerifier,
35		RoleConfiguration $roleConfiguration
36		) {
37		$this->identificationVerifier = $identificationVerifier;
38		$this->roleConfiguration = $roleConfiguration;
39		}
40
41		/**
42		* Tests if a user is allowed to perform an action.
43		*
44		* This method should form a hard, deterministic security barrier, and only return true if it is absolutely sure
45		* that a user should have access to something.
46		*
47		* @param string $page
48		* @param string $route
49		* @param User $user
50		*
51		* @return int
52		*
53		* @category Security-Critical
54		*/
55		public function allows($page, $route, User $user)
56		{
57		$this->getActiveRoles($user, $activeRoles, $inactiveRoles);
58
59		$availableRights = $this->flattenRoles($activeRoles);
60		$testResult = $this->findResult($availableRights, $page, $route);
61
62		if ($testResult !== null) {
63		// We got a firm result here, so just return it.
64		return $testResult;
65		}
66
67		// No firm result yet, so continue testing the inactive roles so we can give a better error.
68		$inactiveRights = $this->flattenRoles($inactiveRoles);
69		$testResult = $this->findResult($inactiveRights, $page, $route);
70
71		if ($testResult === self::ALLOWED) {
72		// The user is allowed to access this, but their role is inactive.
73		return self::ERROR_NOT_IDENTIFIED;
74		}
75
76		// Other options from the secondary test are denied and inconclusive, which at this point defaults to denied.
77		return self::ERROR_DENIED;
78		}
79
80		/**
81		* @param array $pseudoRole The role (flattened) to check
82		* @param string $page The page class to check
83		* @param string $route The page route to check
84		*
85		* @return int\|null
86		*/
87		private function findResult($pseudoRole, $page, $route)
88		{
89		if (isset($pseudoRole[$page])) {
90		// check for deny on catch-all route
91	View Code Duplication	if (isset($pseudoRole[$page][RoleConfiguration::ALL])) {
		0 ignored issues – show Duplication introduced 2017-01-29 14:52 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
92		if ($pseudoRole[$page][RoleConfiguration::ALL] === RoleConfiguration::ACCESS_DENY) {
93		return self::ERROR_DENIED;
94		}
95		}
96
97		// check normal route
98		if (isset($pseudoRole[$page][$route])) {
99		if ($pseudoRole[$page][$route] === RoleConfiguration::ACCESS_DENY) {
100		return self::ERROR_DENIED;
101		}
102
103		if ($pseudoRole[$page][$route] === RoleConfiguration::ACCESS_ALLOW) {
104		return self::ALLOWED;
105		}
106		}
107
108		// check for allowed on catch-all route
109	View Code Duplication	if (isset($pseudoRole[$page][RoleConfiguration::ALL])) {
		0 ignored issues – show Duplication introduced 2017-01-29 14:52 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
110		if ($pseudoRole[$page][RoleConfiguration::ALL] === RoleConfiguration::ACCESS_ALLOW) {
111		return self::ALLOWED;
112		}
113		}
114		}
115
116		// return indeterminate result
117		return null;
118		}
119
120		/**
121		* Takes an array of roles and flattens the values to a single set.
122		*
123		* @param array $activeRoles
124		*
125		* @return array
126		*/
127		private function flattenRoles($activeRoles)
128		{
129		$result = array();
130
131		$roleConfig = $this->roleConfiguration->getApplicableRoles($activeRoles);
132
133		// Iterate over every page in every role
134		foreach ($roleConfig as $role) {
135		foreach ($role as $page => $pageRights) {
136		// Create holder in result for this page
137		if (!isset($result[$page])) {
138		$result[$page] = array();
139		}
140
141		foreach ($pageRights as $action => $permission) {
142		// Deny takes precedence, so if it's set, don't change it.
143		if (isset($result[$page][$action])) {
144		if ($result[$page][$action] === RoleConfiguration::ACCESS_DENY) {
145		continue;
146		}
147		}
148
149		if ($permission === RoleConfiguration::ACCESS_DEFAULT) {
150		// Configured to do precisely nothing.
151		continue;
152		}
153
154		$result[$page][$action] = $permission;
155		}
156		}
157		}
158
159		return $result;
160		}
161
162		/**
163		* @param User $user
164		* @param array $activeRoles
165		* @param array $inactiveRoles
166		*/
167		public function getActiveRoles(User $user, &$activeRoles, &$inactiveRoles)
168		{
169		// Default to the community user here, because the main user is logged out
170		$identified = false;
171		$userRoles = array('public');
172
173		// if we're not the community user, get our real rights.
174		if (!$user->isCommunityUser()) {
175		// Check the user's status - only active users are allowed the effects of roles
176
177		$userRoles[] = 'loggedIn';
178
179		if ($user->isActive()) {
180		$ur = UserRole::getForUser($user->getId(), $user->getDatabase());
181
182		// NOTE: public is still in this array.
183		foreach ($ur as $r) {
184		$userRoles[] = $r->getRole();
185		}
186
187		$identified = $user->isIdentified($this->identificationVerifier);
188		}
189		}
190
191		$activeRoles = array();
192		$inactiveRoles = array();
193
194		/** @var string $v */
195		foreach ($userRoles as $v) {
196		if ($this->roleConfiguration->roleNeedsIdentification($v)) {
197		if ($identified) {
198		$activeRoles[] = $v;
199		}
200		else {
201		$inactiveRoles[] = $v;
202		}
203		}
204		else {
205		$activeRoles[] = $v;
206		}
207		}
208		}
209
210		public function getRoleConfiguration(){
211		return $this->roleConfiguration;
212		}
213		}
214

enwikipedia-acc / waca

Push — newinternal ( 65a0f5...5b021c )

SecurityManager::findResult() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like