refactor: add request_logs methods

it avoids having to change lograge.rb for adding an entry

app/controllers/api/v2/dossiers_controller.rb

@@ -14,10 +14,10 @@ class API::V2::DossiersController < API::V2::BaseController
 
   private
 
-  def append_info_to_payload(payload)
+  def request_logs(logs)
     super
     if dossier.present?
-      payload.merge!(ds_dossier_id: dossier.id.to_s, ds_procedure_id: dossier.procedure.id.to_s)
+      logs.merge!(ds_dossier_id: dossier.id.to_s, ds_procedure_id: dossier.procedure.id.to_s)
     end
   end
 

app/controllers/api/v2/graphql_controller.rb

@@ -18,10 +18,10 @@ class API::V2::GraphqlController < API::V2::BaseController
 
   private
 
-  def append_info_to_payload(payload)
+  def request_logs(logs)
     super
 
-    payload.merge!(@query_info.presence || {})
+    logs.merge!(@query_info.presence || {})
   end
 
   def process_action(*args)

app/controllers/application_controller.rb

@@ -224,23 +224,26 @@ class ApplicationController < ActionController::Base
   def append_info_to_payload(payload)
     super
 
-    payload.merge!({
+    payload[:to_log] = {}
+    request_logs(payload[:to_log])
+  end
+
+  def request_logs(logs)
+    logs.merge!({
       user_agent: request.user_agent,
       user_id: current_user&.id,
       user_roles: current_user_roles,
       client_ip: request.headers['X-Forwarded-For'],
       request_id: request.headers['X-Request-ID']
-    }.compact)
+    })
 
     if browser.known?
-      payload.merge!({
+      logs.merge!({
         browser: browser.name,
         browser_version: browser.version.to_s,
         platform: browser.platform.name
       })
     end
-
-    payload
   end
 
   def reject

app/controllers/manager/application_controller.rb

@@ -38,21 +38,21 @@ module Manager
     def append_info_to_payload(payload)
       super
 
-      payload.merge!({
+      to_log = {
         user_agent: request.user_agent,
         user_id: current_user&.id,
         user_email: current_user&.email
-      }.compact)
+      }
 
       if browser.known?
-        payload.merge!({
+        to_log.merge!({
           browser: browser.name,
           browser_version: browser.version.to_s,
           platform: browser.platform.name
         })
       end
 
-      payload
+      payload[:to_log] = to_log
     end
   end
 end

config/initializers/lograge.rb

@@ -6,30 +6,18 @@ Rails.application.configure do
   # injected by ansible.
   if !config.lograge.custom_options
     config.lograge.custom_options = lambda do |event|
-      {
+      hash = {
         type: 'tps',
         source: ENV['SOURCE'],
         tags: ['request', event.payload[:exception] ? 'exception' : nil].compact,
-        user_id: event.payload[:user_id],
-        user_email: event.payload[:user_email],
-        user_roles: event.payload[:user_roles],
-        user_agent: event.payload[:user_agent],
-        graphql_query: event.payload[:graphql_query],
-        graphql_variables: event.payload[:graphql_variables],
-        graphql_null_error: event.payload[:graphql_null_error],
-        graphql_timeout_error: event.payload[:graphql_timeout_error],
-        graphql_mutation: event.payload[:graphql_mutation],
-        ds_procedure_id: event.payload[:ds_procedure_id],
-        ds_dossier_id: event.payload[:ds_dossier_id],
-        browser: event.payload[:browser],
-        browser_version: event.payload[:browser_version],
-        platform: event.payload[:platform],
-        client_ip: event.payload[:client_ip],
-        request_id: event.payload[:request_id],
         process: {
           pid: Process.pid
         }
-      }.compact
+      }
+
+      hash.merge!(event.payload[:to_log])
+
+      hash.compact
     end
 
     config.lograge.custom_payload do |controller|

spec/controllers/application_controller_spec.rb

@@ -40,7 +40,7 @@ describe ApplicationController, type: :controller do
         [:db_runtime, :view_runtime, :variant, :rendered_format].each do |key|
           payload.delete(key)
         end
-        expect(payload).to eq({
+        expect(payload[:to_log].compact).to eq({
           user_agent: 'Rails Testing',
           user_roles: 'Guest'
         })
@@ -59,7 +59,7 @@ describe ApplicationController, type: :controller do
         [:db_runtime, :view_runtime, :variant, :rendered_format].each do |key|
           payload.delete(key)
         end
-        expect(payload).to eq({
+        expect(payload[:to_log].compact).to eq({
           user_agent: 'Rails Testing',
           user_id: current_user.id,
           user_roles: 'User'
@@ -82,7 +82,7 @@ describe ApplicationController, type: :controller do
         [:db_runtime, :view_runtime, :variant, :rendered_format].each do |key|
           payload.delete(key)
         end
-        expect(payload).to eq({
+        expect(payload[:to_log].compact).to eq({
           user_agent: 'Rails Testing',
           user_id: current_user.id,
           user_roles: 'User, Instructeur, Administrateur, SuperAdmin'

spec/controllers/manager/application_controller_spec.rb

@@ -13,7 +13,7 @@ describe Manager::ApplicationController, type: :controller do
       [:db_runtime, :view_runtime, :variant, :rendered_format].each do |key|
         payload.delete(key)
       end
-      expect(payload).to eq({
+      expect(payload[:to_log]).to eq({
         user_agent: 'Rails Testing',
         user_id: current_user.id,
         user_email: current_user.email