Waca\Providers\CachedRDnsLookupProvider::getReverseDNS()

Complexity

Conditions	6
Paths	6

Size

Total Lines	39
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getReverseDNS($address)
    {
        $address = trim($address);

        // lets look in our cache database first.
        $rDns = RDnsCache::getByAddress($address, $this->database);

        if ($rDns instanceof RDnsCache) {
            // touch cache timer
            $rDns->save();

            return $rDns->getData();
        }

        // OK, it's not there, let's do an rDNS lookup.
        $ptrAddress = inet_pton($address);
        if ($ptrAddress === false) {
            return null; // Invalid IP address
        }

        $reversePointer = implode('.', array_reverse(explode('.', inet_ntop($ptrAddress)))) . '.in-addr.arpa';
        if (filter_var($address, FILTER_VALIDATE_IP, FILTER_FLAG_IPV6)) {
            $reversePointer = implode('.', array_reverse(str_split(bin2hex($ptrAddress)))) . '.ip6.arpa';

It seems like str_split(bin2hex($ptrAddress)) can also be of type true; however, parameter $array of array_reverse() does only seem to accept array, maybe add an additional type check?

        }

        $dnsRecords = dns_get_record($reversePointer, DNS_PTR);
        if (!empty($dnsRecords) && isset($dnsRecords[0]['target'])) {
            $result = $dnsRecords[0]['target'];

            $rDns = new RDnsCache();
            $rDns->setDatabase($this->database);
            $rDns->setAddress($address);
            $rDns->setData($result);
            $rDns->save();

            return $result;
        }

        return null;
    }