elliotjon / weather-module

src/Models/IpValidator.php

<?php

/**
 * Class IpValidator
 * Validate ip and return result.
*/

namespace Anax\Models;

use Anax\Config\access;

The type Anax\Config\access was not found. Maybe you did not declare it correctly or list all dependencies?

class IpValidator
{
    private $keys;

    public function setKeys($config)
    {
        $this->keys = $config;
    }

    /**
     * Curl from url
     */
    public function curl($url)
    {
          $curl = curl_init($url);
          curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);

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

          $data = curl_exec($curl);

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

          curl_close($curl);

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

          return $data;
    }

    /**
     * Validate IP-adress
     */
    public function validateIp($ipAdress)
    {
        $accessToken = $this->keys["ipValidator"];
        $stackUrl = 'http://api.ipstack.com/' . $ipAdress . '?access_key=' . $accessToken . '';

        $host = "";
        $stack = "";
        $type = "";
        $valid = "";

        if (filter_var($ipAdress, FILTER_VALIDATE_IP, FILTER_FLAG_IPV4)) {
            $host = gethostbyaddr($ipAdress);
            $stack = json_decode($this->curl($stackUrl));
            $type = "IPv4";
            $valid = true;
        } elseif (filter_var($ipAdress, FILTER_VALIDATE_IP, FILTER_FLAG_IPV6)) {
            $host = gethostbyaddr($ipAdress);
            $stack = json_decode($this->curl($stackUrl));
            $type = "IPv6";
            $valid = true;
        } else {
            $valid = false;
        }

        $data = [
            "ip" => $ipAdress,
            "valid" => $valid,
            "type" => $type,
            "host" => $host,
            "stack" => $stack
        ];
        return $data;
    }

    /**
     * Get current IP-adress
     */
    public function getYourIp($request)
    {
        if (!empty($request->getServer('HTTP_CLIENT_IP'))) {
            $ipAdress = $server('HTTP_CLIENT_IP');

The variable $server seems to be never defined.

        } elseif (!empty($request->getServer('HTTP_X_FORWARDED_FOR'))) {
            $ipAdress = $request->getServer('HTTP_X_FORWARDED_FOR');
        } else {
            $ipAdress = $request->getServer('REMOTE_ADDR');
        }

        return $ipAdress;
    }
}