.gitignore

@@ -1 +1,3 @@
+_*
+.old*
 vendor/*

LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2020-2022 kristuff
+Copyright (c) 2020-2021 kristuff
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -14,10 +14,9 @@ Features
 - IP block check request **✓** 
 - Blacklist request **✓** 
 - Single IP report request **✓** 
-- Bulk report request (send `csv` file) **✓** 
-- Clear IP address request (remove your own reports) **✓**
-- Auto cleaning report comments from sensitive data (email, custom ip/domain names list) **✓** 
-- Define timeout for cURL internal requests **✓**
+- Bulk report request (send cvs file) **✓** 
+- Clear IP address request (remove own reports) **✓**
+- Auto cleaning report comment from sensitive data  **✓** 
 
 Requirements
 ------------    
@@ -33,24 +32,25 @@ Deploy with composer:
 ```json
 ...
 "require": {
-    "kristuff/abuseipdb": "^1.1-stable"
+    "kristuff/abuseipdb": ">=0.9.7-stable"
 },
 ```
 
-More infos
+Usage
 -----
 
-- [Project website](https://kristuff.fr/projects/abuseipdb)
-- [Api documentation](https://kristuff.fr/projects/abuseipdb/doc)
-- CLI version: [github](https://github.com/kristuff/abuseipdb-cli) | [website](https://kristuff.fr/projects/abuseipdbcli)
+```php
+<?php
 
+echo ('TODO');
+```
 
 License
 -------
 
 The MIT License (MIT)
 
-Copyright (c) 2020-2022 Kristuff
+Copyright (c) 2020-2021 Kristuff
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

composer.json

@@ -2,14 +2,11 @@
     "name": "kristuff/abuseipdb",
     "description": "A PHP wrapper for AbuseIPDB API v2",
     "type": "library",
-    "keywords": ["abuseIPDB", "API"],
     "license": "MIT",
     "authors": [
         {
             "name": "Kristuff",
-            "homepage": "https://github.com/kristuff",
-            "email": "kristuff@kristuff.fr",
-            "role": "Developer"
+            "homepage": "https://github.com/kristuff"
         }
     ],
     "require": {

lib/ApiBase.php

@@ -1,20 +1,21 @@
-<?php declare(strict_types=1);
+<?php
 
 /**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
+ *     _    _                    ___ ____  ____  ____
+ *    / \  | |__  _   _ ___  ___|_ _|  _ \|  _ \| __ )
+ *   / _ \ | '_ \| | | / __|/ _ \| || |_) | | | |  _ \
+ *  / ___ \| |_) | |_| \__ \  __/| ||  __/| |_| | |_) |
+ * /_/   \_\_.__/ \__,_|___/\___|___|_|   |____/|____/
  *
- * (c) Kristuff <kristuff@kristuff.fr>
+ * This file is part of Kristuff\AbsuseIPDB.
+ *
+ * (c) Kristuff <contact@kristuff.fr>
  *
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
+ * @version    0.9.7
+ * @copyright  2020-2021 Kristuff
  */
 
 namespace Kristuff\AbuseIPDB;
@@ -40,16 +41,23 @@ abstract class ApiBase
      * @var string $aipdbApiKey  
      */
     protected $aipdbApiKey = null; 
-  
+
+    /**
+     * AbuseIPDB user id 
+     * 
+     * @access protected
+     * @var string $aipdbUserId  
+     */
+    protected $aipdbUserId = null; 
+
     /**
      * AbuseIPDB API v2 categories
-     * shortname, id (string), long name
-     * last parameter is false when the category can't be used alone
+     * shorname, id (string), long name
+     * last paramter is false when the category cant' be used alone
      * 
-     * @static
      * @var array
      */
