airbrake 9.4.1 → 9.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e5a13561811861e0707b33e7ea71324bba1eb04d
-  data.tar.gz: 1a3ba50fbdbd48920dd9f26de0381a00e10babb1
+  metadata.gz: 9b1a464954b87f553c38390071a1303fdceb5685
+  data.tar.gz: '0783aa316473e43d0f71a9962e0f8fac89cc1092'
 SHA512:
-  metadata.gz: 2463d6422b75772e915c11bf709430a32e3c9b077359d247aecb35f226bd5288a28017bdb1f9967504f5202b2066a2c4feba40d76f4152e959c665ff62f3d227
-  data.tar.gz: 4a5b0dbbad8b83112e6b6d2c2e5257874e6ab14162af5d366986e3e58588233e31fcfb9253cafbabf1e0d9bb1d5f217add67a2cfc92bf9f0dcdaaa3fbfcd2060
+  metadata.gz: 4b10df6a1026885ad9770e6b9e594575356cd0c2d0dbd15aa775ab0ac67c6a079749bf2dbcacce5405ba5cc341481b0ef84a172e6654a3db893476b2326466ee
+  data.tar.gz: 5426af26c8efde7f68b519e3ad7b151291b897fa8d6503c408b0e6c8bcca699554e80e7319ee7afb908a3fd2fd1390d2d59f31f78e1dfe04043d8eb8a26633de

data/lib/airbrake/rails/app.rb

@@ -8,26 +8,36 @@ module Airbrake
       Route = Struct.new(:path, :controller, :action)
 
       def routes
-        @routes ||= [*app_routes, *engine_routes].map do |route|
-          Route.new(
-            route.path.spec.to_s,
-            route.defaults[:controller],
-            route.defaults[:action]
-          )
+        @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
-        ::Rails.application.routes.routes.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 do |engine|
-          engine.routes.routes.routes
+        ::Rails::Engine.subclasses.flat_map.with_object({}) do |engine, hash|
+          next if (eng_routes = engine.routes.routes.routes).none?
+
+          hash[engine.engine_name] = eng_routes
         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]
+        )
+      end
     end
   end
 end

data/lib/airbrake/rails/event.rb

@@ -8,6 +8,8 @@ module Airbrake
       # @see https://github.com/rails/rails/issues/8987
       HTML_RESPONSE_WILDCARD = "*/*".freeze
 
+      include Airbrake::Loggable
+
       def initialize(*args)
         @event = ActiveSupport::Notifications::Event.new(*args)
       end
@@ -64,6 +66,10 @@ module Airbrake
           return status
         end
 
+        logger.error do
+          "#{Airbrake::LOG_LABEL} unknown status code for: #{@event.payload}"
+        end
+
         0
       end
 

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.1'.freeze
+  AIRBRAKE_VERSION = '9.4.2'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: airbrake
 version: !ruby/object:Gem::Version
-  version: 9.4.1
+  version: 9.4.2
 platform: ruby
 authors:
 - Airbrake Technologies, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-08-05 00:00:00.000000000 Z
+date: 2019-08-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake-ruby