appmap 0.87.0 → 0.88.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 340b56e93b02e0d530da30ead22c7fe26d067aeac1be6b4188a38819d51c1c82
-  data.tar.gz: 0bbae1f5fea6e1040973c15307aa3ef10b0b2e14d86755dc11dbb13e03ddfa8a
+  metadata.gz: 22745bf509f674a271a3bc42eb870813cf22d6e0c399a87e99377311a0ddaf67
+  data.tar.gz: 482036f94bc62a732263ac7318e6c942efd22282d9b2ace1ae1245bda28de591
 SHA512:
-  metadata.gz: f41681a8af1625ce7003f8c02cd312a2b4ae1cffb6f47a8825930eb5c3e721ed95fde409cda5b02c6cf82e08614570fb4835f186861909d896bfa033e500e837
-  data.tar.gz: ffc6aac108fd752f599b0af3b0f25b859c3c591789169129e46184f5a8499253f27303acbdbe7985e2154a0ca90a8ed673b5000bed0d3f6f9aa9ca8bbd26c189
+  metadata.gz: dd9fe2dc676c639487b19c5ad4c10e52d8dd1ff95ef10f0829077fc6a4100613d220df7bb8aea5c425e65158617112148827136bc2151cfd8e0a13666e49c347
+  data.tar.gz: 5913ce200514ce676f26a6bf20e348fe9481ecd929ac9b8a5874b9f2e2d20d3bdc82722a535afebe7a153c7a7290ccf8e8e510a336f0b5d7265b6b3665b18a69

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+# [0.88.0](https://github.com/applandinc/appmap-ruby/compare/v0.87.0...v0.88.0) (2022-08-31)
+
+
+### Bug Fixes
+
+* Allow recording of error responses ([e538556](https://github.com/applandinc/appmap-ruby/commit/e5385560cbbab2d27ce6c0a1620d5c7539e0d985))
+
+
+### Features
+
+* Add HTTP server request support for Rails 7 ([bd7f6c9](https://github.com/applandinc/appmap-ruby/commit/bd7f6c9109ee6c850250016276c8fbb6a8a66a2e))
+
 # [0.87.0](https://github.com/applandinc/appmap-ruby/compare/v0.86.0...v0.87.0) (2022-08-19)
 
 

data/lib/appmap/handler/rails/request_handler.rb

@@ -73,8 +73,8 @@ module AppMap
           class << self
             def build_from_invocation(parent_id, return_value, elapsed, response, event: HTTPServerResponse.new)
               event ||= HTTPServerResponse.new
-              event.status = response.status
-              event.headers = response.headers.dup
+              event.status = response[:status] || response.status
+              event.headers = (response[:headers] || response.headers).dup
               AppMap::Event::MethodReturn.build_from_invocation parent_id, return_value, nil, elapsed: elapsed, event: event, parameter_schema: true
             end
           end
@@ -114,6 +114,46 @@ module AppMap
             AppMap.tracing.record_event return_event
           end
         end
+
+        # RequestListener listens to the 'start_processing.action_controller' notification as a
+        # source of HTTP server request events. A strategy other than HookMethod is required for
+        # Rails >= 7 due to the hooked methods visibility dropping to private.
+        class RequestListener
+          def self.begin_request(_name, _started, _finished, _unique_id, payload)
+            RequestListener.new(payload)
+          end
+
+          protected
+
+          def initialize(payload)
+            @request_id = payload[:request].request_id
+            @subscriber = self.class.instance_method(:after_hook).bind(self)
+
+            ActiveSupport::Notifications.subscribe 'process_action.action_controller', @subscriber
+            before_hook payload
+          end
+
+          def before_hook(payload)
+            @call_event = HTTPServerRequest.new payload[:request]
+            AppMap.tracing.record_event @call_event
+          end
+
+          def after_hook(_name, started, finished, _unique_id, payload)
+            return unless @request_id == payload[:request].request_id
+
+            return_value = Thread.current[TEMPLATE_RENDER_VALUE]
+            Thread.current[TEMPLATE_RENDER_VALUE] = nil
+            return_event = HTTPServerResponse.build_from_invocation(
+              @call_event.id,
+              return_value,
+              finished - started,
+              payload[:response] || payload
+            )
+
+            AppMap.tracing.record_event return_event
+            ActiveSupport::Notifications.unsubscribe(@subscriber)
+          end
+        end
       end
     end
   end

data/lib/appmap/railtie.rb

@@ -18,7 +18,15 @@ module AppMap
       ActiveSupport::Notifications.subscribe 'sql.sequel', AppMap::Handler::Rails::SQLHandler.new
       ActiveSupport::Notifications.subscribe 'sql.active_record', AppMap::Handler::Rails::SQLHandler.new
 
-      AppMap::Handler::Rails::RequestHandler::HookMethod.new.activate
+      http_hook_available = ActionController::Instrumentation.public_instance_methods.include?(:process_action)
+      if http_hook_available
+        AppMap::Handler::Rails::RequestHandler::HookMethod.new.activate
+      else
+        ActiveSupport::Notifications.subscribe(
+          'start_processing.action_controller',
+          AppMap::Handler::Rails::RequestHandler::RequestListener.method(:begin_request)
+        )
+      end
     end
   end
 end if ENV['APPMAP'] == 'true'

data/lib/appmap/version.rb

@@ -3,7 +3,7 @@
 module AppMap
   URL = 'https://github.com/applandinc/appmap-ruby'
 
-  VERSION = '0.87.0'
+  VERSION = '0.88.0'
 
   APPMAP_FORMAT_VERSION = '1.7.0'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: appmap
 version: !ruby/object:Gem::Version
-  version: 0.87.0
+  version: 0.88.0
 platform: ruby
 authors:
 - Kevin Gilpin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-08-19 00:00:00.000000000 Z
+date: 2022-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: method_source