-    protected static $aipdbApiCategories = [
+    protected $aipdbApiCategories = [
         
         // Altering DNS records resulting in improper redirection.        
         ['dns-c'           , '1', 'DNS Compromise', true],    
@@ -127,36 +135,33 @@ abstract class ApiBase
 
         // Abuse was targeted at an "Internet of Things" type device. Include 
         // information about what type of device was targeted in the comments.         
-        ['iot'             , '23', 'IoT Targeted', true],     
+        ['oit'             , '23', 'IoT Targeted', true],     
     ];
 
     /**
      * Get the list of report categories
      * 
-     * @access public
-     * @static
-     *  
+     * @access public 
      * @return array
      */
-    public static function getCategories(): array
+    public function getCategories()
     {
-        return self::$aipdbApiCategories;
+        return $this->aipdbApiCategories;
     }
 
     /**
      * Get the category id corresponding to given name
      * 
      * @access public
-     * @static
-     * @param string $categoryName    The report category name
+     * @param string $categoryName    The report categoriy name
      * 
      * @return string|bool            The category id in string format if found, otherwise false
      */
-    public static function getCategoryIdByName(string $categoryName)
+    public function getCategoryIdbyName(string $categoryName)
     {
-        foreach (self::$aipdbApiCategories as $cat){
+        foreach ($this->aipdbApiCategories as $cat){
             if ($cat[0] === $categoryName) {
-                return $cat[1];
+                return $cat;
             }
          }
 
@@ -168,16 +173,15 @@ abstract class ApiBase
      * Get the category name corresponding to given id
      * 
      * @access public
-     * @static
      * @param string    $categoryId   The report category id
      * 
      * @return string|bool            The category name if found, otherwise false
      */
-    public static function getCategoryNameById(string $categoryId)
+    public function getCategoryNameById(string $categoryId)
     {
-        foreach (self::$aipdbApiCategories as $cat){
+        foreach ($this->aipdbApiCategories as $cat){
            if ($cat[1] === $categoryId) {
-               return $cat[0];
+               return $cat;
            }
         }
 
@@ -189,16 +193,15 @@ abstract class ApiBase
      * Get the index of category corresponding to given value
      * 
      * @access protected
-     * @static
-     * @param string        $value          The report category id or name
-     * @param string        $index          The index in value array 
+     * @param string    $value          The report category id or name
+     * @param string    $index          The index in value array 
      * 
-     * @return int|bool     The category index if found, otherwise false
+     * @return int|bool                 The category index if found, otherwise false
      */
-    protected static function getCategoryIndex(string $value, int $index)
+    protected function getCategoryIndex(string $value, int $index)
     {
         $i = 0;
-        foreach (self::$aipdbApiCategories as $cat){
+        foreach ($this->aipdbApiCategories as $cat){
             if ($cat[$index] === $value) {
                 return $i;
             }
@@ -214,9 +217,9 @@ abstract class ApiBase
      * Check for shortname or id, and categories that can't be used alone 
      * 
      * @access protected
-     * @param array         $categories       The report categories list
+     * @param array $categories       The report categories list
      *
-     * @return string       Formatted string id list ('18,2,3...')
+     * @return string               Formatted string id list ('18,2,3...')
      * @throws \InvalidArgumentException
      */
     protected function validateReportCategories(string $categories)
@@ -233,21 +236,21 @@ abstract class ApiBase
         foreach ($cats as $cat) {
 
             // get index on our array of categories
-            $catIndex    = is_numeric($cat) ? self::getCategoryIndex($cat, 1) : self::getCategoryIndex($cat, 0);
+            $catIndex    = is_numeric($cat) ? $this->getCategoryIndex($cat, 1) : $this->getCategoryIndex($cat, 0);
 
             // check if found
             if ($catIndex === false ){
-                throw new \InvalidArgumentException('Invalid report category was given.');
+                throw new \InvalidArgumentException('Invalid report category was given : ['. $cat .  ']');
             }
 
             // get Id
-            $catId = self::$aipdbApiCategories[$catIndex][1];
+            $catId = $this->aipdbApiCategories[$catIndex][1];
 
             // need another ?
             if ($needAnother !== false){
 
                 // is a standalone cat ?
-                if (self::$aipdbApiCategories[$catIndex][3] === false) {
+                if ($this->aipdbApiCategories[$catIndex][3] === false) {
                     $needAnother = true;
 
                 } else {
@@ -262,7 +265,7 @@ abstract class ApiBase
         }
 
         if ($needAnother !== false){
-            throw new \InvalidArgumentException('Invalid report category parameter given: this category can\'t be used alone.');
+            throw new \InvalidArgumentException('Invalid report category paremeter given: some categories can\'t be used alone');
         }
 
         // if here that ok

lib/ApiHandler.php

@@ -1,20 +1,21 @@
-<?php declare(strict_types=1);
+<?php
 
 /**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
+ *     _    _                    ___ ____  ____  ____
+ *    / \  | |__  _   _ ___  ___|_ _|  _ \|  _ \| __ )
+ *   / _ \ | '_ \| | | / __|/ _ \| || |_) | | | |  _ \
+ *  / ___ \| |_) | |_| \__ \  __/| ||  __/| |_| | |_) |
+ * /_/   \_\_.__/ \__,_|___/\___|___|_|   |____/|____/
  *
- * (c) Kristuff <kristuff@kristuff.fr>
+ * This file is part of Kristuff\AbsuseIPDB.
+ *
+ * (c) Kristuff <contact@kristuff.fr>
  *
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
+ * @version    0.9.7
+ * @copyright  2020-2021 Kristuff
  */
 
 namespace Kristuff\AbuseIPDB;
@@ -32,77 +33,84 @@ class ApiHandler extends ApiBase
     use CurlTrait;
 
     /**
-     * @var string
-     */
-    const VERSION = 'v1.1'; 
-
-    /**
-     * The ips to remove from report messages
-     * Generally you will add to this list yours ipv4 and ipv6, hostname, domain names
+     * The ips to remove from message
+     * Generally you will add to this list yours ipv4 and ipv6, and the hostname
      * 
      * @access protected
-     * @var array  
+     * @var array $selfIps  
      */
     protected $selfIps = []; 
 
-    /**
-     * The maximum number of milliseconds to allow cURL functions to execute. If libcurl is 
-     * built to use the standard system name resolver, that portion of the connect will still 
-     * use full-second resolution for timeouts with a minimum timeout allowed of one second. 
-     * 
-     * @access protected
-     * @var int  
-     */
-    protected $timeout = 0; 
-
     /**
      * Constructor
      * 
      * @access public
      * @param string  $apiKey     The AbuseIPDB api key
-     * @param array   $myIps      The Ips/domain name you don't want to display in report messages
-     * @param int     $timeout    The maximum number of milliseconds to allow internal cURL functions 
-     *                            to execute. Default is 0, no timeout
+     * @param string  $userId     The AbuseIPDB user's id
+     * @param array   $myIps      The Ips/domain name you dont want to display in report messages
      * 
      */
-    public function __construct(string $apiKey, array $myIps = [], int $timeout = 0)
+    public function __construct(string $apiKey, string $userId, array $myIps = [])
     {
         $this->aipdbApiKey = $apiKey;
+        $this->aipdbUserId = $userId;
         $this->selfIps = $myIps;
-        $this->timeout = $timeout;
-    }
-
-    /**
-     * Sets the cURL timeout (apply then to any API request). Overwrites the value passed in 
-     * constructor, useful when performing multiple queries with same handler but different timeout.
-     * 
-     * @access public
-     * @param int     $timeout    The maximum number of milliseconds to allow internal cURL functions 
-     *                            to execute.
-     * 
-     * @return void
-     */
-    public function setTimeout(int $timeout): void
-    {
-        $this->timeout = $timeout;
     }
 
     /**
      * Get the current configuration in a indexed array
      * 
-     * @access public
-     * 
+     * @access public 
      * @return array
      */
-    public function getConfig(): array
+    public function getConfig()
     {
         return array(
+            'userId'  => $this->aipdbUserId,
             'apiKey'  => $this->aipdbApiKey,
             'selfIps' => $this->selfIps,
-            'timeout' => $this->timeout, 
+            // TODO  default report cat 
         );
     }
 
+    /**
+     * Get a new instance of ApiHandler with config stored in a Json file
+     * 
+     * @access public 
+     * @static
+     * @param string    $configPath     The configuration file path
+     * 
+     * @return \Kristuff\AbuseIPDB\ApiHandler
+     * @throws \InvalidArgumentException                        If the given file does not exist
+     * @throws \Kristuff\AbuseIPDB\InvalidPermissionException   If the given file is not readable 
+     */
+    public static function fromConfigFile(string $configPath)
+    {
+
+        // check file exists
+        if (!file_exists($configPath) || !is_file($configPath)){
+            throw new \InvalidArgumentException('The file [' . $configPath . '] does not exist.');
+        }
+
+        // check file is readable
+        if (!is_readable($configPath)){
+            throw new InvalidPermissionException('The file [' . $configPath . '] is not readable.');
+        }
+
+        $keyConfig = self::loadJsonFile($configPath);
+        $selfIps = [];
+        
+        // Look for other optional config files in the same directory 
+        $selfIpsConfigPath = pathinfo($configPath, PATHINFO_DIRNAME) . DIRECTORY_SEPARATOR . 'self_ips.json';
+        if (file_exists($selfIpsConfigPath)){
+            $selfIps = self::loadJsonFile($selfIpsConfigPath)->self_ips;
+        }
+
+        $app = new self($keyConfig->api_key, $keyConfig->user_id, $selfIps);
+        
+        return $app;
+    }
+
     /**
      * Performs a 'report' api request
      * 
@@ -116,14 +124,14 @@ class ApiHandler extends ApiBase
      * 
      * @access public
      * @param string    $ip             The ip to report
-     * @param string    $categories     The report category(es)
+     * @param string    $categories     The report categories
      * @param string    $message        The report message
+     * @param bool      $returnArray    True to return an indexed array instead of object. Default is false. 
      *
-     * @return ApiResponse
-     * @throws \RuntimeException
+     * @return object|array
      * @throws \InvalidArgumentException
      */
-    public function report(string $ip, string $categories, string $message): ApiResponse
+    public function report(string $ip = '', string $categories = '', string $message = '', bool $returnArray = false)
     {
          // ip must be set
         if (empty($ip)){
@@ -132,27 +140,29 @@ class ApiHandler extends ApiBase
 
         // categories must be set
         if (empty($categories)){
-            throw new \InvalidArgumentException('Categories list was empty');
+            throw new \InvalidArgumentException('categories list was empty');
         }
 
         // message must be set
-        if (empty($message)){
-            throw new \InvalidArgumentException('Report message was empty');
+          if (empty($message)){
+            throw new \InvalidArgumentException('report message was empty');
         }
 
         // validates categories, clean message 
         $cats = $this->validateReportCategories($categories);
-        $msg  = $this->cleanMessage($message);
+        $msg = $this->cleanMessage($message);
 
         // AbuseIPDB request
-        return $this->apiRequest(
+        $response = $this->apiRequest(
             'report', [
-                'ip'            => $ip,
-                'categories'    => $cats,
-                'comment'       => $msg
+                'ip' => $ip,
+                'categories' => $cats,
+                'comment' => $msg
             ],
             'POST'
         );
+
+        return json_decode($response, $returnArray);
     }
 
     /**
@@ -183,14 +193,15 @@ class ApiHandler extends ApiBase
      *   }
      *  
      * @access public
-     * @param string    $filePath       The CSV file path. Could be an absolute or relative path.
+     * @param string    $ip             The ip to report
+     * @param string    $categories     The report categories
+     * @param string    $message        The report message
+     * @param bool      $returnArray    True to return an indexed array instead of object. Default is false. 
      *
-     * @return ApiResponse
-     * @throws \RuntimeException
+     * @return object|array
      * @throws \InvalidArgumentException
-     * @throws InvalidPermissionException
      */
-    public function bulkReport(string $filePath): ApiResponse
+    public function bulkReport(string $filePath, bool $returnArray = false)
     {
         // check file exists
         if (!file_exists($filePath) || !is_file($filePath)){
@@ -202,7 +213,10 @@ class ApiHandler extends ApiBase
             throw new InvalidPermissionException('The file [' . $filePath . '] is not readable.');
         }
 
-        return $this->apiRequest('bulk-report', [], 'POST', $filePath);
+        // AbuseIPDB request
+        $response = $this->apiRequest('bulk-report', [], 'POST', $filePath);
+
+        return json_decode($response, $returnArray);
     }
 
     /**
@@ -217,20 +231,26 @@ class ApiHandler extends ApiBase
      *    }
      * 
      * @access public
-     * @param string    $ip             The IP to clear reports
+     * @param string    $ip             The ip to check
+     * @param bool      $returnArray    True to return an indexed array instead of object. Default is false. 
      * 
-     * @return ApiResponse
-     * @throws \RuntimeException
+     * @return object|array
      * @throws \InvalidArgumentException    When ip value was not set. 
      */
-    public function clearAddress(string $ip): ApiResponse
+    public function clear(string $ip = null, bool $returnArray = false)
     {
         // ip must be set
         if (empty($ip)){
-            throw new \InvalidArgumentException('IP argument must be set.');
+            throw new \InvalidArgumentException('ip argument must be set (null given)');
         }
 
-        return $this->apiRequest('clear-address',  ['ipAddress' => $ip ], "DELETE") ;
+        // minimal data
+        $data = [
+            'ipAddress'     => $ip, 
+        ];
+
+        $response = $this->apiRequest('clear-address', $data, "DELETE") ;
+        return json_decode($response, $returnArray);
     }
 
     /**
@@ -238,29 +258,29 @@ class ApiHandler extends ApiBase
      * 
      * @access public
      * @param string    $ip             The ip to check
-     * @param int       $maxAgeInDays   Max age in days. Default is 30.
-     * @param bool      $verbose        True to get the full response (last reports and countryName). Default is false
+     * @param int       $maxAge         Max age in days
+     * @param bool      $verbose        True to get the full response. Default is false
+     * @param bool      $returnArray    True to return an indexed array instead of object. Default is false. 
      * 
-     * @return ApiResponse
-     * @throws \RuntimeException
+     * @return object|array
      * @throws \InvalidArgumentException    when maxAge is less than 1 or greater than 365, or when ip value was not set. 
      */
-    public function check(string $ip, int $maxAgeInDays = 30, bool $verbose = false): ApiResponse
+    public function check(string $ip = null, int $maxAge = 30, bool $verbose = false, bool $returnArray = false)
     {
         // max age must be less or equal to 365
-        if ( $maxAgeInDays > 365 || $maxAgeInDays < 1 ){
-            throw new \InvalidArgumentException('maxAgeInDays must be between 1 and 365.');
+        if ($maxAge > 365 || $maxAge < 1){
+            throw new \InvalidArgumentException('maxAge must be at least 1 and less than 365 (' . $maxAge . ' was given)');
         }
 
         // ip must be set
         if (empty($ip)){
-            throw new \InvalidArgumentException('ip argument must be set (empty value given)');
+            throw new \InvalidArgumentException('ip argument must be set (null given)');
         }
 
         // minimal data
         $data = [
             'ipAddress'     => $ip, 
-            'maxAgeInDays'  => $maxAgeInDays,  
+            'maxAgeInDays'  => $maxAge,  
         ];
 
         // option
@@ -268,7 +288,9 @@ class ApiHandler extends ApiBase
            $data['verbose'] = true;
         }
 
-        return $this->apiRequest('check', $data, 'GET') ;
+        $response = $this->apiRequest('check', $data, 'GET') ;
+
+        return json_decode($response, $returnArray);
     }
 
     /**
@@ -308,56 +330,57 @@ class ApiHandler extends ApiBase
      * 
      * @access public
      * @param string    $network        The network to check
-     * @param int       $maxAgeInDays   The Max age in days, must 
+     * @param int       $maxAge         Max age in days
+     * @param bool      $returnArray    True to return an indexed array instead of object. Default is false. 
      * 
-     * @return ApiResponse
-     * @throws \RuntimeException
-     * @throws \InvalidArgumentException    when $maxAgeInDays is less than 1 or greater than 365, or when $network value was not set. 
+     * @return object|array
+     * @throws \InvalidArgumentException    when maxAge is less than 1 or greater than 365, or when network value was not set. 
      */
-    public function checkBlock(string $network, int $maxAgeInDays = 30): ApiResponse
+    public function checkBlock(string $network = null, int $maxAge = 30, bool $returnArray = false)
     {
-        // max age must be between 1 and 365
-        if ($maxAgeInDays > 365 || $maxAgeInDays < 1){
-            throw new \InvalidArgumentException('maxAgeInDays must be between 1 and 365 (' . $maxAgeInDays . ' was given)');
+        // max age must be less or equal to 365
+        if ($maxAge > 365 || $maxAge < 1){
+            throw new \InvalidArgumentException('maxAge must be at least 1 and less than 365 (' . $maxAge . ' was given)');
         }
 
         // ip must be set
         if (empty($network)){
-            throw new \InvalidArgumentException('network argument must be set (empty value given)');
+            throw new \InvalidArgumentException('network argument must be set (null given)');
         }
 
         // minimal data
         $data = [
             'network'       => $network, 
-            'maxAgeInDays'  => $maxAgeInDays,  
+            'maxAgeInDays'  => $maxAge,  
         ];
 
-        return $this->apiRequest('check-block', $data, 'GET');
+        $response = $this->apiRequest('check-block', $data, 'GET') ;
+
+        return json_decode($response, $returnArray);
     }
 
     /**
      * Perform a 'blacklist' api request
      * 
      * @access public
-     * @param int       $limit              The blacklist limit. Default is 10000 (the api default limit) 
-     * @param bool      $plainText          True to get the response in plaintext list. Default is false
-     * @param int       $confidenceMinimum  The abuse confidence score minimum (subscribers feature). Default is 100.
-     *                                      The confidence minimum must be between 25 and 100.
-     *                                      This parameter is a subscriber feature (not honored otherwise).
+     * @param int       $limit          The blacklist limit. Default is TODO (the api default limit) 
+     * @param bool      $plainText      True to get the response in plain text list. Default is false
+     * @param bool      $returnArray    True to return an indexed array instead of object (when $plainText is set to false). Default is false. 
      * 
-     * @return ApiResponse
-     * @throws \RuntimeException
-     * @throws \InvalidArgumentException    When maxAge is not a numeric value, when $limit is less than 1. 
+     * @return object|array|string
+     * @throws \InvalidArgumentException    When maxAge is not a numeric value, when maxAge is less than 1 or 
+     *                                      greater than 365, or when ip value was not set. 
      */
-    public function blacklist(int $limit = 10000, bool $plainText = false, int $confidenceMinimum = 100): ApiResponse
+    public function getBlacklist(int $limit = 10000, bool $plainText = false, bool $returnArray = false)
     {
+
         if ($limit < 1){
             throw new \InvalidArgumentException('limit must be at least 1 (' . $limit . ' was given)');
         }
 
         // minimal data
         $data = [
-            'confidenceMinimum' => $confidenceMinimum, 
+            'confidenceMinimum' => 100, // The abuseConfidenceScore parameter is a subscriber feature. 
             'limit'             => $limit,
         ];
 
@@ -367,7 +390,13 @@ class ApiHandler extends ApiBase
             $data['plaintext'] = $plainText;
         }
 
-        return $this->apiRequest('blacklist', $data, 'GET');
+        $response = $this->apiRequest('blacklist', $data, 'GET');
+
+        if ($plainText){
+            return $response;
+        } 
+       
+        return json_decode($response, $returnArray);
     }
   
     /**
@@ -377,18 +406,17 @@ class ApiHandler extends ApiBase
      * @param string    $path           The api end path 
      * @param array     $data           The request data 
      * @param string    $method         The request method. Default is 'GET' 
-     * @param string    $csvFilePath    The file path for csv file. When not empty, $data parameter is ignored and in place,
+     * @param bool      $csvFilePath    The file path for csv file. When not empty, $data parameter is ignored and in place,
      *                                  the content of the given file if passed as csv. Default is empty string. 
      * 
-     * @return ApiResponse
+     * @return mixed
      * @throws \RuntimeException
      */
-    protected function apiRequest(string $path, array $data, string $method = 'GET', string $csvFilePath = ''): ApiResponse
+    protected function apiRequest(string $path, array $data, string $method = 'GET', string $csvFilePath = '') 
     {
-        $curlErrorNumber = -1;                   // will be used later to check curl execution
-        $curlErrorMessage = '';
-        $url = $this->aipdbApiEndpoint . $path;  // api url
-        
+        // set api url
+        $url = $this->aipdbApiEndpoint . $path; 
+
         // set the wanted format, JSON (required to prevent having full html page on error)
         // and the AbuseIPDB API Key as a header
         $headers = [
@@ -410,42 +438,24 @@ class ApiHandler extends ApiBase
         if ($method == 'POST') {
             $this->setCurlOption($ch, CURLOPT_POST, true);
             $this->setCurlOption($ch, CURLOPT_POSTFIELDS, $data);
-        
         } else {
             $this->setCurlOption($ch, CURLOPT_CUSTOMREQUEST, $method);
             $url .= '?' . http_build_query($data);
         }
 
-        // set url and options
+        // set the url to call
         $this->setCurlOption($ch, CURLOPT_URL, $url);
         $this->setCurlOption($ch, CURLOPT_RETURNTRANSFER, 1); 
         $this->setCurlOption($ch, CURLOPT_HTTPHEADER, $headers);
-        
-        /**
-         * set timeout  
-         * 
-         * @see https://curl.se/libcurl/c/CURLOPT_TIMEOUT_MS.html
-         * @see https://curl.se/libcurl/c/CURLOPT_CONNECTTIMEOUT_MS.html
-         *  If libcurl is built to use the standard system name resolver, that portion of the transfer 
-         *  will still use full-second resolution for timeouts with a minimum timeout allowed of one second. 
-         *  In unix-like systems, this might cause signals to be used unless CURLOPT_NOSIGNAL is set. 
-         */
-        $this->setCurlOption($ch, CURLOPT_NOSIGNAL, 1);
-        $this->setCurlOption($ch, CURLOPT_TIMEOUT_MS, $this->timeout);
-
+    
         // execute curl call
         $result = curl_exec($ch);
-        $curlErrorNumber = curl_errno($ch);
-        $curlErrorMessage = curl_error($ch);
-
+    
         // close connection
         curl_close($ch);
-
-        if ($curlErrorNumber !== 0){
-            throw new \RuntimeException($curlErrorMessage);
-        }
-
-        return new ApiResponse($result !== false ? $result : '');
+  
+        // return response as is (JSON or plain text)
+        return $result;
     }
 
     /** 
@@ -458,7 +468,7 @@ class ApiHandler extends ApiBase
      *  
 	 * @return string
      */
-    public function cleanMessage(string $message): string
+    protected function cleanMessage(string $message)
     {
         // Remove backslashes
         $message = str_replace('\\', '', $message);
@@ -469,10 +479,44 @@ class ApiHandler extends ApiBase
         }
 
         // If we're reporting spam, further munge any email addresses in the report
-        $emailPattern   = "/\b[A-Z0-9!#$%&'*`\/?^{|}~=+_.-]+@[A-Z0-9.-]+\b/i";
-        $message        = preg_replace($emailPattern, "*", $message);
+        $emailPattern = "/[^@\s]*@[^@\s]*\.[^@\s]*/";
+        $message = preg_replace($emailPattern, "*", $message);
         
         // Make sure message is less 1024 chars
         return substr($message, 0, 1024);
     }
+
+    /** 
+     * Load and returns decoded Json from given file  
+     *
+     * @access public
+     * @static
+	 * @param string    $filePath       The file's full path
+	 * @param bool      $throwError     Throw error on true or silent process. Default is true
+     *  
+	 * @return object|null 
+     * @throws \Exception
+     * @throws \LogicException
+     */
+    protected static function loadJsonFile(string $filePath, bool $throwError = true)
+    {
+        // check file exists
+        if (!file_exists($filePath) || !is_file($filePath)){
+           if ($throwError) {
+                throw new \Exception('Config file not found');
+           }
+           return null;  
+        }
+
+        // get and parse content
+        $content = utf8_encode(file_get_contents($filePath));
+        $json    = json_decode($content);
+
+        // check for errors
+        if ($json == null && json_last_error() != JSON_ERROR_NONE && $throwError) {
+            throw new \LogicException(sprintf("Failed to parse config file Error: '%s'", json_last_error_msg()));
+        }
+
+        return $json;        
+    }
 }

lib/ApiResponse.php

@@ -1,137 +0,0 @@
-<?php declare(strict_types=1);
-
-/**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
- *
- * (c) Kristuff <kristuff@kristuff.fr>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
- */
-
-namespace Kristuff\AbuseIPDB;
-
-/**
- * Class ApiResponse
- * 
- */
-class ApiResponse
-{
-    /**
-     * 
-     * @access protected
-     * @var string 
-     */
-    protected $curlResponse; 
-
-    /**
-     * 
-     * @access protected
-     * @var object 
-     */
-    protected $decodedResponse; 
-
-    /**
-     * Constructor
-     * 
-     * @access public
-     * @param string     $plaintext      AbuseIPDB response in plaintext 
-     * 
-     */
-    public function __construct(?string $plaintext = null)
-    {
-        $this->curlResponse = $plaintext;
-        $this->decodedResponse = !empty($plaintext) ? json_decode($plaintext, false) : null;
-    }
-
-    /**
-     * Get response as array. May return null
-     * 
-     * @access public
-     * 
-     * @return array|null
-     */
-    public function getArray(): ?array
-    {
-        return json_decode($this->curlResponse, true);
-    }
-
-    /**
-     * Get response as object. May return null
-     * 
-     * @access public
-     * 
-     * @return \stdClass|null
-     */
-    public function getObject(): ?\stdClass
-    {
-        return $this->decodedResponse;
-    }
-
-    /**
-     * Get response as plaintext. May return null
-     * 
-     * @access public
-     * 
-     * @return string|null
-     */
-    public function getPlaintext(): ?string
-    {
-        return $this->curlResponse;
-    }
-    
-    /**
-     * Get whether the response contains error(s)
-     * 
-     * @access public
-     * 
-     * @return bool
-     */
-    public function hasError(): bool
-    {
-        return count($this->errors()) > 0;
-    }
-
-    /**
-     * Get an array of errors (object) contained is response 
-     * 
-     * @access public
-     * 
-     * @return array
-     */
-    public function errors(): array
-    {
-        return ($this->decodedResponse && property_exists($this->decodedResponse, 'errors')) ? $this->decodedResponse->errors : [];
-    }
-
-    /**
-     * Get an internal error message in an ApiResponse object
-     * 
-     * @access public
-     * @static
-     * @param string    $message        The error message
-     *
-     * @return ApiResponse
-     */
-    public static function createErrorResponse(string $message): ApiResponse
-    {
-        $response = [
-            "errors" => [
-                [
-                    "title"  => "Internal Error",
-                    "detail" => $message
-                ]
-            ]
-        ];
-
-        return new ApiResponse(json_encode($response));
-    }
-}

lib/CurlTrait.php

@@ -1,20 +1,21 @@
-<?php declare(strict_types=1);
+<?php
 
 /**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
+ *     _    _                    ___ ____  ____  ____
+ *    / \  | |__  _   _ ___  ___|_ _|  _ \|  _ \| __ )
+ *   / _ \ | '_ \| | | / __|/ _ \| || |_) | | | |  _ \
+ *  / ___ \| |_) | |_| \__ \  __/| ||  __/| |_| | |_) |
+ * /_/   \_\_.__/ \__,_|___/\___|___|_|   |____/|____/
  *
- * (c) Kristuff <kristuff@kristuff.fr>
+ * This file is part of Kristuff\AbsuseIPDB.
+ *
+ * (c) Kristuff <contact@kristuff.fr>
  *
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
+ * @version    0.9.7
+ * @copyright  2020-2021 Kristuff
  */
 
 namespace Kristuff\AbuseIPDB;
@@ -35,8 +36,7 @@ trait CurlTrait
      * @return void
      * @throws \RuntimeException
      */
-    protected function setCurlOption($ch, int $option, $value): void
-    {
+    protected function setCurlOption($ch,int $option,$value){
         if(!curl_setopt($ch,$option,$value)){
             throw new \RuntimeException('curl_setopt failed! '.curl_error($ch));
         }

lib/InvalidPermissionException.php

@@ -1,20 +1,21 @@
-<?php declare(strict_types=1);
+<?php
 
 /**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
+ *     _    _                    ___ ____  ____  ____
+ *    / \  | |__  _   _ ___  ___|_ _|  _ \|  _ \| __ )
+ *   / _ \ | '_ \| | | / __|/ _ \| || |_) | | | |  _ \
+ *  / ___ \| |_) | |_| \__ \  __/| ||  __/| |_| | |_) |
+ * /_/   \_\_.__/ \__,_|___/\___|___|_|   |____/|____/
  *
- * (c) Kristuff <kristuff@kristuff.fr>
+ * This file is part of Kristuff\AbsuseIPDB.
+ *
+ * (c) Kristuff <contact@kristuff.fr>
  *
  * For the full copyright and license information, please view the LICENSE
  * file that was distributed with this source code.
  *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
+ * @version    0.9.7
+ * @copyright  2020-2021 Kristuff
  */
 
 namespace Kristuff\AbuseIPDB;

lib/QuietApiHandler.php

@@ -1,141 +0,0 @@
-<?php declare(strict_types=1);
-
-/**
- *       _                 ___ ___ ___  ___
- *  __ _| |__ _  _ ___ ___|_ _| _ \   \| _ )
- * / _` | '_ \ || (_-</ -_)| ||  _/ |) | _ \
- * \__,_|_.__/\_,_/__/\___|___|_| |___/|___/
- * 
- * This file is part of Kristuff\AbuseIPDB.
- *
- * (c) Kristuff <kristuff@kristuff.fr>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- *
- * @version    1.1
- * @copyright  2020-2022 Kristuff
- */
-
-namespace Kristuff\AbuseIPDB;
-
-/**
- * Class QuietApiHandler
- * 
- * Overwrite ApiHandler with Exception handling
- * Instead of Exception, all methods return an ApiResponse that may 
- * contains errors from the AbuseIPDB API, or internal errors  
- */
-class QuietApiHandler extends ApiHandler
-{
-    /**
-     * Performs a 'report' api request, with Exception handling
-     * 
-     * @access public
-     * @param string    $ip             The ip to report
-     * @param string    $categories     The report category(es)
-     * @param string    $message        The report message
-     *
-     * @return ApiResponse
-     */
-    public function report(string $ip, string $categories, string $message): ApiResponse
-    {
-        try {
-            return parent::report($ip,$categories,$message);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-
-    /**
-     * Performs a 'bulk-report' api request, with Exception handling
-     * 
-     * @access public
-     * @param string    $filePath       The CSV file path. Could be an absolute or relative path.
-     *
-     * @return ApiResponse
-     */
-    public function bulkReport(string $filePath): ApiResponse
-    {
-        try {
-            return parent::bulkReport($filePath);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-
-    /**
-     * Perform a 'clear-address' api request, with Exception handling
-     * 
-     * @access public
-     * @param string    $ip             The IP to clear reports
-     * 
-     * @return ApiResponse
-     */
-    public function clearAddress(string $ip): ApiResponse
-    {
-        try {
-            return parent::clearAddress($ip);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-
-    /**
-     * Perform a 'check' api request, with Exception handling
-     * 
-     * @access public
-     * @param string    $ip             The ip to check
-     * @param int       $maxAgeInDays   Max age in days. Default is 30.
-     * @param bool      $verbose        True to get the full response (last reports and countryName). Default is false
-     * 
-     * @return ApiResponse
-     */
-    public function check(string $ip, int $maxAgeInDays = 30, bool $verbose = false): ApiResponse
-    {
-        try {
-            return parent::check($ip, $maxAgeInDays, $verbose);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-
-    /**
-     * Perform a 'check-block' api request, with Exception handling
-     * 
-     * @access public
-     * @param string    $network        The network to check
-     * @param int       $maxAgeInDays   The Max age in days, must 
-     * 
-     * @return ApiResponse
-     */
-    public function checkBlock(string $network, int $maxAgeInDays = 30): ApiResponse
-    {
-        try {
-            return parent::checkBlock($network, $maxAgeInDays);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-
-    /**
-     * Perform a 'blacklist' api request, with Exception handling
-     * 
-     * @access public
-     * @param int       $limit              The blacklist limit. Default is 10000 (the api default limit) 
-     * @param bool      $plainText          True to get the response in plaintext list. Default is false
-     * @param int       $confidenceMinimum  The abuse confidence score minimum (subscribers feature). Default is 100.
-     *                                      The confidence minimum must be between 25 and 100.
-     *                                      This parameter is a subscriber feature (not honored otherwise).
-     * 
-     * @return ApiResponse
-     */
-    public function blacklist(int $limit = 10000, bool $plainText = false, int $confidenceMinimum = 100): ApiResponse
-    {
-        try {
-            return parent::blacklist($limit, $plainText, $confidenceMinimum);
-        } catch (\Exception $e) {
-            return ApiResponse::createErrorResponse($e->getMessage());
-        }
-    }
-}