modern handling of http request

src/Database/Mysql/Connection.php

@@ -4,6 +4,8 @@ use SokoWeb\Interfaces\Database\IConnection;
 use SokoWeb\Interfaces\Database\IResultSet;
 use SokoWeb\Interfaces\Database\IStatement;
 use mysqli;
+use DateTime;
+use DateTimeZone;
 
 class Connection implements IConnection
 {
@@ -109,5 +111,16 @@ class Connection implements IConnection
         mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
         $this->connection = new mysqli($this->host, $this->user, $this->password, $this->db, $this->port, $this->socket);
         $this->connection->set_charset('utf8mb4');
+        $this->connection->query('SET time_zone = \'' . $this->getTimeZone() . '\'');
+    }
+
+    private function getTimeZone(): string {
+        $tz = new DateTimeZone(date_default_timezone_get());
+        $offset = $tz->getOffset(new DateTime('now', new DateTimeZone('UTC')));
+
+        $hours = intdiv($offset, 3600);
+        $minutes = abs(($offset % 3600) / 60);
+
+        return sprintf("%+03d:%02d", $hours, $minutes);
     }
 }

src/Http/Request.php

@@ -7,13 +7,15 @@ class Request implements IRequest
 {
     private string $url;
 
-    private int $method;
+    private ?string $method = null;
 
     private string $query = '';
 
+    private ?string $body = null;
+
     private array $headers = [];
 
-    public function __construct(string $url = '', int $method = self::HTTP_GET)
+    public function __construct(string $url = '', ?string $method = null)
     {
         $this->url = $url;
         $this->method = $method;
@@ -24,7 +26,7 @@ class Request implements IRequest
         $this->url = $url;
     }
 
-    public function setMethod(int $method): void
+    public function setMethod(string $method): void
     {
         $this->method = $method;
     }
@@ -38,6 +40,11 @@ class Request implements IRequest
         }
     }
 
+    public function setBody(string $body): void
+    {
+        $this->body = $body;
+    }
+
     public function setHeaders(array $headers): void
     {
         $this->headers = array_merge($this->headers, $headers);
@@ -47,13 +54,20 @@ class Request implements IRequest
     {
         $ch = curl_init();
 
-        if ($this->method === self::HTTP_POST) {
-            $url = $this->url;
-
-            curl_setopt($ch, CURLOPT_POST, 1);
-            curl_setopt($ch, CURLOPT_POSTFIELDS, $this->query);
-        } else {
         $url = $this->url . '?' . $this->query;
+
+        if ($this->body !== null) {
+            if ($this->method === null) {
+                $this->method = self::HTTP_POST;
+            }
+
+            if ($this->method === self::HTTP_POST) {
+                curl_setopt($ch, CURLOPT_POST, 1);
+            } else {
+                curl_setopt($ch, CURLOPT_CUSTOMREQUEST, $this->method);
+            }
+
+            curl_setopt($ch, CURLOPT_POSTFIELDS, $this->body);
         }
 
         curl_setopt($ch, CURLOPT_URL, $url);

src/Interfaces/Http/IRequest.php

@@ -2,16 +2,24 @@
 
 interface IRequest
 {
-    const HTTP_GET = 0;
+    const HTTP_GET = 'GET';
 
-    const HTTP_POST = 1;
+    const HTTP_POST = 'POST';
+
+    const HTTP_PUT = 'PUT';
+
+    const HTTP_PATCH = 'PATCH';
+
+    const HTTP_DELETE = 'DELETE';
 
     public function setUrl(string $url): void;
 
-    public function setMethod(int $method): void;
+    public function setMethod(string $method): void;
 
     public function setQuery($query): void;
 
+    public function setBody(string $body): void;
+
     public function setHeaders(array $headers): void;
 
     public function send(): IResponse;

src/Mailing/Mail.php

@@ -44,8 +44,10 @@ class Mail
         if (!empty($_ENV['MAIL_HOST'])) {
             $mailer->Mailer = 'smtp';
             $mailer->Host = $_ENV['MAIL_HOST'];
-            $mailer->Port = !empty($_ENV['MAIL_PORT']) ? $_ENV['MAIL_PORT'] : 25;
+            $mailer->Port = !empty($_ENV['MAIL_PORT']) ? $_ENV['MAIL_PORT'] : 587;
-            $mailer->SMTPSecure = !empty($_ENV['MAIL_SECURE']) ? $_ENV['MAIL_SECURE'] : '';
+
+            $secureMaping = ['none' => '', 'starttls' => PHPMailer::ENCRYPTION_STARTTLS, 'smtps' => PHPMailer::ENCRYPTION_SMTPS];
+            $mailer->SMTPSecure = !empty($_ENV['MAIL_SECURE']) ? $secureMaping[$_ENV['MAIL_SECURE']] : '';
 
             if (!empty($_ENV['MAIL_USER'])) {
                 $mailer->SMTPAuth = true;

src/Response/HttpResponse.php

@@ -168,7 +168,10 @@ class HttpResponse
     private function redirectToLogin(): void
     {
         $this->request->session()->set('redirect_after_login', $this->rawUrl);
-        $response = new Redirect($this->routeCollection->getRoute($this->appConfig['loginRouteId'])->generateLink(), IRedirect::TEMPORARY);
+        $response = new Redirect(
+            $this->routeCollection->getRoute($this->appConfig['loginRouteId'])
+                ->generateLink(['redirect_after_login' => $this->rawUrl]),
+            IRedirect::TEMPORARY);
         header('Location: ' . $this->getRedirectUrl($response), true, $response->getHttpCode());
     }
 

src/Routing/Route.php

@@ -34,7 +34,7 @@ class Route implements IRoute
         foreach ($this->pattern as $fragment) {
             if (preg_match('/^{(\\w+)(\\?)?}$/', $fragment, $matches) === 1) {
                 if (isset($parameters[$matches[1]])) {
-                    $link[] = $parameters[$matches[1]];
+                    $link[] = rawurlencode($parameters[$matches[1]]);
                     unset($parameters[$matches[1]]);
                 } elseif (!isset($matches[2])) {//TODO: why? parameter not found but not optional
                     $link[] = $fragment;
@@ -53,7 +53,7 @@ class Route implements IRoute
             $queryParams[$key] = $value;
         }
 
-        $query = count($queryParams) > 0 ? '?' . http_build_query($queryParams) : '';
+        $query = count($queryParams) > 0 ? '?' . http_build_query($queryParams, encoding_type: PHP_QUERY_RFC3986) : '';
 
         return '/' . implode('/', $link) . $query;
     }
@@ -64,7 +64,7 @@ class Route implements IRoute
 
         foreach ($path as $i => $fragment) {
             if (preg_match('/^{(\\w+)(?:\\?)?}$/', $this->pattern[$i], $matches) === 1) {
-                $parameters[$matches[1]] = $fragment;
+                $parameters[$matches[1]] = rawurldecode($fragment);
             } elseif ($fragment != $this->pattern[$i]) {
                 return null;
             }