Authorize::__construct() - Code Metrics - Inspection of "feat(reauthenticate): Add flag to allow re-authent..." - simplesamlphp/simplesamlphp-module-authorize - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#18)

unknown

created 2024-08-05 22:09 UTC

Authorize::__construct() D

↳ Parent: Authorize

Complexity

Conditions	16
Paths	256

Size

Total Lines	58
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	32
c	1
b	0
f	0
dl	0
loc	58
rs	4.0333
cc	16
nc	256
nop	2

How to fix Long Method Complexity

<?php

declare(strict_types=1);

namespace SimpleSAML\Module\authorize\Auth\Process;

use Exception;
use SimpleSAML\Assert\Assert;
use SimpleSAML\Auth;
use SimpleSAML\Module;
use SimpleSAML\Utils;

use function array_diff;
use function array_key_exists;
use function array_keys;
use function array_push;
use function implode;
use function is_array;
use function is_bool;
use function is_string;
use function preg_match;
use function var_export;

/**
 * Filter to authorize only certain users.
 * See docs directory.
 *
 * @package SimpleSAMLphp
 */

class Authorize extends Auth\ProcessingFilter
{
    /**
     * Flag to deny/unauthorize the user a attribute filter IS found
     *
     * @var bool
     */
    protected bool $deny = false;

    /**
     * Flag to turn the REGEX pattern matching on or off
     *
     * @var bool
     */
    protected bool $regex = true;

    /**
     * Array of localised rejection messages
     *
     * @var array
     */
    protected array $reject_msg = [];

    /**
     * Flag to toggle generation of errorURL
     *
     * @var bool
     */
    protected bool $errorURL = true;

    /**
     * Array of valid users. Each element is a regular expression. You should
     * user \ to escape special chars, like '.' etc.
     *
     * @param array
     */
    protected array $valid_attribute_values = [];

    /**
     * Flag to allow re-authentication when user is not authorized
     * @var bool
     */
    protected bool $allow_reauthentication = false;

    /**
     * Initialize this filter.
     * Validate configuration parameters.
     *
     * @param array $config  Configuration information about this filter.
     * @param mixed $reserved  For future use.
     */
    public function __construct(array $config, $reserved)
    {
        parent::__construct($config, $reserved);

        // Check for the deny option
        // Must be bool specifically, if not, it might be for an attrib filter below
        if (isset($config['deny']) && is_bool($config['deny'])) {
            $this->deny = $config['deny'];
            unset($config['deny']);
        }

        // Check for the regex option
        // Must be bool specifically, if not, it might be for an attrib filter below
        if (isset($config['regex']) && is_bool($config['regex'])) {
            $this->regex = $config['regex'];
            unset($config['regex']);
        }

        // Check for the reject_msg option; Must be array of languages
        if (isset($config['reject_msg']) && is_array($config['reject_msg'])) {
            $this->reject_msg = $config['reject_msg'];
            unset($config['reject_msg']);
        }

        // Check for the errorURL option
        // Must be bool specifically, if not, it might be for an attrib filter below
        if (isset($config['errorURL']) && is_bool($config['errorURL'])) {
            $this->errorURL = $config['errorURL'];
            unset($config['errorURL']);
        }

        if (isset($config['allow_reauthentication']) && is_bool($config['allow_reauthentication'])) {
            $this->allow_reauthentication = $config['allow_reauthentication'];
            unset($config['allow_reauthentication']);
        }

        foreach ($config as $attribute => $values) {
            if (is_string($values)) {
                $arrayUtils = new Utils\Arrays();
                $values = $arrayUtils->arrayize($values);
            } elseif (!is_array($values)) {
                throw new Exception(sprintf(
                    'Filter Authorize: Attribute values is neither string nor array: %s',
                    var_export($attribute, true),
                ));
            }

            foreach ($values as $value) {
                if (!is_string($value)) {
                    throw new Exception(sprintf(
                        'Filter Authorize: Each value should be a string for attribute: %s value: %s config: %s',
                        var_export($attribute, true),
                        var_export($value, true),
                        var_export($config, true),
                    ));
                }
            }
            $this->valid_attribute_values[$attribute] = $values;
        }
    }


