Mediadevs\Validator\Traits\HostTrait::checkWhetherHostIsLive()

Complexity

Conditions	2
Paths	2

Size

Total Lines	31
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	2

Importance

Changes

0

<?php
    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;
    }