sentry-rails 5.28.1 → 6.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '090819d1b26b5802e42a93acf2f5adb8014db90325beb42d3a570dbf04fa4499'
-  data.tar.gz: c26c54d43bccab705478865a08b38fb8bb87797cabd7b5f531ec8be1b1371303
+  metadata.gz: 2f1d7d083cce7bf8f0e04a82c63fd1284f836e2ae3aa30a9237eba989e72ae93
+  data.tar.gz: 681cc6a28160e9f8138cdfc2a9c9c8ab81e2a90e43559306d30061e92fe63c91
 SHA512:
-  metadata.gz: 66bf25eb2d40931c96efda40c16f09360a69dd6f97ae9e587c46683304a5157c8ab05918374cbfda80b728a7b4033e05244ebf0efa2dd1863cb76888d26c9bee
-  data.tar.gz: d8324fcbdd8576961d12eaccd18142cfe3e79685a5dcbbbb6d6d337fb3dd08a6cf9ee85b04c2440b47ea3b5981e3cc6d863ae5df5efac6fe3cffd2406f3d99c0
+  metadata.gz: db6546de2db3507a022eff373b10bed7380568fd40055c50b3e80cfe907ecc2479799e7fbfc5f598baa7ebbf92e410823fe71951cd8ae664c15714bb8e49c607
+  data.tar.gz: c71fe0109e60f59e36c91d0e3cbc95c8f34cb5b7b767261e68f40c04716b7d296939ad507ff7166bb618f63fac78b0865313ba2f6b685ba46b659c7175a38575

data/Gemfile

@@ -21,10 +21,8 @@ rails_version = ENV.fetch("RAILS_VERSION") do
     "8.0"
   elsif ruby_version >= Gem::Version.new("3.1")
     "7.2"
-  elsif ruby_version >= Gem::Version.new("2.7")
+  else
     "7.1"
-  elsif ruby_version >= Gem::Version.new("2.4")
-    "5.2"
   end
 end
 
@@ -42,11 +40,7 @@ elsif rails_version >= Gem::Version.new("7.1.0")
 elsif rails_version >= Gem::Version.new("6.1.0")
   gem "rspec-rails", "~> 4.0"
 
-  if ruby_version >= Gem::Version.new("2.7.0")
-    gem "sqlite3", "~> 1.7.3", platform: :ruby
-  else
-    gem "sqlite3", "~> 1.6.9", platform: :ruby
-  end
+  gem "sqlite3", "~> 1.7.3", platform: :ruby
 else
   gem "psych", "~> 3.0.0"
   gem "rspec-rails", "~> 4.0"
@@ -58,12 +52,6 @@ else
   end
 end
 
-if ruby_version < Gem::Version.new("2.5.0")
-  # https://github.com/flavorjones/loofah/pull/267
-  # loofah changed the required ruby version in a patch so we need to explicitly pin it
-  gem "loofah", "2.20.0"
-end
-
 gem "mini_magick"
 
 gem "sprockets-rails"

data/lib/sentry/rails/active_job.rb

@@ -32,17 +32,13 @@ module Sentry
             Sentry.with_scope do |scope|
               begin
                 scope.set_transaction_name(job.class.name, source: :task)
-                transaction =
-                  if job.is_a?(::Sentry::SendEventJob)
-                    nil
-                  else
-                    Sentry.start_transaction(
-                      name: scope.transaction_name,
-                      source: scope.transaction_source,
-                      op: OP_NAME,
-                      origin: SPAN_ORIGIN
-                    )
-                  end
+
+                transaction = Sentry.start_transaction(
+                  name: scope.transaction_name,
+                  source: scope.transaction_source,
+                  op: OP_NAME,
+                  origin: SPAN_ORIGIN
+                )
 
                 scope.set_span(transaction) if transaction
 

data/lib/sentry/rails/configuration.rb

@@ -1,6 +1,5 @@
 # frozen_string_literal: true
 
-require "sentry/rails/tracing/action_controller_subscriber"
 require "sentry/rails/tracing/action_view_subscriber"
 require "sentry/rails/tracing/active_record_subscriber"
 require "sentry/rails/tracing/active_storage_subscriber"

data/lib/sentry/rails/overrides/streaming_reporter.rb

@@ -9,17 +9,6 @@ module Sentry
           super
         end
       end
-
-      module OldStreamingReporter
-        def self.included(base)
-          base.send(:alias_method_chain, :log_error, :raven)
-        end
-
-        def log_error_with_raven(exception)
-          Sentry::Rails.capture_exception(exception)
-          log_error_without_raven(exception)
-        end
-      end
     end
   end
 end

