From 6e34a76a3fbc020f0c4132956c64e33b39411fb1 Mon Sep 17 00:00:00 2001
From: Juan Pablo Vial <jpvialb@incoviba.cl>
Date: Tue, 3 Jun 2025 12:34:08 -0400
Subject: [PATCH] FIX: Request log processor no funciona

---
 .../Implement/Log/Processor/Request.php       | 30 -------------------
 app/setup/setups/logs.php                     | 12 ++++++--
 2 files changed, 10 insertions(+), 32 deletions(-)
 delete mode 100644 app/common/Implement/Log/Processor/Request.php

diff --git a/app/common/Implement/Log/Processor/Request.php b/app/common/Implement/Log/Processor/Request.php
deleted file mode 100644
index b195cb9..0000000
--- a/app/common/Implement/Log/Processor/Request.php
+++ /dev/null
@@ -1,30 +0,0 @@
-<?php
-namespace Incoviba\Common\Implement\Log\Processor;
-
-use Psr\Http\Message\ServerRequestInterface;
-use Monolog\LogRecord;
-use Monolog\Processor\ProcessorInterface;
-
-class Request implements ProcessorInterface
-{
-    public function __construct(protected ServerRequestInterface $request) {}
-
-    public function __invoke(LogRecord $record): LogRecord
-    {
-        $serverFilters = [
-            'HTTP_',
-            'QUERY_',
-            'REDIRECT_',
-            'REMOTE_',
-            'REQUEST_',
-        ];
-        $serverParams = array_filter($this->request->getServerParams(),
-            fn($key) => count(array_filter($serverFilters, fn($prefix) => str_starts_with($key, $prefix))) > 0,
-            ARRAY_FILTER_USE_KEY);
-        $record->extra['request'] = [
-            'server' => $serverParams,
-            'headers' => $this->request->getHeaders(),
-        ];
-        return $record;
-    }
-}
diff --git a/app/setup/setups/logs.php b/app/setup/setups/logs.php
index dffccf9..0a94a68 100644
--- a/app/setup/setups/logs.php
+++ b/app/setup/setups/logs.php
@@ -3,6 +3,16 @@
 use Psr\Container\ContainerInterface;
 
 return [
+    Monolog\Processor\WebProcessor::class => function(ContainerInterface $container) {
+        return new Monolog\Processor\WebProcessor(null, [
+            'HTTP_X_FORWARDED_FOR',
+            'HTTP_CLIENT_IP',
+            'HTTP_X_CLIENT_IP',
+            'HTTP_X_REAL_IP',
+            'REMOTE_ADDR',
+            'REMOTE_HOST',
+        ]);
+    },
     Monolog\Formatter\LineFormatter::class => function(ContainerInterface $container) {
         return (new Monolog\Formatter\LineFormatter(null, null, false, false, true))
             ->setBasePath('/code/');
@@ -59,7 +69,6 @@ return [
         return new Monolog\Logger('login', [
             new Monolog\Handler\RedisHandler($container->get(Predis\ClientInterface::class), 'logs:login'),
         ], [
-            $container->get(Incoviba\Common\Implement\Log\Processor\Request::class),
             $container->get(Monolog\Processor\IntrospectionProcessor::class),
             $container->get(Monolog\Processor\WebProcessor::class),
             $container->get(Monolog\Processor\MemoryUsageProcessor::class),
@@ -71,7 +80,6 @@ return [
         return new Monolog\Logger('access', [
             new Monolog\Handler\RedisHandler($container->get(Predis\ClientInterface::class), 'logs:access'),
         ], [
-            $container->get(Incoviba\Common\Implement\Log\Processor\Request::class),
             $container->get(Monolog\Processor\IntrospectionProcessor::class),
             $container->get(Monolog\Processor\WebProcessor::class),
             $container->get(Monolog\Processor\MemoryUsageProcessor::class),