From d1fbd830fc4681b0c23b863368772de7f2dedeb2 Mon Sep 17 00:00:00 2001
From: Tom de Bruijn <tom@tomdebruijn.com>
Date: Thu, 12 Sep 2024 09:55:00 +0200
Subject: [PATCH] Refactor Resque parameters storage (#1292)

The Resque integration is sanitizing the parameters. This is already
done in the Transaction class, so we don't need to do it again in the
integration.

Move the value being set to a block, so the parameters are only fetched
when the transaction is sampled.

[skip changeset]
---
 lib/appsignal/integrations/resque.rb | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/lib/appsignal/integrations/resque.rb b/lib/appsignal/integrations/resque.rb
index ba2654b9d..bf77dd140 100644
--- a/lib/appsignal/integrations/resque.rb
+++ b/lib/appsignal/integrations/resque.rb
@@ -16,12 +16,7 @@ def perform
       ensure
         if transaction
           transaction.set_action_if_nil("#{payload["class"]}#perform")
-          args =
-            Appsignal::Utils::HashSanitizer.sanitize(
-              ResqueHelpers.arguments(payload),
-              Appsignal.config[:filter_parameters]
-            )
-          transaction.add_params_if_nil(args)
+          transaction.add_params_if_nil { ResqueHelpers.arguments(payload) }
           transaction.add_tags("queue" => queue)
 
           Appsignal::Transaction.complete_current!