    /**
     * Apply filter to validate attributes.
     *
     * @param array &$state  The current request
     */
    public function process(array &$state): void
    {
        Assert::keyExists($state, 'Attributes');

        $authorize = $this->deny;
        $attributes = &$state['Attributes'];
        $ctx = [];

        // Store the rejection message array in the $state
        if (!empty($this->reject_msg)) {
            $state['authprocAuthorize_reject_msg'] = $this->reject_msg;
        }
        $state['authprocAuthorize_errorURL'] = $this->errorURL;
        $state['authprocAuthorize_allow_reauthentication'] = $this->allow_reauthentication;
        $arrayUtils = new Utils\Arrays();
        foreach ($this->valid_attribute_values as $name => $patterns) {
            if (array_key_exists($name, $attributes)) {
                foreach ($patterns as $pattern) {
                    $values = $arrayUtils->arrayize($attributes[$name]);
                    foreach ($values as $value) {
                        if ($this->regex) {
                            $matched = preg_match($pattern, $value);
                        } else {
                            $matched = ($value === $pattern);
                        }

                        if ($matched) {
                            $authorize = ($this->deny ? false : true);
                            array_push($ctx, $name);
                            break 3;
                        }
                    }
                }
            }
        }

        if (!$authorize) {
            // Try to hint at which attributes may have failed as context for errorURL processing
            if ($this->deny) {
                $state['authprocAuthorize_ctx'] = implode(' ', $ctx);
            } else {
                $state['authprocAuthorize_ctx'] = implode(
                    ' ',
                    array_diff(array_keys($this->valid_attribute_values), $ctx),
                );
            }
            $this->unauthorized($state);
        }
    }


