DnsValidator::validateNameServers() - Code Metrics - Inspection of "Revert "Fix docker machine create https://github.c..." - chrif/cocotte - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 956163...dedbc7 )

by Christian

created 2021-03-06 15:38 UTC

DnsValidator::validateNameServers() A

↳ Parent: DnsValidator

Complexity

Conditions	3
Paths	3

Size

Total Lines	23
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	17
nc	3
nop	1
dl	0
loc	23
rs	9.7
c	0
b	0
f	0

<?php declare(strict_types=1);

namespace Cocotte\DigitalOcean;

use Assert\Assertion;
use Cocotte\Shell\Env;
use Iodev\Whois\Modules\Tld\DomainInfo;
use Iodev\Whois\Whois;

class DnsValidator
{
    /**
     * @var Whois
     */
    private $whois;
    /**
     * @var Env
     */
    private $env;

    public function __construct(Whois $whois, Env $env)
    {
        $this->whois = $whois;
        $this->env = $env;
    }

    public function validateHost(Hostname $hostname)
    {
        if ($this->env->get(SkipDnsValidation::SKIP_DNS_VALIDATION)) {
            return;
        }

        try {
            $this->validateNameServers($hostname->toRoot());
        } catch (\Throwable $domainException) {
            $message = [];
            $message[] = "Failed to validate name servers for '$hostname':";
            $message[] = $domainException->getMessage();
            throw new \Exception(implode("\n", $message));
        }
    }

    protected function validateNameServers(Hostname $hostname)
    {
        $info = $this->whois->loadDomainInfo($hostname->domainName());
        if (!$info instanceof DomainInfo) {

            throw new \Exception("Could not load domain info of $hostname");
        }
        $nameServers = $info->getNameServers();
        Assertion::isArray(
            $nameServers,
            "Expected array. Got: ".var_export($nameServers, true)
        );
        Assertion::greaterThan(
            count($nameServers),
            0,
            "Expected 1 or more name servers. Got ".count($nameServers)
        );
        foreach ($nameServers as $nameServer) {
            $nameServer = Hostname::parse($nameServer);
            Assertion::regex(
                $nameServer->toString(),
                "/ns[0-9]\.digitalocean\.com/",
                "'{$nameServer->toString()}' is not a Digital Ocean's name server in the ".
                "form of ns[0-9].digitalocean.com"
            );
        }
    }
}


1			<?php declare(strict_types=1);
2
3			namespace Cocotte\DigitalOcean;
4
5			use Assert\Assertion;
6			use Cocotte\Shell\Env;
7			use Iodev\Whois\Modules\Tld\DomainInfo;
8			use Iodev\Whois\Whois;
9
10			class DnsValidator
11			{
12			/**
13			* @var Whois
14			*/
15			private $whois;
16			/**
17			* @var Env
18			*/
19			private $env;
20
21			public function __construct(Whois $whois, Env $env)
22			{
23			$this->whois = $whois;
24			$this->env = $env;
25			}
26
27			public function validateHost(Hostname $hostname)
28			{
29			if ($this->env->get(SkipDnsValidation::SKIP_DNS_VALIDATION)) {
30			return;
31			}
32
33			try {
34			$this->validateNameServers($hostname->toRoot());
35			} catch (\Throwable $domainException) {
36			$message = [];
37			$message[] = "Failed to validate name servers for '$hostname':";
38			$message[] = $domainException->getMessage();
39			throw new \Exception(implode("\n", $message));
40			}
41			}
42
43			protected function validateNameServers(Hostname $hostname)
44			{
45			$info = $this->whois->loadDomainInfo($hostname->domainName());
46			if (!$info instanceof DomainInfo) {
			0 ignored issues – show introduced 2018-11-24 23:17 UTC by Report Bug Copy Issue Report `$info` is always a sub-type of `Iodev\Whois\Modules\Tld\DomainInfo`. Loading history...
47			throw new \Exception("Could not load domain info of $hostname");
48			}
49			$nameServers = $info->getNameServers();
50			Assertion::isArray(
51			$nameServers,
52			"Expected array. Got: ".var_export($nameServers, true)
53			);
54			Assertion::greaterThan(
55			count($nameServers),
56			0,
57			"Expected 1 or more name servers. Got ".count($nameServers)
58			);
59			foreach ($nameServers as $nameServer) {
60			$nameServer = Hostname::parse($nameServer);
61			Assertion::regex(
62			$nameServer->toString(),
63			"/ns[0-9]\.digitalocean\.com/",
64			"'{$nameServer->toString()}' is not a Digital Ocean's name server in the ".
65			"form of ns[0-9].digitalocean.com"
66			);
67			}
68			}
69			}
70

chrif / cocotte

Push — master ( 956163...dedbc7 )

DnsValidator::validateNameServers() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like