Add params, path and method to errors reported via Rails

lib/appsignal/integrations/railtie.rb

@@ -60,9 +60,13 @@ def report(error, handled:, severity:, context: {}, source: nil)
           return unless handled
 
           Appsignal.send_error(error) do |transaction|
-            namespace, action_name, tags, custom_data = context_for(context.dup)
+            namespace, action_name, path, method, params, tags, custom_data =
+              context_for(context.dup)
             transaction.set_namespace(namespace) if namespace
             transaction.set_action(action_name) if action_name
+            transaction.set_metadata("path", path)
+            transaction.set_metadata("method", method)
+            transaction.params = params
             transaction.set_sample_data("custom_data", custom_data) if custom_data
 
             tags[:severity] = severity
@@ -81,9 +85,15 @@ def context_for(context)
           # Fetch the namespace and action name based on the Rails execution
           # context.
           controller = context.delete(:controller)
+          path = nil
+          method = nil
+          params = nil
           if controller
             namespace = Appsignal::Transaction::HTTP_REQUEST
             action_name = "#{controller.class.name}##{controller.action_name}"
+            path = controller.request.path
+            method = controller.request.method
+            params = controller.request.filtered_parameters
           end
           # ActiveJob transaction naming relies on the current AppSignal
           # transaction namespace and action name copying done after this.
@@ -115,7 +125,7 @@ def context_for(context)
           end
           tags.merge!(context)
 
-          [namespace, action_name, tags, custom_data]
+          [namespace, action_name, path, method, params, tags, custom_data]
         end
       end
     end

spec/lib/appsignal/integrations/railtie_spec.rb

@@ -256,10 +256,28 @@ def subscribe(subscriber)
             end
 
             context "when no transaction is active" do
+              class ExampleRailsRequestMock
+                def path
+                  "path"
+                end
+
+                def method
+                  "GET"
+                end
+
+                def filtered_parameters
+                  { :user_id => 123, :password => "[FILTERED]" }
+                end
+              end
+
               class ExampleRailsControllerMock
                 def action_name
                   "index"
                 end
+
+                def request
+                  @request ||= ExampleRailsRequestMock.new
+                end
               end
 
               class ExampleRailsJobMock
@@ -275,7 +293,7 @@ def arguments
                 clear_current_transaction!
               end
 
-              it "fetches the action from the controller in the context" do
+              it "fetches the action, path and method from the controller in the context" do
                 # The controller key is set by Rails when raised in a controller
                 given_context = { :controller => ExampleRailsControllerMock.new }
                 with_rails_error_reporter do
@@ -285,7 +303,14 @@ def arguments
                 transaction = last_transaction
                 transaction_hash = transaction.to_h
                 expect(transaction_hash).to include(
-                  "action" => "ExampleRailsControllerMock#index"
+                  "action" => "ExampleRailsControllerMock#index",
+                  "metadata" => hash_including(
+                    "path" => "path",
+                    "method" => "GET"
+                  ),
+                  "sample_data" => hash_including(
+                    "params" => { "user_id" => 123, "password" => "[FILTERED]" }
+                  )
                 )
               end