    /**
     * When the process logic determines that the user is not
     * authorized for this service, then forward the user to
     * an 403 unauthorized page.
     *
     * Separated this code into its own method so that child
     * classes can override it and change the action. Forward
     * thinking in case a "chained" ACL is needed, more complex
     * permission logic.
     *
     * @param array $state
     */
    protected function unauthorized(array &$state): void
    {
        // Save state and redirect to 403 page
        $id = Auth\State::saveState($state, 'authorize:Authorize');
        $url = Module::getModuleURL('authorize/error/forbidden');
        $httpUtils = new Utils\HTTP();
        $httpUtils->redirectTrustedURL($url, ['StateId' => $id]);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace SimpleSAML\Module\authorize\Auth\Process;
6
7			use Exception;
8			use SimpleSAML\Assert\Assert;
9			use SimpleSAML\Auth;
10			use SimpleSAML\Module;
11			use SimpleSAML\Utils;
12
13			use function array_diff;
14			use function array_key_exists;
15			use function array_keys;
16			use function array_push;
17			use function implode;
18			use function is_array;
19			use function is_bool;
20			use function is_string;
21			use function preg_match;
22			use function var_export;
23
24			/**
25			* Filter to authorize only certain users.
26			* See docs directory.
27			*
28			* @package SimpleSAMLphp
29			*/
30
31			class Authorize extends Auth\ProcessingFilter
32			{
33			/**
34			* Flag to deny/unauthorize the user a attribute filter IS found
35			*
36			* @var bool
37			*/
38			protected bool $deny = false;
39
40			/**
41			* Flag to turn the REGEX pattern matching on or off
42			*
43			* @var bool
44			*/
45			protected bool $regex = true;
46
47			/**
48			* Array of localised rejection messages
49			*
50			* @var array
51			*/
52			protected array $reject_msg = [];
53
54			/**
55			* Flag to toggle generation of errorURL
56			*
57			* @var bool
58			*/
59			protected bool $errorURL = true;
60
61			/**
62			* Array of valid users. Each element is a regular expression. You should
63			* user \ to escape special chars, like '.' etc.
64			*
65			* @param array
66			*/
67			protected array $valid_attribute_values = [];
68
69			/**
70			* Flag to allow re-authentication when user is not authorized
71			* @var bool
72			*/
73			protected bool $allow_reauthentication = false;
74
75			/**
76			* Initialize this filter.
77			* Validate configuration parameters.
78			*
79			* @param array $config Configuration information about this filter.
80			* @param mixed $reserved For future use.
81			*/
82			public function __construct(array $config, $reserved)
83			{
84			parent::__construct($config, $reserved);
85
86			// Check for the deny option
87			// Must be bool specifically, if not, it might be for an attrib filter below
88			if (isset($config['deny']) && is_bool($config['deny'])) {
89			$this->deny = $config['deny'];
90			unset($config['deny']);
91			}
92
93			// Check for the regex option
94			// Must be bool specifically, if not, it might be for an attrib filter below
95			if (isset($config['regex']) && is_bool($config['regex'])) {
96			$this->regex = $config['regex'];
97			unset($config['regex']);
98			}
99
100			// Check for the reject_msg option; Must be array of languages
101			if (isset($config['reject_msg']) && is_array($config['reject_msg'])) {
102			$this->reject_msg = $config['reject_msg'];
103			unset($config['reject_msg']);
104			}
105
106			// Check for the errorURL option
107			// Must be bool specifically, if not, it might be for an attrib filter below
108			if (isset($config['errorURL']) && is_bool($config['errorURL'])) {
109			$this->errorURL = $config['errorURL'];
110			unset($config['errorURL']);
111			}
112
113			if (isset($config['allow_reauthentication']) && is_bool($config['allow_reauthentication'])) {
114			$this->allow_reauthentication = $config['allow_reauthentication'];
115			unset($config['allow_reauthentication']);
116			}
117
118			foreach ($config as $attribute => $values) {
119			if (is_string($values)) {
120			$arrayUtils = new Utils\Arrays();
121			$values = $arrayUtils->arrayize($values);
122			} elseif (!is_array($values)) {
123			throw new Exception(sprintf(
124			'Filter Authorize: Attribute values is neither string nor array: %s',
125			var_export($attribute, true),
126			));
127			}
128
129			foreach ($values as $value) {
130			if (!is_string($value)) {
131			throw new Exception(sprintf(
132			'Filter Authorize: Each value should be a string for attribute: %s value: %s config: %s',
133			var_export($attribute, true),
134			var_export($value, true),
135			var_export($config, true),
136			));
137			}
138			}
139			$this->valid_attribute_values[$attribute] = $values;
140			}
141			}
142
143
144			/**
145			* Apply filter to validate attributes.
146			*
147			* @param array &$state The current request
148			*/
149			public function process(array &$state): void
150			{
151			Assert::keyExists($state, 'Attributes');
152
153			$authorize = $this->deny;
154			$attributes = &$state['Attributes'];
155			$ctx = [];
156
157			// Store the rejection message array in the $state
158			if (!empty($this->reject_msg)) {
159			$state['authprocAuthorize_reject_msg'] = $this->reject_msg;
160			}
161			$state['authprocAuthorize_errorURL'] = $this->errorURL;
162			$state['authprocAuthorize_allow_reauthentication'] = $this->allow_reauthentication;
163			$arrayUtils = new Utils\Arrays();
164			foreach ($this->valid_attribute_values as $name => $patterns) {
165			if (array_key_exists($name, $attributes)) {
166			foreach ($patterns as $pattern) {
167			$values = $arrayUtils->arrayize($attributes[$name]);
168			foreach ($values as $value) {
169			if ($this->regex) {
170			$matched = preg_match($pattern, $value);
171			} else {
172			$matched = ($value === $pattern);
173			}
174
175			if ($matched) {
176			$authorize = ($this->deny ? false : true);
177			array_push($ctx, $name);
178			break 3;
179			}
180			}
181			}
182			}
183			}
184
185			if (!$authorize) {
186			// Try to hint at which attributes may have failed as context for errorURL processing
187			if ($this->deny) {
188			$state['authprocAuthorize_ctx'] = implode(' ', $ctx);
189			} else {
190			$state['authprocAuthorize_ctx'] = implode(
191			' ',
192			array_diff(array_keys($this->valid_attribute_values), $ctx),
193			);
194			}
195			$this->unauthorized($state);
196			}
197			}
198
199
200			/**
201			* When the process logic determines that the user is not
202			* authorized for this service, then forward the user to
203			* an 403 unauthorized page.
204			*
205			* Separated this code into its own method so that child
206			* classes can override it and change the action. Forward
207			* thinking in case a "chained" ACL is needed, more complex
208			* permission logic.
209			*
210			* @param array $state
211			*/
212			protected function unauthorized(array &$state): void
213			{
214			// Save state and redirect to 403 page
215			$id = Auth\State::saveState($state, 'authorize:Authorize');
216			$url = Module::getModuleURL('authorize/error/forbidden');
217			$httpUtils = new Utils\HTTP();
218			$httpUtils->redirectTrustedURL($url, ['StateId' => $id]);
219			}
220			}
221

simplesamlphp / simplesamlphp-module-authorize

Pull Request — master (#18)

Authorize::__construct() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like