sentry-rails 6.1.2 → 6.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fd85a162258fc33137da34d2f08bf5ce0feb0286213a0da47aa697f82e44396c
-  data.tar.gz: a5a14058e287ea961a28d1ca19d96fa954460148fe7366f323ba00663e5b2144
+  metadata.gz: b63ec61238dbea2a3fee7cebbdcf72d1810dc50fe4863502ecdbc66b8da7b31e
+  data.tar.gz: 165e2974b74f63c062db1587c1852c151a5b350a15df13c408cea97476d6c10f
 SHA512:
-  metadata.gz: 6caef5a92e27daf55c92d2dc1b3e713b40be00645a3fdecd07b731acda5f7824266b766bb8614d90f1995a3a642c28f6f449487def3e72cf25bf2e33937aaab3
-  data.tar.gz: 3894faa286057a1cb21f6eec1affa796e6e22cea3f1e1ff2ecfc04b306873067a61c5d87fd9beab69cf21045a7b3c60f5e30319c95fd1db1e3adc6f323fec33b
+  metadata.gz: ec0d90eb1b44f1dfb45bb369586cb8ee22da1a79604679ec12f475e1916eefe646a34c1ab97de386a43e7a27c3e3999c684d61038571ad88c6e0dca667f23f45
+  data.tar.gz: fa67c7c00d3d9f71efd13fecb195ea945538f6535464e704115d5e1a8bf7f6c3cf65c3423369270809a6d49cb99ce4b7ab700cca0885756da673f39f99fac166

data/lib/sentry/rails/log_subscriber.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "active_support/log_subscriber"
+require "sentry/utils/logging_helper"
 
 module Sentry
   module Rails
@@ -27,6 +28,8 @@ module Sentry
     #     end
     #   end
     class LogSubscriber < ActiveSupport::LogSubscriber
+      include Sentry::LoggingHelper
+
       ORIGIN = "auto.log.rails.log_subscriber"
 
       class << self

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

@@ -24,6 +24,7 @@ module Sentry
         include ParameterFilter
 
         EXCLUDED_NAMES = ["SCHEMA", "TRANSACTION"].freeze
+        EMPTY_ARRAY = [].freeze
 
         # Handle sql.active_record events
         #
@@ -40,8 +41,6 @@ module Sentry
           cached = event.payload.fetch(:cached, false)
           connection_id = event.payload[:connection_id]
 
-          db_config = extract_db_config(event.payload)
-
           attributes = {
             sql: sql,
             duration_ms: duration_ms(event),
@@ -53,18 +52,18 @@ module Sentry
           if Sentry.configuration.send_default_pii && (binds && !binds.empty?)
             type_casted_binds = type_casted_binds(event)
 
-            binds.each_with_index do |bind, index|
-              key = bind.respond_to?(:name) ? bind.name : index.to_s
-              value = type_casted_binds[index].to_s
+            type_casted_binds.each_with_index do |value, index|
+              bind = binds[index]
+              name = bind.respond_to?(:name) ? bind.name : index.to_s
 
-              attributes["db.query.parameter.#{key}"] = value
+              attributes["db.query.parameter.#{name}"] = value.to_s
             end
           end
 
           attributes[:statement_name] = statement_name if statement_name && statement_name != "SQL"
           attributes[:connection_id] = connection_id if connection_id
 
-          add_db_config_attributes(attributes, db_config)
+          maybe_add_db_config_attributes(attributes, event.payload)
 
           message = build_log_message(statement_name)
 
@@ -73,6 +72,8 @@ module Sentry
             level: :info,
             attributes: attributes
           )
+        rescue => e
+          log_debug("[#{self.class}] failed to log sql event: #{e.message}")
         end
 
         def type_casted_binds(event)
@@ -97,15 +98,9 @@ module Sentry
           end
         end
 
-        def extract_db_config(payload)
-          connection = payload[:connection]
-
-          return unless connection
-
-          extract_db_config_from_connection(connection)
-        end
+        def maybe_add_db_config_attributes(attributes, payload)
+          db_config = extract_db_config(payload)
 
-        def add_db_config_attributes(attributes, db_config)
           return unless db_config
 
           attributes[:db_system] = db_config[:adapter] if db_config[:adapter]
@@ -125,6 +120,14 @@ module Sentry
           attributes[:server_socket_address] = db_config[:socket] if db_config[:socket]
         end
 
+        def extract_db_config(payload)
+          connection = payload[:connection]
+
+          return unless connection
+
+          extract_db_config_from_connection(connection)
+        end
+
         if ::Rails.version.to_f >= 6.1
           def extract_db_config_from_connection(connection)
             if connection.pool.respond_to?(:db_config)

data/lib/sentry/rails/version.rb

@@ -2,6 +2,6 @@
 
 module Sentry
   module Rails
-    VERSION = "6.1.2"
+    VERSION = "6.2.0"
   end
 end

data/sentry-rails.gemspec

@@ -31,5 +31,5 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency "railties", ">= 5.2.0"
-  spec.add_dependency "sentry-ruby", "~> 6.1.2"
+  spec.add_dependency "sentry-ruby", "~> 6.2.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sentry-rails
 version: !ruby/object:Gem::Version
-  version: 6.1.2
+  version: 6.2.0
 platform: ruby
 authors:
 - Sentry Team
@@ -29,14 +29,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.1.2
+        version: 6.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 6.1.2
+        version: 6.2.0
 description: A gem that provides Rails integration for the Sentry error logger
 email: accounts@sentry.io
 executables: []
@@ -89,15 +89,15 @@ files:
 - lib/sentry/rails/tracing/active_support_subscriber.rb
 - lib/sentry/rails/version.rb
 - sentry-rails.gemspec
-homepage: https://github.com/getsentry/sentry-ruby/tree/6.1.2/sentry-rails
+homepage: https://github.com/getsentry/sentry-ruby/tree/6.2.0/sentry-rails
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/getsentry/sentry-ruby/tree/6.1.2/sentry-rails
-  source_code_uri: https://github.com/getsentry/sentry-ruby/tree/6.1.2/sentry-rails
-  changelog_uri: https://github.com/getsentry/sentry-ruby/blob/6.1.2/CHANGELOG.md
+  homepage_uri: https://github.com/getsentry/sentry-ruby/tree/6.2.0/sentry-rails
+  source_code_uri: https://github.com/getsentry/sentry-ruby/tree/6.2.0/sentry-rails
+  changelog_uri: https://github.com/getsentry/sentry-ruby/blob/6.2.0/CHANGELOG.md
   bug_tracker_uri: https://github.com/getsentry/sentry-ruby/issues
-  documentation_uri: http://www.rubydoc.info/gems/sentry-rails/6.1.2
+  documentation_uri: http://www.rubydoc.info/gems/sentry-rails/6.2.0
 rdoc_options: []
 require_paths:
 - lib