WebProcessorMonolog::processRecord() - Code Metrics - Inspection of "Log host, URI, and session token in error reports" - x-tools/xtools - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d3179a...94c21a )

by MusikAnimal

created 2019-01-05 03:23 UTC

WebProcessorMonolog::processRecord() A

↳ Parent: WebProcessorMonolog

Complexity

Conditions	5
Paths	4

Size

Total Lines	22
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	5

Importance

Changes

Metric	Value
cc	5
eloc	11
nc	4
nop	1
dl	0
loc	22
ccs	12
cts	12
cp	1
crap	5
rs	9.6111
c	0
b	0
f	0

<?php
/**
 * This file contains only the WebProcessorMonolog class.
 */

declare(strict_types = 1);

namespace AppBundle\Monolog;

use Symfony\Component\HttpFoundation\RequestStack;

/**
 * WebProcessorMonolog extends information included in error reporting.
 */
class WebProcessorMonolog
{
    /** @var RequestStack The request stack. */
    private $requestStack;

    /** @var string The unique identifier for the session. */
    private $sessionId;

    /**
     * WebProcessorMonolog constructor.
     * @param RequestStack $requestStack
     */
    public function __construct(RequestStack $requestStack)
    {
        $this->requestStack = $requestStack;
    }

    /**
     * Adds extra information to the log entry.
     * @param array $record
     * @return array
     */
    public function processRecord(array $record): array
    {
        $request = $this->requestStack->getCurrentRequest();

        if ($request) {
            $record['extra']['host'] = $request->getHost();
            $record['extra']['url'] = $request->getUri();

            $session = $request->getSession();

            if (!$session->isStarted()) {
                return $record;
            }

            if (!$this->sessionId) {
                $this->sessionId = substr($session->getId(), 0, 8) ?: '????????';
            }

            $record['extra']['token'] = $this->sessionId.'-'.substr(uniqid('', true), -8);
        }

        return $record;
    }
}


1		<?php
2		/**
3		* This file contains only the WebProcessorMonolog class.
4		*/
5
6		declare(strict_types = 1);
7
8		namespace AppBundle\Monolog;
9
10		use Symfony\Component\HttpFoundation\RequestStack;
11
12		/**
13		* WebProcessorMonolog extends information included in error reporting.
14		*/
15		class WebProcessorMonolog
16		{
17		/** @var RequestStack The request stack. */
18		private $requestStack;
19
20		/** @var string The unique identifier for the session. */
21		private $sessionId;
22
23		/**
24		* WebProcessorMonolog constructor.
25		* @param RequestStack $requestStack
26		*/
27	27	public function __construct(RequestStack $requestStack)
28		{
29	27	$this->requestStack = $requestStack;
30	27	}
31
32		/**
33		* Adds extra information to the log entry.
34		* @param array $record
35		* @return array
36		*/
37	17	public function processRecord(array $record): array
38		{
39	17	$request = $this->requestStack->getCurrentRequest();
40
41	17	if ($request) {
42	15	$record['extra']['host'] = $request->getHost();
43	15	$record['extra']['url'] = $request->getUri();
44
45	15	$session = $request->getSession();
46
47	15	if (!$session->isStarted()) {
48	15	return $record;
49		}
50
51	2	if (!$this->sessionId) {
52	2	$this->sessionId = substr($session->getId(), 0, 8) ?: '????????';
53		}
54
55	2	$record['extra']['token'] = $this->sessionId.'-'.substr(uniqid('', true), -8);
56		}
57
58	17	return $record;
59		}
60		}
61

x-tools / xtools

Push — master ( d3179a...94c21a )

WebProcessorMonolog::processRecord() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like