GlobalSession::start() - Code Metrics - Inspection of "Update jasny/phpunit-extension" - jasny/sso - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( d43928...b69075 )

by Arnold

created 2020-12-17 11:01 UTC

GlobalSession::start() A

↳ Parent: GlobalSession

Complexity

Conditions	3
Paths	4

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	7
c	1
b	0
f	0
dl	0
loc	13
rs	10
cc	3
nc	4
nop	0

<?php

declare(strict_types=1);

namespace Jasny\SSO\Server;

/**
 * Interact with the global session using PHP's session_* functions.
 *
 * @codeCoverageIgnore
 */
class GlobalSession implements SessionInterface
{
    /**
     * Options passed to session_start().
     * @var array<string,mixed>
     */
    protected $options;

    /**
     * Class constructor.
     *
     * @param array<string,mixed> $options  Options passed to session_start().
     */
    public function __construct(array $options = [])
    {
        $this->options = $options + ['cookie_samesite' => 'None'];
    }

    /**
     * @inheritDoc
     */
    public function getId(): string
    {
        return session_id();

    }

    /**
     * @inheritDoc
     */
    public function start(): void
    {
        $started = session_status() !== PHP_SESSION_ACTIVE
            ? session_start($this->options)
            : true;

        if (!$started) {
            $err = error_get_last() ?? ['message' => 'Failed to start session'];
            throw new ServerException($err['message'], 500);
        }

        // Session shouldn't be empty when resumed.
        $_SESSION['_sso_init'] = 1;
    }

    /**
     * @inheritDoc
     */
    public function resume(string $id): void
    {
        session_id($id);
        $started = session_start($this->options);

        if (!$started) {
            $err = error_get_last() ?? ['message' => 'Failed to start session'];
            throw new ServerException($err['message'], 500);
        }

        if ($_SESSION === []) {
            session_abort();
            throw new BrokerException("Session has expired. Client must attach with new token.", 401);
        }
    }

    /**
     * @inheritDoc
     */
    public function isActive(): bool
    {
        return session_status() === PHP_SESSION_ACTIVE;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Jasny\SSO\Server;
6
7			/**
8			* Interact with the global session using PHP's session_* functions.
9			*
10			* @codeCoverageIgnore
11			*/
12			class GlobalSession implements SessionInterface
13			{
14			/**
15			* Options passed to session_start().
16			* @var array<string,mixed>
17			*/
18			protected $options;
19
20			/**
21			* Class constructor.
22			*
23			* @param array<string,mixed> $options Options passed to session_start().
24			*/
25			public function __construct(array $options = [])
26			{
27			$this->options = $options + ['cookie_samesite' => 'None'];
28			}
29
30			/**
31			* @inheritDoc
32			*/
33			public function getId(): string
34			{
35			return session_id();
			0 ignored issues – show introduced 2020-12-17 11:10 UTC by Report Bug Copy Issue Report Method Jasny\SSO\Server\GlobalSession::getId() should return string but returns string\|false. Loading history...
36			}
37
38			/**
39			* @inheritDoc
40			*/
41			public function start(): void
42			{
43			$started = session_status() !== PHP_SESSION_ACTIVE
44			? session_start($this->options)
45			: true;
46
47			if (!$started) {
48			$err = error_get_last() ?? ['message' => 'Failed to start session'];
49			throw new ServerException($err['message'], 500);
50			}
51
52			// Session shouldn't be empty when resumed.
53			$_SESSION['_sso_init'] = 1;
54			}
55
56			/**
57			* @inheritDoc
58			*/
59			public function resume(string $id): void
60			{
61			session_id($id);
62			$started = session_start($this->options);
63
64			if (!$started) {
65			$err = error_get_last() ?? ['message' => 'Failed to start session'];
66			throw new ServerException($err['message'], 500);
67			}
68
69			if ($_SESSION === []) {
70			session_abort();
71			throw new BrokerException("Session has expired. Client must attach with new token.", 401);
72			}
73			}
74
75			/**
76			* @inheritDoc
77			*/
78			public function isActive(): bool
79			{
80			return session_status() === PHP_SESSION_ACTIVE;
81			}
82			}
83

jasny / sso

Push — master ( d43928...b69075 )

GlobalSession::start() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like