refactor: WebProcessor by extracting context building into a separate method

- Moved the context construction logic from the process() method into a new private method buildContext().
- This enhances code readability and adheres to the Single Responsibility Principle (SRP) by separating the concerns of processing and context building.
- Improves maintainability by centralizing the context logic, making future modifications easier.

Author: walmir-silva

src/Processor/EncryptionProcessor.php

@@ -19,6 +19,11 @@ public function process(ImmutableValue $record): ImmutableValue
     {
         $encryptedMessage = $this->encryptor->encrypt($record->message);
 
-        return new LogRecord($record->level, $encryptedMessage, $record->context, $record->datetime);
+        return new LogRecord(
+            $record->level,
+            $encryptedMessage,
+            $record->context,
+            $record->datetime
+        );
     }
 }

src/Processor/Metric/CpuUsageProcessor.php

@@ -10,9 +10,6 @@
 
 class CpuUsageProcessor extends AbstractProcessor
 {
-    /**
-     * @param LogRecord $record
-     */
     public function process(ImmutableValue $record): ImmutableValue
     {
         $cpuUsage = sys_getloadavg()[0];

src/Processor/Metric/ExecutionTimeProcessor.php

@@ -14,9 +14,6 @@ public function __construct(private float $threshold = 1000)
     {
     }
 
-    /**falta cibnfi
-     * @param LogRecord $record
-     */
     public function process(ImmutableValue $record): ImmutableValue
     {
         $executionTime = (microtime(true) - $_SERVER['REQUEST_TIME_FLOAT']) * 1000; // Convert to milliseconds

src/Processor/Metric/MemoryUsageProcessor.php

@@ -8,9 +8,6 @@
 
 class MemoryUsageProcessor extends AbstractProcessor
 {
-    /**
-     * @param LogRecord $record
-     */
     public function process(ImmutableValue $record): ImmutableValue
     {
         $memoryUsage = memory_get_usage(true);

src/Processor/MetricsProcessor.php

@@ -10,17 +10,13 @@
 
 class MetricsProcessor implements LogProcessor
 {
-    /** @var LogProcessor[] */
     private array $processors;
 
     public function __construct(array $processors)
     {
         $this->processors = $processors;
     }
 
-    /**
-     * @param LogRecord $record
-     */
     public function process(ImmutableValue $record): ImmutableValue
     {
         foreach ($this->processors as $processor) {

src/Processor/WebProcessor.php

@@ -4,21 +4,19 @@
 
 use KaririCode\Contract\ImmutableValue;
 use KaririCode\Logging\LogRecord;
+use KaririCode\Logging\Util\Http\Contract\HttpRequest;
+use KaririCode\Logging\Util\Http\ServerHttpRequest;
 
 class WebProcessor extends AbstractProcessor
 {
+    public function __construct(
+        private HttpRequest $request = new ServerHttpRequest()
+    ) {
+    }
+
     public function process(ImmutableValue $record): ImmutableValue
     {
-        $server = $_SERVER;
-        $context = array_merge($record->context, [
-            'url' => ($server['HTTPS'] ?? 'off') === 'on' ? 'https://' : 'http://' .
-                     ($server['HTTP_HOST'] ?? 'localhost') .
-                     ($server['REQUEST_URI'] ?? '/'),
-            'ip' => $server['REMOTE_ADDR'] ?? null,
-            'http_method' => $server['REQUEST_METHOD'] ?? null,
-            'server' => $server['SERVER_NAME'] ?? null,
-            'referrer' => $server['HTTP_REFERER'] ?? null,
-        ]);
+        $context = $this->buildContext($record->context);
 
         return new LogRecord(
             $record->level,
@@ -28,4 +26,15 @@ public function process(ImmutableValue $record): ImmutableValue
             $record->extra
         );
     }
+
+    private function buildContext(array $existingContext): array
+    {
+        return array_merge($existingContext, [
+            'url' => $this->request->getUrl(),
+            'ip' => $this->request->getIp(),
+            'http_method' => $this->request->getMethod(),
+            'server' => $this->request->getServerName(),
+            'referrer' => $this->request->getReferrer(),
+        ]);
+    }
 }

src/Util/Http/HttpRequest.php

@@ -0,0 +1,14 @@
+<?php
+
+declare(strict_types=1);
+
+namespace KaririCode\Logging\Util\Http\Contract;
+
+interface HttpRequest
+{
+    public function getUrl(): string;
+    public function getIp(): ?string;
+    public function getMethod(): string;
+    public function getServerName(): ?string;
+    public function getReferrer(): ?string;
+}

src/Util/Http/ServerHttpRequest.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace KaririCode\Logging\Util\Http;
+
+use KaririCode\Logging\Util\Http\Contract\HttpRequest;
+
+class ServerHttpRequest implements HttpRequest
+{
+    private array $serverParams;
+
+    public function __construct(array $serverParams = [])
+    {
+        $this->serverParams = array_merge($_SERVER, $serverParams);
+    }
+
+    public function getUrl(): string
+    {
+        $scheme = ($this->serverParams['HTTPS'] ?? 'off') === 'on' ? 'https://' : 'http://';
+        $host = $this->serverParams['HTTP_HOST'] ?? 'localhost';
+        $uri = $this->serverParams['REQUEST_URI'] ?? '/';
+        return $scheme . $host . $uri;
+    }
+
+    public function getIp(): ?string
+    {
+        return $this->serverParams['REMOTE_ADDR'] ?? null;
+    }
+
+    public function getMethod(): string
+    {
+        return $this->serverParams['REQUEST_METHOD'] ?? 'GET';
+    }
+
+    public function getServerName(): ?string
+    {
+        return $this->serverParams['SERVER_NAME'] ?? null;
+    }
+
+    public function getReferrer(): ?string
+    {
+        return $this->serverParams['HTTP_REFERER'] ?? null;
+    }
+}