From 13561a5f717af796371a244daf11b2e7d3d029d7 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Tue, 19 Apr 2022 10:07:10 -0300
Subject: [PATCH 1/2] =?UTF-8?q?no=20generar=20errores=20si=20se=20env?=
 =?UTF-8?q?=C3=ADa=20el=20reporte=20incompleto?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

closes #5136
---
 app/controllers/api/v1/csp_reports_controller.rb | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/app/controllers/api/v1/csp_reports_controller.rb b/app/controllers/api/v1/csp_reports_controller.rb
index bc6cfae0..f1d7a376 100644
--- a/app/controllers/api/v1/csp_reports_controller.rb
+++ b/app/controllers/api/v1/csp_reports_controller.rb
@@ -6,6 +6,9 @@ module Api
     class CspReportsController < BaseController
       skip_forgery_protection
 
+      # No queremos indicar que algo salió mal
+      rescue_from ActionController::ParameterMissing, with: :csp_report_created
+
       # Crea un reporte de CSP intercambiando los guiones medios por
       # bajos
       #
@@ -18,7 +21,7 @@ module Api
         csp.id = SecureRandom.uuid
         csp.save
 
-        render json: {}, status: :created
+        csp_report_created
       end
 
       private
@@ -39,6 +42,10 @@ module Api
                       :'column-number',
                       :'source-file')
       end
+
+      def csp_report_created
+        render json: {}, status: :created
+      end
     end
   end
 end

From 52f446fcf1e8fc9c31a1ee9ef1028f320bb24304 Mon Sep 17 00:00:00 2001
From: f <f@sutty.nl>
Date: Tue, 19 Apr 2022 10:57:01 -0300
Subject: [PATCH 2/2] refactorizacion

---
 app/controllers/api/v1/csp_reports_controller.rb | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/controllers/api/v1/csp_reports_controller.rb b/app/controllers/api/v1/csp_reports_controller.rb
index f1d7a376..ea186729 100644
--- a/app/controllers/api/v1/csp_reports_controller.rb
+++ b/app/controllers/api/v1/csp_reports_controller.rb
@@ -14,9 +14,9 @@ module Api
       #
       # TODO: Aplicar rate_limit
       def create
-        csp = CspReport.new(csp_report_params.to_h.map do |k, v|
-                              [k.tr('-', '_'), v]
-                            end.to_h)
+        csp = CspReport.new(csp_report_params.to_h.transform_keys do |k|
+                              k.tr('-', '_')
+                            end)
 
         csp.id = SecureRandom.uuid
         csp.save