Mediadevs\Validator\Traits\HostTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	checkWhetherHostIsLive()	0	31	2

<?php

namespace Mediadevs\Validator\Traits;

trait HostTrait
{
    /**
     * Pinging the hostname to see if it is live / an actual working hostname.
     *
     * @param string $host
     *
     * @return bool
     */
    protected function checkWhetherHostIsLive(string $host): bool
    {
        $statuses = array(
            200,
            301,
            302,
        );

        // Initializing curl
        $curlHandle = curl_init($host);

        // Calling the host for a response
        curl_setopt_array($curlHandle, array(

It seems like $curlHandle can also be of type false; however, parameter $ch of curl_setopt_array() does only seem to accept resource, maybe add an additional type check?

            CURLOPT_HEADER          => true,
            CURLOPT_NOBODY          => true,
            CURLOPT_RETURNTRANSFER  => true,
            CURLOPT_TIMEOUT         => 10,
            CURLOPT_USERAGENT       => 'page-check/1.0',
        ));

        // Executing request
        curl_exec($curlHandle);

It seems like $curlHandle can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        // Collecting the status code and casting it to an integer
        $status = (int) curl_getinfo($curlHandle, CURLINFO_HTTP_CODE);

It seems like $curlHandle can also be of type false; however, parameter $ch of curl_getinfo() does only seem to accept resource, maybe add an additional type check?

        // Closing curl
        curl_close($curlHandle);

It seems like $curlHandle can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        // Validating whether the http status is inside the allowed statuses array (Whether the page is live or not)
        return in_array($status, $statuses) ? true : false;
    }
}