RedisCollection::parseDsn() - Code Metrics - Inspection of "Merge pull request #208 from Basster/master" - liip/LiipMonitorBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8fe011...a16b04 )

by Kevin

created 2019-05-23 14:27 UTC

RedisCollection::parseDsn() A

↳ Parent: RedisCollection

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.6666
c	0
b	0
f	0
cc	3
nc	3
nop	1

<?php

namespace Liip\MonitorBundle\Check;

use ZendDiagnostics\Check\CheckCollectionInterface;
use ZendDiagnostics\Check\Redis;

/**
 * @author Kevin Bond <[email protected]>
 */
class RedisCollection implements CheckCollectionInterface
{
    private $checks = [];

    public function __construct(array $configs)
    {
        foreach ($configs as $name => $config) {
            if (isset($config['dsn'])) {
                $this->parseDsn($config);
            }

            $check = new Redis($config['host'], $config['port'], $config['password']);
            $check->setLabel(\sprintf('Redis "%s"', $name));

            $this->checks[\sprintf('redis_%s', $name)] = $check;
        }
    }

    private function parseDsn(array &$config)
    {
        $config = \array_merge($config, \parse_url($config['dsn']));

        if (isset($config['pass'])) {
            $config['password'] = $config['pass'];
            // Cleanup
            unset($config['pass']);
        } elseif (isset($config['user'])) {
            /*
             * since "redis://my-super-secret-password@redis-host:6379" is a valid redis
             * dsn but \parse_url does not understand this notation and extracts the auth as user,
             * we need to check for it.
             */
            $config['password'] = $config['user'];
            unset($config['user']);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function getChecks()
    {
        return $this->checks;
    }
}


1			<?php
2
3			namespace Liip\MonitorBundle\Check;
4
5			use ZendDiagnostics\Check\CheckCollectionInterface;
6			use ZendDiagnostics\Check\Redis;
7
8			/**
9			* @author Kevin Bond <[email protected]>
10			*/
11			class RedisCollection implements CheckCollectionInterface
12			{
13			private $checks = [];
14
15			public function __construct(array $configs)
16			{
17			foreach ($configs as $name => $config) {
18			if (isset($config['dsn'])) {
19			$this->parseDsn($config);
20			}
21
22			$check = new Redis($config['host'], $config['port'], $config['password']);
23			$check->setLabel(\sprintf('Redis "%s"', $name));
24
25			$this->checks[\sprintf('redis_%s', $name)] = $check;
26			}
27			}
28
29			private function parseDsn(array &$config)
30			{
31			$config = \array_merge($config, \parse_url($config['dsn']));
32
33			if (isset($config['pass'])) {
34			$config['password'] = $config['pass'];
35			// Cleanup
36			unset($config['pass']);
37			} elseif (isset($config['user'])) {
38			/*
39			* since "redis://my-super-secret-password@redis-host:6379" is a valid redis
40			* dsn but \parse_url does not understand this notation and extracts the auth as user,
41			* we need to check for it.
42			*/
43			$config['password'] = $config['user'];
44			unset($config['user']);
45			}
46			}
47
48			/**
49			* {@inheritdoc}
50			*/
51			public function getChecks()
52			{
53			return $this->checks;
54			}
55			}
56

liip / LiipMonitorBundle

Push — master ( 8fe011...a16b04 )

RedisCollection::parseDsn() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like