data/lib/sentry/rails/railtie.rb

@@ -101,17 +101,12 @@ module Sentry
     end
 
     def inject_breadcrumbs_logger
-      if Sentry.configuration.breadcrumbs_logger.include?(:active_support_logger)
+      if Sentry.configuration.breadcrumbs_logger.include?(:active_support_logger) ||
+        ## legacy name redirected for backwards compat
+        Sentry.configuration.breadcrumbs_logger.include?(:monotonic_active_support_logger)
         require "sentry/rails/breadcrumb/active_support_logger"
         Sentry::Rails::Breadcrumb::ActiveSupportLogger.inject(Sentry.configuration.rails.active_support_logger_subscription_items)
       end
-
-      if Sentry.configuration.breadcrumbs_logger.include?(:monotonic_active_support_logger)
-        return warn "Usage of `monotonic_active_support_logger` require a version of Rails >= 6.1, please upgrade your Rails version or use another logger" if ::Rails.version.to_f < 6.1
-
-        require "sentry/rails/breadcrumb/monotonic_active_support_logger"
-        Sentry::Rails::Breadcrumb::MonotonicActiveSupportLogger.inject
-      end
     end
 
     def setup_backtrace_cleanup_callback

data/lib/sentry/rails/version.rb

@@ -2,6 +2,6 @@
 
 module Sentry
   module Rails
-    VERSION = "5.28.1"
+    VERSION = "6.0.0"
   end
 end

data/sentry-rails.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
   spec.license = 'MIT'
 
   spec.platform = Gem::Platform::RUBY
-  spec.required_ruby_version = '>= 2.4'
+  spec.required_ruby_version = '>= 2.7'
   spec.extra_rdoc_files = ["README.md", "LICENSE.txt"]
   spec.files = `git ls-files | grep -Ev '^(spec|benchmarks|examples|\.rubocop\.yml)'`.split("\n")
 
@@ -30,6 +30,6 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "railties", ">= 5.0"
-  spec.add_dependency "sentry-ruby", "~> 5.28.1"
+  spec.add_dependency "railties", ">= 5.2.0"
+  spec.add_dependency "sentry-ruby", "~> 6.0.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sentry-rails
 version: !ruby/object:Gem::Version
-  version: 5.28.1
+  version: 6.0.0
 platform: ruby
 authors:
 - Sentry Team
@@ -15,28 +15,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: 5.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: 5.2.0
 - !ruby/object:Gem::Dependency
   name: sentry-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.28.1
+        version: 6.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.28.1
+        version: 6.0.0
 description: A gem that provides Rails integration for the Sentry error logger
 email: accounts@sentry.io
 executables: []
@@ -65,14 +65,12 @@ files:
 - lib/sentry/rails/background_worker.rb
 - lib/sentry/rails/backtrace_cleaner.rb
 - lib/sentry/rails/breadcrumb/active_support_logger.rb
-- lib/sentry/rails/breadcrumb/monotonic_active_support_logger.rb
 - lib/sentry/rails/capture_exceptions.rb
 - lib/sentry/rails/configuration.rb
 - lib/sentry/rails/controller_methods.rb
 - lib/sentry/rails/controller_transaction.rb
 - lib/sentry/rails/engine.rb
 - lib/sentry/rails/error_subscriber.rb
-- lib/sentry/rails/instrument_payload_cleanup_helper.rb
 - lib/sentry/rails/log_subscriber.rb
 - lib/sentry/rails/log_subscribers/action_controller_subscriber.rb
 - lib/sentry/rails/log_subscribers/action_mailer_subscriber.rb
@@ -85,22 +83,21 @@ files:
 - lib/sentry/rails/structured_logging.rb
 - lib/sentry/rails/tracing.rb
 - lib/sentry/rails/tracing/abstract_subscriber.rb
-- lib/sentry/rails/tracing/action_controller_subscriber.rb
 - lib/sentry/rails/tracing/action_view_subscriber.rb
 - lib/sentry/rails/tracing/active_record_subscriber.rb
 - lib/sentry/rails/tracing/active_storage_subscriber.rb
 - lib/sentry/rails/tracing/active_support_subscriber.rb
 - lib/sentry/rails/version.rb
 - sentry-rails.gemspec
