CorsMiddleware - Code Metrics - ThrusterIO/http-middlewares - Measure and Improve Code Quality continuously with Scrutinizer

CorsMiddleware A
last analyzed 2016-08-30 10:44 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

Importance

Changes

Metric	Value
wmc	13
lcom	0
cbo	5
dl	0
loc	70
ccs	0
cts	40
cp	0
rs	10
c	0
b	0
f	0

4 Methods

Rating	Name	Size	Complexity
A	__construct()	10	2
D	__invoke()	26	9
A	getSettings()	4	1
A	setAnalyzer()	6	1

<?php

namespace Thruster\Component\HttpMiddlewares;

use Neomerx\Cors\Analyzer;
use Neomerx\Cors\Contracts\AnalysisResultInterface;
use Neomerx\Cors\Contracts\AnalyzerInterface;
use Neomerx\Cors\Contracts\Strategies\SettingsStrategyInterface;
use Neomerx\Cors\Strategies\Settings;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Log\LoggerInterface;

/**
 * Class CorsMiddleware
 *
 * @package Thruster\Component\HttpMiddlewares
 * @author  Aurimas Niekis <[email protected]>
 */
class CorsMiddleware
{
    /**
     * @var AnalyzerInterface
     */
    private $analyzer;

    /**
     * @var SettingsStrategyInterface
     */
    private $settings;

    public function __construct(SettingsStrategyInterface $settings = null, LoggerInterface $logger = null)
    {
        $this->settings = $settings ?? new Settings();

        $this->analyzer = Analyzer::instance($this->settings);

        if (null !== $logger) {
            $this->analyzer->setLogger($logger);
        }
    }

    public function __invoke(ServerRequestInterface $request, ResponseInterface $response, callable $next)
    {
        $cors = $this->analyzer->analyze($request);

        switch ($cors->getRequestType()) {
            case AnalysisResultInterface::ERR_NO_HOST_HEADER:
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
            case AnalysisResultInterface::ERR_ORIGIN_NOT_ALLOWED:
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B"; //wrong
        doSomething();
        break;
    case "C": //right
        doSomething();
        break;
}
            case AnalysisResultInterface::ERR_METHOD_NOT_SUPPORTED:
            case AnalysisResultInterface::ERR_HEADERS_NOT_SUPPORTED:
                return $response->withStatus(403);
            case AnalysisResultInterface::TYPE_REQUEST_OUT_OF_CORS_SCOPE:
                return $next($request, $response);
            case AnalysisResultInterface::TYPE_PRE_FLIGHT_REQUEST:
                foreach ($cors->getResponseHeaders() as $name => $value) {
                    $response = $response->withHeader($name, $value);
                }

                return $response->withStatus(200);
            default:
                foreach ($cors->getResponseHeaders() as $name => $value) {
                    $response = $response->withHeader($name, $value);
                }

                return $next($request, $response);
        }
    }

    /**
     * @return SettingsStrategyInterface
     */
    public function getSettings()
    {
        return $this->settings;
    }

    /**
     * @param AnalyzerInterface $analyzer
     *
     * @return $this
     */
    public function setAnalyzer($analyzer)
    {
        $this->analyzer = $analyzer;

        return $this;
    }
}


1			<?php
2
3			namespace Thruster\Component\HttpMiddlewares;
4
5			use Neomerx\Cors\Analyzer;
6			use Neomerx\Cors\Contracts\AnalysisResultInterface;
7			use Neomerx\Cors\Contracts\AnalyzerInterface;
8			use Neomerx\Cors\Contracts\Strategies\SettingsStrategyInterface;
9			use Neomerx\Cors\Strategies\Settings;
10			use Psr\Http\Message\ResponseInterface;
11			use Psr\Http\Message\ServerRequestInterface;
12			use Psr\Log\LoggerInterface;
13
14			/**
15			* Class CorsMiddleware
16			*
17			* @package Thruster\Component\HttpMiddlewares
18			* @author Aurimas Niekis <[email protected]>
19			*/
20			class CorsMiddleware
21			{
22			/**
23			* @var AnalyzerInterface
24			*/
25			private $analyzer;
26
27			/**
28			* @var SettingsStrategyInterface
29			*/
30			private $settings;
31
32			public function __construct(SettingsStrategyInterface $settings = null, LoggerInterface $logger = null)
33			{
34			$this->settings = $settings ?? new Settings();
35
36			$this->analyzer = Analyzer::instance($this->settings);
37
38			if (null !== $logger) {
39			$this->analyzer->setLogger($logger);
40			}
41			}
42
43			public function __invoke(ServerRequestInterface $request, ResponseInterface $response, callable $next)
44			{
45			$cors = $this->analyzer->analyze($request);
46
47			switch ($cors->getRequestType()) {
48			case AnalysisResultInterface::ERR_NO_HOST_HEADER:
			0 ignored issues – show Coding Style introduced 2016-08-30 10:15 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
49			case AnalysisResultInterface::ERR_ORIGIN_NOT_ALLOWED:
			0 ignored issues – show Coding Style introduced 2016-08-30 10:15 UTC by Report Bug Copy Issue Report `case` statements should be defined using a colon. As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. There is also the option to use a semicolon instead of a colon, this is discouraged because many programmers do not even know it works and the colon is universal between programming languages. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B"; //wrong doSomething(); break; case "C": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
50			case AnalysisResultInterface::ERR_METHOD_NOT_SUPPORTED:
51			case AnalysisResultInterface::ERR_HEADERS_NOT_SUPPORTED:
52			return $response->withStatus(403);
53			case AnalysisResultInterface::TYPE_REQUEST_OUT_OF_CORS_SCOPE:
54			return $next($request, $response);
55			case AnalysisResultInterface::TYPE_PRE_FLIGHT_REQUEST:
56			foreach ($cors->getResponseHeaders() as $name => $value) {
57			$response = $response->withHeader($name, $value);
58			}
59
60			return $response->withStatus(200);
61			default:
62			foreach ($cors->getResponseHeaders() as $name => $value) {
63			$response = $response->withHeader($name, $value);
64			}
65
66			return $next($request, $response);
67			}
68			}
69
70			/**
71			* @return SettingsStrategyInterface
72			*/
73			public function getSettings()
74			{
75			return $this->settings;
76			}
77
78			/**
79			* @param AnalyzerInterface $analyzer
80			*
81			* @return $this
82			*/
83			public function setAnalyzer($analyzer)
84			{
85			$this->analyzer = $analyzer;
86
87			return $this;
88			}
89			}
90

ThrusterIO / http-middlewares

CorsMiddleware A last analyzed 2016-08-30 10:44 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

4 Methods

Duplication Side-by-Side

Filter issues like

CorsMiddleware A
last analyzed 2016-08-30 10:44 UTC