airbrake 9.4.5 → 9.5.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e3451e5b75e5f477d87dc437e48bc95bee4783bd
-  data.tar.gz: c13ff23bd4a54accb6ba3b9eadd70fa283836f7e
+SHA256:
+  metadata.gz: 1674eebe809e2624774c0d3166fe5e214db9f947a9b41cfcdf4d510268e77993
+  data.tar.gz: c902febfb24d2a8147324b8cb9d225ba41529ee89b97ace68be9c5a529885578
 SHA512:
-  metadata.gz: 92c00e765d4c72494d5c8c3046df7c6ba28a8a10168c1ca06131838c8ca6225ee461694d61c2cf8ef9a9189cf102eee31b2aec6ad5a8621223ebbf04011ef861
-  data.tar.gz: eefbe3fdb7e7a9157a5426114fdfe030a8f05cfa8602827bd7efe8d5279f9fc7ee4d8f900d6e94766d78ccdade11359a09edf196bf8ae439f595110ad00eed72
+  metadata.gz: ccc91fb99278364f2539ed0a09bc8e60aef3780c651f92b65f05108b320d464f6627daf18c3c838def9d7ac6203fd46c84f333d6d905b0959c9b93aae509a2ff
+  data.tar.gz: 151914bb3405cdd4c094a664ef81eb214fe269cacd3af17f8922e2b6bfb79b375f63dcdafe7df06c05aad7a69a1f58358196c3ade5e2fa6f5c43f84347ebab37

data/lib/airbrake/rack/route_filter.rb

@@ -1,3 +1,5 @@
+require 'airbrake/rails/app'
+
 module Airbrake
   module Rack
     # Adds route slugs to context/route.
@@ -23,16 +25,8 @@ module Airbrake
       private
 
       def rails_route(request)
-        ::Rails.application.routes.router.recognize(request) do |route, _parameters|
-          # Rails can recognize multiple routes for the given request. For
-          # example, if we visit /users/2/edit, then Rails sees these routes:
-          #   * "/users/:id/edit(.:format)"
-          #   *  "/"
-          #
-          # We return the first route as, what it seems, the most optimal
-          # approach.
-          return route.path.spec.to_s
-        end
+        return unless (route = Airbrake::Rails::App.recognize_route(request))
+        route.path
       end
 
       def sinatra_route(request)

data/lib/airbrake/rails/action_controller_route_subscriber.rb

@@ -3,27 +3,19 @@ require 'airbrake/rails/app'
 
 module Airbrake
   module Rails
-    # @return [String]
-    CONTROLLER_KEY = 'controller'.freeze
-
-    # @return [String]
-    ACTION_KEY = 'action'.freeze
-
     # ActionControllerRouteSubscriber sends route stat information, including
     # performance data.
     #
     # @since v8.0.0
     class ActionControllerRouteSubscriber
-      def initialize
-        @app = Airbrake::Rails::App.new
-      end
-
       def call(*args)
         # We don't track routeless events.
         return unless (routes = Airbrake::Rack::RequestStore[:routes])
 
         event = Airbrake::Rails::Event.new(*args)
-        route = find_route(event.params)
+        route = Airbrake::Rails::App.recognize_route(
+          Airbrake::Rack::RequestStore[:request]
+        )
         return unless route
 
         routes[route.path] = {
@@ -32,15 +24,6 @@ module Airbrake
           groups: {}
         }
       end
-
-      private
-
-      def find_route(params)
-        @app.routes.find do |route|
-          route.controller == params[CONTROLLER_KEY] &&
-            route.action == params[ACTION_KEY]
-        end
-      end
     end
   end
 end

data/lib/airbrake/rails/app.rb

@@ -1,42 +1,34 @@
 module Airbrake
   module Rails
-    # App is a wrapper around Rails.application and Rails::Engine.
+    # App is a wrapper around Rails.application.
     #
     # @since v9.0.3
     # @api private
     class App
-      Route = Struct.new(:path, :controller, :action)
+      Route = Struct.new(:path)
 
-      def routes
-        @routes ||= app_routes.merge(engine_routes).flat_map do |(engine_name, routes)|
-          routes.map { |rails_route| build_route(engine_name, rails_route) }
-        end
-      end
-
-      private
-
-      def app_routes
-        # Engine name is nil because this is default (non-engine) routes.
-        { nil => ::Rails.application.routes.routes.routes }
-      end
-
-      def engine_routes
-        ::Rails::Engine.subclasses.flat_map.with_object({}) do |engine, hash|
-          next if (eng_routes = engine.routes.routes.routes).none?
+      # @param [] request
+      # @return [Airbrake::Rails::App::Route, nil]
+      def self.recognize_route(request)
+        ::Rails.application.routes.router.recognize(request) do |route, _params|
+          path =
+            if route.app.respond_to?(:app) && route.app.app.respond_to?(:engine_name)
+              "#{route.app.app.engine_name}##{route.path.spec}"
+            else
+              route.path.spec.to_s
+            end
 
-          hash[engine.engine_name] = eng_routes
+          # Rails can recognize multiple routes for the given request. For
+          # example, if we visit /users/2/edit, then Rails sees these routes:
+          #   * "/users/:id/edit(.:format)"
+          #   *  "/"
+          #
+          # We return the first route as, what it seems, the most optimal
+          # approach.
+          return Route.new(path)
         end
-      end
-
-      def build_route(engine_name, rails_route)
-        engine_prefix = engine_name
-        engine_prefix += '#' if engine_prefix
 
-        Route.new(
-          "#{engine_prefix}#{rails_route.path.spec}",
-          rails_route.defaults[:controller],
-          rails_route.defaults[:action]
-        )
+        nil
       end
     end
   end

data/lib/airbrake/rails/railtie.rb

@@ -96,8 +96,11 @@ module Airbrake
         ActiveSupport.on_load(:active_record, run_once: true) do
           # Reports exceptions occurring in some bugged ActiveRecord callbacks.
           # Applicable only to the versions of Rails lower than 4.2.
-          require 'airbrake/rails/active_record'
-          include Airbrake::Rails::ActiveRecord
+          if defined?(::Rails) &&
+             Gem::Version.new(::Rails.version) <= Gem::Version.new('4.2')
+            require 'airbrake/rails/active_record'
+            include Airbrake::Rails::ActiveRecord
+          end
 
           if defined?(ActiveRecord) && Airbrake::Config.instance.query_stats
             # Send SQL queries.

data/lib/airbrake/version.rb

@@ -1,5 +1,5 @@
 # We use Semantic Versioning v2.0.0
 # More information: http://semver.org/
 module Airbrake
-  AIRBRAKE_VERSION = '9.4.5'.freeze
+  AIRBRAKE_VERSION = '9.5.4'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 9.4.5
+  version: 9.5.4
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-03 00:00:00.000000000 Z
+date: 2019-11-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake-ruby
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.7'
+        version: '4.8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.7'
+        version: '4.8'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -240,6 +240,20 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 5.11.3
+- !ruby/object:Gem::Dependency
+  name: rack-cache
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.9.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.9.0
 - !ruby/object:Gem::Dependency
   name: sidekiq
   requirement: !ruby/object:Gem::Requirement
@@ -408,7 +422,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14.4
+rubygems_version: 2.7.6.2
 signing_key: 
 specification_version: 4
 summary: Airbrake is an online tool that provides robust exception tracking in any