feat: Ignore ActionController::TooManyRequests for Rails 8+

Add Rails 8 rate limiting error to excluded exceptions conditionally.
Only applies when Rails version >= 8.0.

Author: camallen

CHANGELOG.md

@@ -3,6 +3,7 @@
 ### Features
 
 - Include otel as custom sampling context ([2683](https://github.com/getsentry/sentry-ruby/pull/2683))
+- Ignore new rails rate limit errors ([#2774](https://github.com/getsentry/sentry-ruby/pull/2774))
 
 ## 6.1.2
 

sentry-rails/lib/sentry/rails/configuration.rb

@@ -15,6 +15,7 @@ class Configuration
     after(:initialize) do
       @rails = Sentry::Rails::Configuration.new
       @excluded_exceptions = @excluded_exceptions.concat(Sentry::Rails::IGNORE_DEFAULT)
+      @excluded_exceptions = @excluded_exceptions.concat(Sentry::Rails::RAILS_8_IGNORE_DEFAULT) if ::Rails.version.to_f >= 8.0
 
       if ::Rails.logger
         if defined?(::ActiveSupport::BroadcastLogger) && ::Rails.logger.is_a?(::ActiveSupport::BroadcastLogger)
@@ -48,7 +49,6 @@ module Rails
       "ActionController::NotImplemented",
       "ActionController::ParameterMissing",
       "ActionController::RoutingError",
-      "ActionController::TooManyRequests",
       "ActionController::UnknownAction",
       "ActionController::UnknownFormat",
       "ActionController::UnknownHttpMethod",
@@ -57,6 +57,11 @@ module Rails
       "ActiveRecord::RecordNotFound"
     ].freeze
 
+    # Rails 8.0 introduced rate limiting which raises ActionController::TooManyRequests
+    RAILS_8_IGNORE_DEFAULT = [
+      "ActionController::TooManyRequests"
+    ].freeze
+
     ACTIVE_SUPPORT_LOGGER_SUBSCRIPTION_ITEMS_DEFAULT = {
       # action_controller
       "write_fragment.action_controller" => %i[key],

sentry-rails/spec/sentry/rails/configuration_spec.rb

@@ -15,6 +15,12 @@
     expect(config.excluded_exceptions).to include("ActiveRecord::RecordNotFound")
   end
 
+  it "ignores ActionController::TooManyRequests by default on Rails 8+", skip: Rails.version.to_f < 8.0 do
+    config = Sentry::Configuration.new
+
+    expect(config.excluded_exceptions).to include("ActionController::TooManyRequests")
+  end
+
   describe "#report_rescued_exceptions" do
     it "has correct default value" do
       expect(subject.report_rescued_exceptions).to eq(true)