From cae85a6a0c4afb31f26b1b74a59bf6f04230a5e1 Mon Sep 17 00:00:00 2001
From: Zankaria <zankaria.auxa@mailu.io>
Date: Thu, 3 Oct 2024 19:57:56 +0200
Subject: [PATCH] http-driver.php: move to Data

---
 inc/{driver => Data/Driver}/http-driver.php | 60 ++++++++++-----------
 inc/context.php                             |  3 +-
 inc/service/captcha-queries.php             |  2 +-
 post.php                                    |  2 +-
 4 files changed, 33 insertions(+), 34 deletions(-)
 rename inc/{driver => Data/Driver}/http-driver.php (65%)

diff --git a/inc/driver/http-driver.php b/inc/Data/Driver/http-driver.php
similarity index 65%
rename from inc/driver/http-driver.php
rename to inc/Data/Driver/http-driver.php
index cfbedfad..1a714963 100644
--- a/inc/driver/http-driver.php
+++ b/inc/Data/Driver/http-driver.php
@@ -1,7 +1,5 @@
 <?php // Honestly this is just a wrapper for cURL. Still useful to mock it and have an OOP API on PHP 7.
-namespace Vichan\Driver;
-
-use RuntimeException;
+namespace Vichan\Data\Driver;
 
 defined('TINYBOARD') or exit;
 