-homepage: https://github.com/getsentry/sentry-ruby/tree/5.28.1/sentry-rails
+homepage: https://github.com/getsentry/sentry-ruby/tree/6.0.0/sentry-rails
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/getsentry/sentry-ruby/tree/5.28.1/sentry-rails
-  source_code_uri: https://github.com/getsentry/sentry-ruby/tree/5.28.1/sentry-rails
-  changelog_uri: https://github.com/getsentry/sentry-ruby/blob/5.28.1/CHANGELOG.md
+  homepage_uri: https://github.com/getsentry/sentry-ruby/tree/6.0.0/sentry-rails
+  source_code_uri: https://github.com/getsentry/sentry-ruby/tree/6.0.0/sentry-rails
+  changelog_uri: https://github.com/getsentry/sentry-ruby/blob/6.0.0/CHANGELOG.md
   bug_tracker_uri: https://github.com/getsentry/sentry-ruby/issues
-  documentation_uri: http://www.rubydoc.info/gems/sentry-rails/5.28.1
+  documentation_uri: http://www.rubydoc.info/gems/sentry-rails/6.0.0
 rdoc_options: []
 require_paths:
 - lib
@@ -108,7 +105,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/lib/sentry/rails/breadcrumb/monotonic_active_support_logger.rb

@@ -1,46 +0,0 @@
-# frozen_string_literal: true
-
-require "sentry/rails/instrument_payload_cleanup_helper"
-
-module Sentry
-  module Rails
-    module Breadcrumb
-      module MonotonicActiveSupportLogger
-        class << self
-          include InstrumentPayloadCleanupHelper
-
-          def add(name, started, _finished, _unique_id, data)
-            # skip Rails' internal events
-            return if name.start_with?("!")
-
-            if data.is_a?(Hash)
-              # we should only mutate the copy of the data
-              data = data.dup
-              cleanup_data(data)
-            end
-
-            crumb = Sentry::Breadcrumb.new(
-              data: data,
-              category: name,
-              timestamp: started.to_i
-            )
-            Sentry.add_breadcrumb(crumb)
-          end
-
-          def inject
-            @subscriber = ::ActiveSupport::Notifications.monotonic_subscribe(/.*/) do |name, started, finished, unique_id, data|
-              # we only record events that has a float as started timestamp
-              if started.is_a?(Float)
-                add(name, started, finished, unique_id, data)
-              end
-            end
-          end
-
-          def detach
-            ::ActiveSupport::Notifications.unsubscribe(@subscriber)
-          end
-        end
-      end
-    end
-  end
-end

data/lib/sentry/rails/instrument_payload_cleanup_helper.rb

@@ -1,15 +0,0 @@
-# frozen_string_literal: true
-
-module Sentry
-  module Rails
-    module InstrumentPayloadCleanupHelper
-      IGNORED_DATA_TYPES = [:request, :response, :headers, :exception, :exception_object, Tracing::START_TIMESTAMP_NAME]
-
-      def cleanup_data(data)
-        IGNORED_DATA_TYPES.each do |key|
-          data.delete(key) if data.key?(key)
-        end
-      end
-    end
-  end
-end

data/lib/sentry/rails/tracing/action_controller_subscriber.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-require "sentry/rails/tracing/abstract_subscriber"
-require "sentry/rails/instrument_payload_cleanup_helper"
-
-module Sentry
-  module Rails
-    module Tracing
-      class ActionControllerSubscriber < AbstractSubscriber
-        extend InstrumentPayloadCleanupHelper
-
-        EVENT_NAMES = ["process_action.action_controller"].freeze
-        OP_NAME = "view.process_action.action_controller"
-        SPAN_ORIGIN = "auto.view.rails"
-
-        def self.subscribe!
-          Sentry.sdk_logger.warn <<~MSG
-            DEPRECATION WARNING: sentry-rails has changed its approach on controller span recording and #{self.name} is now depreacted.
-            Please stop using or referencing #{self.name} as it will be removed in the next major release.
-          MSG
-
-          subscribe_to_event(EVENT_NAMES) do |event_name, duration, payload|
-            controller = payload[:controller]
-            action = payload[:action]
-
-            record_on_current_span(
-              op: OP_NAME,
-              origin: SPAN_ORIGIN,
-              start_timestamp: payload[START_TIMESTAMP_NAME],
-              description: "#{controller}##{action}",
-              duration: duration
-            ) do |span|
-              payload = payload.dup
-              cleanup_data(payload)
-              span.set_data(:payload, payload)
-              span.set_http_status(payload[:status])
-            end
-          end
-        end
-      end
-    end
-  end
-end