feat(rails): add binds to ActiveRecord logs

Author: solnic

sentry-rails/lib/sentry/rails/log_subscribers/active_record_subscriber.rb

@@ -46,6 +46,15 @@ def sql(event)
             cached: cached
           }
 
+          if Sentry.configuration.send_default_pii && !(binds = event.payload[:binds]).empty?
+            type_casted_binds = event.payload[:type_casted_binds]
+
+            binds.each_with_index do |bind, index|
+              name = bind.is_a?(Symbol) ? bind : bind.name.to_sym
+              attributes[name] = type_casted_binds[index]
+            end
+          end
+
           attributes[:statement_name] = statement_name if statement_name && statement_name != "SQL"
           attributes[:connection_id] = connection_id if connection_id
 

sentry-rails/spec/dummy/test_rails_app/apps/5-2.rb

@@ -30,6 +30,8 @@
   end
 
   create_table :posts, force: true do |t|
+    t.string :title
+    t.timestamps
   end
 
   create_table :comments, force: true do |t|

sentry-rails/spec/dummy/test_rails_app/apps/6-0.rb

@@ -30,6 +30,8 @@
   end
 
   create_table :posts, force: true do |t|
+    t.string :title
+    t.timestamps
   end
 
   create_table :comments, force: true do |t|

sentry-rails/spec/dummy/test_rails_app/apps/6-1.rb

@@ -30,6 +30,8 @@
   end
 
   create_table :posts, force: true do |t|
+    t.string :title
+    t.timestamps
   end
 
   create_table :comments, force: true do |t|

sentry-rails/spec/dummy/test_rails_app/apps/7-0.rb

@@ -30,6 +30,8 @@
   end
 
   create_table :posts, force: true do |t|
+    t.string :title
+    t.timestamps
   end
 
   create_table :comments, force: true do |t|

sentry-rails/spec/dummy/test_rails_app/apps/7-1.rb

@@ -30,6 +30,8 @@
   end
 
   create_table :posts, force: true do |t|
+    t.string :title
+    t.timestamps
   end
 
   create_table :comments, force: true do |t|

sentry-rails/spec/sentry/rails/log_subscribers/active_record_subscriber_spec.rb

@@ -12,6 +12,7 @@
         config.rails.structured_logging.subscribers = { active_record: Sentry::Rails::LogSubscribers::ActiveRecordSubscriber }
       end
     end
+
     describe "integration with ActiveSupport::Notifications" do
       it "logs SQL events when database queries are executed" do
         Post.create!
@@ -45,6 +46,61 @@
         expect(log_event[:attributes][:sql][:value]).to include("posts")
       end
 
+      context "when send_default_pii is enabled" do
+        before do
+          Sentry.configuration.send_default_pii = true
+        end
+
+        after do
+          Sentry.configuration.send_default_pii = false
+        end
+
+        it "logs SELECT queries with binds in attributes" do
+          post = Post.create!(title: "test")
+
+          Sentry.get_current_client.flush
+          sentry_transport.events.clear
+          sentry_transport.envelopes.clear
+
+          created_at = Time.new(2025, 10, 28, 13, 11, 44)
+          Post.where(id: post.id, title: post.title, created_at: created_at).to_a
+
+          Sentry.get_current_client.flush
+
+          log_event = sentry_logs.find { |log| log[:body]&.include?("Database query") }
+          expect(log_event).not_to be_nil
+
+          expect(log_event[:attributes][:id][:value]).to be(post.id)
+          expect(log_event[:attributes][:id][:type]).to eql("integer")
+
+          expect(log_event[:attributes][:title][:value]).to eql(post.title)
+          expect(log_event[:attributes][:title][:type]).to eql("string")
+
+          expect(log_event[:attributes][:created_at][:value]).to include("2025-10-28 13:11:44")
+          expect(log_event[:attributes][:created_at][:type]).to eql("string")
+        end
+      end
+
+      context "when send_default_pii is disabled" do
+        it "logs SELECT queries without binds in attributes" do
+          post = Post.create!(title: "test")
+
+          Sentry.get_current_client.flush
+          sentry_transport.events.clear
+          sentry_transport.envelopes.clear
+
+          Post.where(id: post.id, title: post.title).to_a
+
+          Sentry.get_current_client.flush
+
+          log_event = sentry_logs.find { |log| log[:body]&.include?("Database query") }
+          expect(log_event).not_to be_nil
+
+          expect(log_event[:attributes][:id]).to be_nil
+          expect(log_event[:attributes][:title]).to be_nil
+        end
+      end
+
       if Rails.version.to_f > 5.1
         it "excludes SCHEMA events" do
           ActiveSupport::Notifications.instrument("sql.active_record",