SessionCheck::check() - Code Metrics - Inspection of "Merge pull request #60 from silverstripe-terraform..." - silverstripe/silverstripe-environmentcheck - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 128ddb...c5783a )

unknown

created 2019-03-19 22:51 UTC

SessionCheck::check() A

↳ Parent: SessionCheck

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	10
nc	2
nop	0
dl	0
loc	15
rs	9.9332
c	0
b	0
f	0

<?php

namespace SilverStripe\EnvironmentCheck\Checks;

use Psr\Http\Message\ResponseInterface;
use SilverStripe\EnvironmentCheck\Traits\Fetcher;
use SilverStripe\EnvironmentCheck\EnvironmentCheck;

/**
 * Check that a given URL does not generate a session.
 *
 * @author Adrian Humphreys
 * @package environmentcheck
 */
class SessionCheck implements EnvironmentCheck
{
    use Fetcher;

    /**
     * Set up check with URL
     *
     * @param string $url The route, excluding the domain
     * @inheritdoc
     */
    public function __construct($url = '')
    {
        $this->setURL($url);
    }

    /**
     * Check that the response for URL does not create a session
     *
     * @return array
     */
    public function check()
    {
        $response = $this->client->get($this->getURL());
        $cookie = $this->getCookie($response);
        $fullURL = $this->getURL();

        if ($cookie) {
            return [
                EnvironmentCheck::ERROR,
                "Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
            ];
        }
        return [
            EnvironmentCheck::OK,
            "Sessions are not being created for {$fullURL} 👍",
        ];
    }

    /**
     * Get PHPSESSID or SECSESSID cookie set from the response if it exists.
     *
     * @param ResponseInterface $response
     * @return string|null Cookie contents or null if it doesn't exist
     */
    public function getCookie(ResponseInterface $response)
    {
        $result = null;
        $cookies = $response->getHeader('Set-Cookie');

        foreach ($cookies as $cookie) {
            if (strpos($cookie, 'SESSID') !== false) {
                $result = $cookie;
            }
        }
        return $result;
    }
}


1			<?php
2
3			namespace SilverStripe\EnvironmentCheck\Checks;
4
5			use Psr\Http\Message\ResponseInterface;
6			use SilverStripe\EnvironmentCheck\Traits\Fetcher;
7			use SilverStripe\EnvironmentCheck\EnvironmentCheck;
8
9			/**
10			* Check that a given URL does not generate a session.
11			*
12			* @author Adrian Humphreys
13			* @package environmentcheck
14			*/
15			class SessionCheck implements EnvironmentCheck
16			{
17			use Fetcher;
18
19			/**
20			* Set up check with URL
21			*
22			* @param string $url The route, excluding the domain
23			* @inheritdoc
24			*/
25			public function __construct($url = '')
26			{
27			$this->setURL($url);
28			}
29
30			/**
31			* Check that the response for URL does not create a session
32			*
33			* @return array
34			*/
35			public function check()
36			{
37			$response = $this->client->get($this->getURL());
38			$cookie = $this->getCookie($response);
39			$fullURL = $this->getURL();
40
41			if ($cookie) {
42			return [
43			EnvironmentCheck::ERROR,
44			"Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
45			];
46			}
47			return [
48			EnvironmentCheck::OK,
49			"Sessions are not being created for {$fullURL} 👍",
50			];
51			}
52
53			/**
54			* Get PHPSESSID or SECSESSID cookie set from the response if it exists.
55			*
56			* @param ResponseInterface $response
57			* @return string\|null Cookie contents or null if it doesn't exist
58			*/
59			public function getCookie(ResponseInterface $response)
60			{
61			$result = null;
62			$cookies = $response->getHeader('Set-Cookie');
63
64			foreach ($cookies as $cookie) {
65			if (strpos($cookie, 'SESSID') !== false) {
66			$result = $cookie;
67			}
68			}
69			return $result;
70			}
71			}
72

silverstripe / silverstripe-environmentcheck

Push — master ( 128ddb...c5783a )

SessionCheck::check() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like