silverstripe / silverstripe-environmentcheck

src/Traits/Fetcher.php

Indentation +29 added lines, -29 removed lines

@@ -12,35 +12,35 @@
  */
 trait Fetcher
 {
-    /**
-     * Configuration for the Guzzle client
-     *
-     * @var array
-     */
-    protected $clientConfig = [];
+	/**
+	 * Configuration for the Guzzle client
+	 *
+	 * @var array
+	 */
+	protected $clientConfig = [];
 
-    /**
-     * Merges configuration arrays and returns the result
-     *
-     * @param array $extraConfig
-     * @return array
-     */
-    private function getClientConfig(array $extraConfig = [])
-    {
-        return array_merge($this->clientConfig, $extraConfig);
-    }
+	/**
+	 * Merges configuration arrays and returns the result
+	 *
+	 * @param array $extraConfig
+	 * @return array
+	 */
+	private function getClientConfig(array $extraConfig = [])
+	{
+		return array_merge($this->clientConfig, $extraConfig);
+	}
 
-    /**
-     * Fetch a response for a URL using Guzzle client.
-     *
-     * @param string $url
-     * @param array|null $extraConfig Extra configuration
-     * @return ResponseInterface
-     */
-    public function fetchResponse(string $url, array $extraConfig = [])
-    {
-        $config = $this->getClientConfig($extraConfig);
-        $client = new Client($config);
-        return $client->get($url);
-    }
+	/**
+	 * Fetch a response for a URL using Guzzle client.
+	 *
+	 * @param string $url
+	 * @param array|null $extraConfig Extra configuration
+	 * @return ResponseInterface
+	 */
+	public function fetchResponse(string $url, array $extraConfig = [])
+	{
+		$config = $this->getClientConfig($extraConfig);
+		$client = new Client($config);
+		return $client->get($url);
+	}
 }

src/Checks/SessionCheck.php

Indentation +60 added lines, -60 removed lines

@@ -17,70 +17,70 @@
  */
 class SessionCheck implements EnvironmentCheck
 {
-    use Configurable;
-    use Fetcher;
+	use Configurable;
+	use Fetcher;
 
-    /**
-     * URL to check
-     *
-     * @var string
-     */
-    protected $url;
+	/**
+	 * URL to check
+	 *
+	 * @var string
+	 */
+	protected $url;
 
-    /**
-     * Set up check with URL
-     *
-     * @param string $url The route, excluding the domain
-     * @inheritdoc
-     */
-    public function __construct($url = '')
-    {
-        $this->url = $url;
-        $this->clientConfig = [
-            'base_uri' => Director::absoluteBaseURL(),
-            'timeout' => 10.0,
-        ];
-    }
+	/**
+	 * Set up check with URL
+	 *
+	 * @param string $url The route, excluding the domain
+	 * @inheritdoc
+	 */
+	public function __construct($url = '')
+	{
+		$this->url = $url;
+		$this->clientConfig = [
+			'base_uri' => Director::absoluteBaseURL(),
+			'timeout' => 10.0,
+		];
+	}
 
-    /**
-     * Check that the response for URL does not create a session
-     *
-     * @return array
-     */
-    public function check()
-    {
-        $response = $this->fetchResponse($this->url);
-        $cookie = $this->getCookie($response);
-        $fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
+	/**
+	 * Check that the response for URL does not create a session
+	 *
+	 * @return array
+	 */
+	public function check()
+	{
+		$response = $this->fetchResponse($this->url);
+		$cookie = $this->getCookie($response);
+		$fullURL = Controller::join_links(Director::absoluteBaseURL(), $this->url);
 
-        if ($cookie) {
-            return [
-                EnvironmentCheck::ERROR,
-                "Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
-            ];
-        }
-        return [
-            EnvironmentCheck::OK,
-            "Sessions are not being created for {$fullURL}