@@ -25,37 +23,37 @@ class HttpDriver {
 	private function resetTowards(string $url, int $timeout): void {
 		curl_reset($this->inner);
 		curl_setopt_array($this->inner, array(
-			CURLOPT_URL => $url,
-			CURLOPT_TIMEOUT => $this->timeout,
-			CURLOPT_USERAGENT => $this->user_agent,
-			CURLOPT_PROTOCOLS => CURLPROTO_HTTP | CURLPROTO_HTTPS,
+			\CURLOPT_URL => $url,
+			\CURLOPT_TIMEOUT => $timeout,
+			\CURLOPT_USERAGENT => $this->user_agent,
+			\CURLOPT_PROTOCOLS => \CURLPROTO_HTTP | \CURLPROTO_HTTPS,
 		));
 	}
 
 	private function setSizeLimit(): void {
 		// Adapted from: https://stackoverflow.com/a/17642638
-		curl_setopt($this->inner, CURLOPT_NOPROGRESS, false);
+		\curl_setopt($this->inner, \CURLOPT_NOPROGRESS, false);
 
-		if (PHP_MAJOR_VERSION >= 8 && PHP_MINOR_VERSION >= 2) {
-			curl_setopt($this->inner, CURLOPT_XFERINFOFUNCTION, function($res, $next_dl, $dl, $next_up, $up) {
+		if (\PHP_MAJOR_VERSION >= 8 && \PHP_MINOR_VERSION >= 2) {
+			\curl_setopt($this->inner, \CURLOPT_XFERINFOFUNCTION, function($res, $next_dl, $dl, $next_up, $up) {
 				return (int)($dl <= $this->max_file_size);
 			});
 		} else {
-			curl_setopt($this->inner, CURLOPT_PROGRESSFUNCTION, function($res, $next_dl, $dl, $next_up, $up) {
+			\curl_setopt($this->inner, \CURLOPT_PROGRESSFUNCTION, function($res, $next_dl, $dl, $next_up, $up) {
 				return (int)($dl <= $this->max_file_size);
 			});
 		}
 	}
 
-	function __construct($timeout, $user_agent, $max_file_size) {
-		$this->inner = curl_init();
+	function __construct(int $timeout, string $user_agent, int $max_file_size) {
+		$this->inner = \curl_init();
 		$this->timeout = $timeout;
 		$this->user_agent = $user_agent;
 		$this->max_file_size = $max_file_size;
 	}
 
 	function __destruct() {
-		curl_close($this->inner);
+		\curl_close($this->inner);
 	}
 
 	/**
@@ -69,18 +67,18 @@ class HttpDriver {
 	 */
 	public function requestGet(string $endpoint, ?array $data, int $timeout = 0): string {
 		if (!empty($data)) {
-			$endpoint .= '?' . http_build_query($data);
+			$endpoint .= '?' . \http_build_query($data);
 		}
 		if ($timeout == 0) {
 			$timeout = $this->timeout;
 		}
 
 		$this->resetTowards($endpoint, $timeout);
-		curl_setopt($this->inner, CURLOPT_RETURNTRANSFER, true);
-		$ret = curl_exec($this->inner);
+		\curl_setopt($this->inner, \CURLOPT_RETURNTRANSFER, true);
+		$ret = \curl_exec($this->inner);
 
 		if ($ret === false) {
-			throw new \RuntimeException(curl_error($this->inner));
+			throw new \RuntimeException(\curl_error($this->inner));
 		}
 		return $ret;
 	}
@@ -100,15 +98,15 @@ class HttpDriver {
 		}
 
 		$this->resetTowards($endpoint, $timeout);
-		curl_setopt($this->inner, CURLOPT_POST, true);
+		\curl_setopt($this->inner, \CURLOPT_POST, true);
 		if (!empty($data)) {
-			curl_setopt($this->inner, CURLOPT_POSTFIELDS, http_build_query($data));
+			\curl_setopt($this->inner, \CURLOPT_POSTFIELDS, \http_build_query($data));
 		}
-		curl_setopt($this->inner, CURLOPT_RETURNTRANSFER, true);
-		$ret = curl_exec($this->inner);
+		\curl_setopt($this->inner, \CURLOPT_RETURNTRANSFER, true);
+		$ret = \curl_exec($this->inner);
 
 		if ($ret === false) {
-			throw new \RuntimeException(curl_error($this->inner));
+			throw new \RuntimeException(\curl_error($this->inner));
 		}
 		return $ret;
 	}
@@ -125,26 +123,26 @@ class HttpDriver {
 	 */
 	public function requestGetInto(string $endpoint, ?array $data, mixed $fd, int $timeout = 0): bool {
 		if (!empty($data)) {
-			$endpoint .= '?' . http_build_query($data);
+			$endpoint .= '?' . \http_build_query($data);
 		}
 		if ($timeout == 0) {
 			$timeout = $this->timeout;
 		}
 
 		$this->resetTowards($endpoint, $timeout);
-		curl_setopt($this->inner, CURLOPT_FAILONERROR, true);
-		curl_setopt($this->inner, CURLOPT_FOLLOWLOCATION, false);
-		curl_setopt($this->inner, CURLOPT_FILE, $fd);
-		curl_setopt($this->inner, CURLOPT_IPRESOLVE, CURL_IPRESOLVE_V4);
+		\curl_setopt($this->inner, \CURLOPT_FAILONERROR, true);
+		\curl_setopt($this->inner, \CURLOPT_FOLLOWLOCATION, false);
+		\curl_setopt($this->inner, \CURLOPT_FILE, $fd);
+		\curl_setopt($this->inner, \CURLOPT_IPRESOLVE, CURL_IPRESOLVE_V4);
 		$this->setSizeLimit();
-		$ret = curl_exec($this->inner);
+		$ret = \curl_exec($this->inner);
 
 		if ($ret === false) {
-			if (curl_errno($this->inner) === CURLE_ABORTED_BY_CALLBACK) {
+			if (\curl_errno($this->inner) === CURLE_ABORTED_BY_CALLBACK) {
 				return false;
 			}
 
-			throw new \RuntimeException(curl_error($this->inner));
+			throw new \RuntimeException(\curl_error($this->inner));
 		}
 		return true;
 	}
diff --git a/inc/context.php b/inc/context.php
index 3f8a75d6..70242502 100644
--- a/inc/context.php
+++ b/inc/context.php
@@ -2,7 +2,8 @@
 namespace Vichan;
 
 use RuntimeException;
-use Vichan\Driver\{HttpDriver, HttpDrivers, Log, LogDrivers};
+use Vichan\Driver\{Log, LogDrivers};
+use Vichan\Data\Driver\{HttpDriver, HttpDrivers};
 use Vichan\Service\HCaptchaQuery;
 use Vichan\Service\NativeCaptchaQuery;
 use Vichan\Service\ReCaptchaQuery;
diff --git a/inc/service/captcha-queries.php b/inc/service/captcha-queries.php
index cd9a1b84..76d7acd8 100644
--- a/inc/service/captcha-queries.php
+++ b/inc/service/captcha-queries.php
@@ -1,7 +1,7 @@
 <?php // Verify captchas server side.
 namespace Vichan\Service;
 
-use Vichan\Driver\HttpDriver;
+use Vichan\Data\Driver\HttpDriver;
 
 defined('TINYBOARD') or exit;
 
diff --git a/post.php b/post.php
index b20fdfc9..bfd41639 100644
--- a/post.php
+++ b/post.php
@@ -6,7 +6,7 @@
 require_once 'inc/bootstrap.php';
 
 use Vichan\{Context, WebDependencyFactory};
-use Vichan\Driver\{HttpDriver, Log};
+use Vichan\Data\Driver\{HttpDriver, Log};
 use Vichan\Service\{RemoteCaptchaQuery, NativeCaptchaQuery};
 use Vichan\Functions\Format;