appsignal 4.7.0 → 4.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 882fa8b34594a9f3bf529528056c1578707ec0cd45f5ef1563933863e6690bc4
-  data.tar.gz: a0e19b771ec45cd1b6e734e7e04b6f26ca2bb9abc6b713fca7b752ec93f2f427
+  metadata.gz: e4c55770d818ef7e9133cfef98452cab49a36af81a5102b8c45ad291232c4ff3
+  data.tar.gz: b54eb07277e5bf0127b483f5d8415d9bb348a025e41ceea71875506af0c49d44
 SHA512:
-  metadata.gz: 0aaa738024f11463a5731dc423891ce7bcc34b124ed5db021b6456ddbf4cd42b735e024c5cfba390aea82f6578d6070f17e7cd03cb70f51f1aa78172c34735aa
-  data.tar.gz: 20c7183bd8fbb5848c469fdd5b2103041fdd09f4e8e45caa81df31e539e4fc76eaf728b0c6de169043c9ede80c91694606fcaa0dd5fad77660a7b47b38a839a1
+  metadata.gz: 331d5cf75a725cfb7d38471bf40ddac613047cf422e9054ce370073606ac1867d0791d3bd1b51d8599aa849e4528fee1dacfa607a5617001f6d23a843564c4b4
+  data.tar.gz: '0138506f1c46bc2275f5713a2d43db2fffc6d7c8faf082c06b2bcb0ff5f65abb75d6cce8a478ebc593fa5237eddd4f523995400909755237c2027eb8975d02af'

data/CHANGELOG.md

@@ -1,5 +1,43 @@
 # AppSignal for Ruby gem Changelog
 
+## 4.7.2
+
+_Published on 2025-09-24._
+
+### Fixed
+
+- Fix sample data not being reported for JRuby applications. Data like tags, parameters, session data, etc. would not be set if a transaction was sampled. (patch [1cd9ea24](https://github.com/appsignal/appsignal-ruby/commit/1cd9ea248943496780a3ca0fdb2fb4482282f9ba))
+
+## 4.7.1
+
+_Published on 2025-09-19._
+
+### Added
+
+- Add support for the [CodeOwnership](https://github.com/rubyatscale/code_ownership) gem, which allows engineering teams to declare ownership of specific parts of a codebase.
+
+  When an error is reported, the AppSignal gem will tag the transaction with the owner of the file that caused the error.
+
+  This feature is enabled by default. To disable it, set the [`instrument_code_ownership` configuration option](https://docs.appsignal.com/ruby/configuration/options.html#option-instrument_code_ownership) to `false`.
+
+  (patch [40860cb2](https://github.com/appsignal/appsignal-ruby/commit/40860cb2c64f9f0ff93dbba253514ddb06020a5f))
+
+### Changed
+
+- Do not patch `Rack::Events` when using recent Rack versions. When using versions 3.2.1 and above,
+  which contain a fix for the bug where using `Rack::Events` breaks requests with streaming bodies,
+  use `Rack::Events` directly, instead of our patched subclass.
+
+  (patch [64f5b0ee](https://github.com/appsignal/appsignal-ruby/commit/64f5b0eebe07e6937a6397ab089d2b6bf1bd234d))
+
+### Fixed
+
+- Fix Rails version detection when only one of Rails's gems is present.
+
+  This prevents loading errors when non-Rails code defines a Rails constant without the full Rails framework.
+
+  (patch [f06952d9](https://github.com/appsignal/appsignal-ruby/commit/f06952d9ac095bb9c7455acd8ab563beb84d8433))
+
 ## 4.7.0
 
 _Published on 2025-08-29._

data/build_matrix.yml

@@ -115,6 +115,7 @@ matrix:
       - "rails-7.1"
       - "rails-7.2"
       - "rails-8.0"
+      - "rails-8.1"
 
   ruby:
     - ruby: "3.5.0-preview1"
@@ -130,6 +131,7 @@ matrix:
     - gem: "no_dependencies"
     - gem: "capistrano2"
     - gem: "capistrano3"
+    - gem: "code_ownership"
     - gem: "dry-monitor"
       only:
         ruby:
@@ -243,6 +245,13 @@ matrix:
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
+    - gem: "rails-8.1"
+      only:
+        ruby:
+          - "3.5.0-preview1"
+          - "3.4.1"
+          - "3.3.4"
+          - "3.2.5"
     - gem: "sequel"
     - gem: "sinatra"
     - gem: "webmachine2"

data/lib/appsignal/config.rb

@@ -112,6 +112,7 @@ module Appsignal
       :ignore_errors => [],
       :ignore_logs => [],
       :ignore_namespaces => [],
+      :instrument_code_ownership => true,
       :instrument_http_rb => true,
       :instrument_net_http => true,
       :instrument_ownership => true,
@@ -186,6 +187,7 @@ module Appsignal
       :enable_rake_performance_instrumentation =>
         "APPSIGNAL_ENABLE_RAKE_PERFORMANCE_INSTRUMENTATION",
       :files_world_accessible => "APPSIGNAL_FILES_WORLD_ACCESSIBLE",
+      :instrument_code_ownership => "APPSIGNAL_INSTRUMENT_CODE_OWNERSHIP",
       :instrument_http_rb => "APPSIGNAL_INSTRUMENT_HTTP_RB",
       :instrument_net_http => "APPSIGNAL_INSTRUMENT_NET_HTTP",
       :instrument_ownership => "APPSIGNAL_INSTRUMENT_OWNERSHIP",

data/lib/appsignal/environment.rb

@@ -80,6 +80,7 @@ module Appsignal
       activerecord
       capistrano
       celluloid
+      code_ownership
       data_mapper
       delayed_job
       dry-monitor

data/lib/appsignal/extension/jruby.rb

@@ -128,7 +128,7 @@ module Appsignal
           :void
         attach_function :appsignal_finish_transaction,
           [:pointer, :long],
-          :void
+          :bool
         attach_function :appsignal_complete_transaction,
           [:pointer],
           :void

data/lib/appsignal/hooks/code_ownership.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module Appsignal
+  class Hooks
+    # @!visibility private
+    class CodeOwnershipHook < Appsignal::Hooks::Hook
+      register :code_ownership
+
+      def dependencies_present?
+        defined?(::CodeOwnership) &&
+          Gem::Specification.find_by_name("code_ownership").version < Gem::Version.new("2.0") &&
+          Appsignal.config && Appsignal.config[:instrument_code_ownership]
+      end
+
+      def install
+        require "appsignal/integrations/code_ownership"
+
+        Appsignal::Transaction.before_complete <<
+          Appsignal::Integrations::CodeOwnershipIntegration.method(:before_complete)
+
+        Appsignal::Environment.report_enabled("code_ownership")
+      end
+    end
+  end
+end

data/lib/appsignal/hooks.rb

@@ -80,6 +80,7 @@ require "appsignal/hooks/action_mailer"
 require "appsignal/hooks/active_job"
 require "appsignal/hooks/active_support_notifications"
 require "appsignal/hooks/celluloid"
+require "appsignal/hooks/code_ownership"
 require "appsignal/hooks/delayed_job"
 require "appsignal/hooks/gvl"
 require "appsignal/hooks/dry_monitor"

data/lib/appsignal/integrations/code_ownership.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Appsignal
+  module Integrations
+    # @!visibility private
+    module CodeOwnershipIntegration
+      class << self
+        def before_complete(transaction, error)
+          team = ::CodeOwnership.for_backtrace(error.backtrace)
+          transaction.add_tags(:owner => team.name)
+        rescue => ex
+          logger = Appsignal.internal_logger
+          logger.error(
+            "Error while looking up CodeOwnership team: #{ex.class}: #{ex.message}\n" \
+              "#{ex.backtrace.join("\n")}"
+          )
+        end
+      end
+    end
+  end
+end

data/lib/appsignal/integrations/railtie.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-Appsignal.internal_logger.debug("Loading Rails (#{Rails.version}) integration")
+rails_version = Rails.respond_to?(:version) ? Rails.version : "unknown"
+Appsignal.internal_logger.debug("Loading Rails (#{rails_version}) integration")
 
 require "appsignal/utils/rails_helper"
 require "appsignal/rack/rails_instrumentation"

data/lib/appsignal/rack/event_handler.rb

@@ -39,12 +39,12 @@ module Appsignal
 
       # @api private
       attr_reader :id
-      attr_writer :using_appsignal_rack_events_middleware
+      attr_writer :using_appsignal_event_middleware
 
       # @api private
       def initialize
         @id = SecureRandom.uuid
-        @using_appsignal_rack_events_middleware = false
+        @using_appsignal_event_middleware = false
       end
 
       # @api private
@@ -159,7 +159,7 @@ module Appsignal
       end
 
       def emit_warning_once
-        return if @using_appsignal_rack_events_middleware
+        return if @using_appsignal_event_middleware
 
         Appsignal.internal_logger.warn <<~MSG
           Rack::Events is not compatible with streaming bodies. Using `Appsignal::Rack::EventHandler`#{" "}
@@ -168,7 +168,7 @@ module Appsignal
           with streaming bodies.#{" "}
           See https://docs.appsignal.com/ruby/integrations/rack.html for more information.
         MSG
-        @using_appsignal_rack_events_middleware = true
+        @using_appsignal_event_middleware = true
       end
     end
   end

data/lib/appsignal/rack/event_middleware.rb

@@ -2,11 +2,19 @@
 
 module Appsignal
   module Rack
+    # @api private
+    def self.rack_3_2_1_or_newer?
+      return false unless ::Rack.respond_to?(:release)
+
+      Gem::Version.new(::Rack.release) >= Gem::Version.new("3.2.1")
+    end
+
     # Modified version of the {::Rack::Events} instrumentation
     # middleware.
     #
     # We recommend using this instead of {::Rack::Events}, as it
-    # is compatible with streaming bodies.
+    # is compatible with streaming bodies when using Rack versions
+    # before 3.2.0.
     #
     # We do not recommend using this middleware directly, instead
     # recommending the use of {EventMiddleware}, which is a
@@ -15,64 +23,57 @@ module Appsignal
     #
     # See the original implementation at:
     # https://github.com/rack/rack/blob/8d3d7857fcd9e5df057a6c22458bab35b3a19c12/lib/rack/events.rb
-    class Events < ::Rack::Events
-      # A stub for {::Rack::Events::EventedBodyProxy}. It
-      # allows the same initialization arguments, but
-      # otherwise behaves identically to {::Rack::BodyProxy}.
-      #
-      # It does not implement `#each`, fixing an issue
-      # where the evented body proxy would break
-      # streaming responses by always responding to `#each`
-      # even if the proxied body did not implement it.
-      #
-      # Because it ignores the handlers passed to it and
-      # behaves like a normal body proxy, the `on_send`
-      # event on the handlers is never called.
-      class EventedBodyProxy < ::Rack::BodyProxy
-        def initialize(body, _request, _response, _handlers, &block)
-          super(body, &block)
-        end
-      end
 
-      # Notifies the AppSignal event handler that it's being
-      # used with the modified middleware, suppressing a
-      # deprecation warning.
-      def initialize(app, handlers = [])
-        super
-        handlers.each do |handler|
-          if handler.respond_to?(:using_appsignal_rack_events_middleware=)
-            handler.using_appsignal_rack_events_middleware = true
+    if rack_3_2_1_or_newer?
+      Events = ::Rack::Events
+    else
+      class Events < ::Rack::Events
+        # A stub for {::Rack::Events::EventedBodyProxy}. It
+        # allows the same initialization arguments, but
+        # otherwise behaves identically to {::Rack::BodyProxy}.
+        #
+        # It does not implement `#each`, fixing an issue
+        # where the evented body proxy would break
+        # streaming responses by always responding to `#each`
+        # even if the proxied body did not implement it.
+        #
+        # Because it ignores the handlers passed to it and
+        # behaves like a normal body proxy, the `on_send`
+        # event on the handlers is never called.
+        class EventedBodyProxy < ::Rack::BodyProxy
+          def initialize(body, _request, _response, _handlers, &block)
+            super(body, &block)
           end
         end
-      end
 
-      # The `call` method, exactly as implemented by {::Rack::Events},
-      # but redefined here so that it uses our {EventedBodyProxy}
-      # instead of the original {::Rack::Events::EventedBodyProxy}.
-      #
-      # This fixes streaming bodies, but it also means that the
-      # `on_send` event on the handlers is never called.
-      #
-      # See the original implementation at:
-      # https://github.com/rack/rack/blob/8d3d7857fcd9e5df057a6c22458bab35b3a19c12/lib/rack/events.rb#L111-L129
-      def call(env)
-        request = make_request env
-        on_start request, nil
+        # The `call` method, exactly as implemented by {::Rack::Events},
+        # but redefined here so that it uses our {EventedBodyProxy}
+        # instead of the original {::Rack::Events::EventedBodyProxy}.
+        #
+        # This fixes streaming bodies, but it also means that the
+        # `on_send` event on the handlers is never called.
+        #
+        # See the original implementation at:
+        # https://github.com/rack/rack/blob/8d3d7857fcd9e5df057a6c22458bab35b3a19c12/lib/rack/events.rb#L111-L129
+        def call(env)
+          request = make_request env
+          on_start request, nil
 
-        begin
-          status, headers, body = @app.call request.env
-          response = make_response status, headers, body
-          on_commit request, response
-        rescue StandardError => e
-          on_error request, response, e
-          on_finish request, response
-          raise
-        end
+          begin
+            status, headers, body = @app.call request.env
+            response = make_response status, headers, body
+            on_commit request, response
+          rescue StandardError => e
+            on_error request, response, e
+            on_finish request, response
+            raise
+          end
 
-        body = EventedBodyProxy.new(body, request, response, @handlers) do
-          on_finish request, response
+          body = EventedBodyProxy.new(body, request, response, @handlers) do
+            on_finish request, response
+          end
+          [response.status, response.headers, body]
         end
-        [response.status, response.headers, body]
       end
     end
 
@@ -104,7 +105,9 @@ module Appsignal
     # @api public
     class EventMiddleware < Events
       def initialize(app)
-        super(app, [Appsignal::Rack::EventHandler.new])
+        super(app, [Appsignal::Rack::EventHandler.new.tap do |handler|
+          handler.using_appsignal_event_middleware = true
+        end])
       end
     end
   end

data/lib/appsignal/version.rb

@@ -2,5 +2,5 @@
 
 module Appsignal
   # @return [String]
-  VERSION = "4.7.0"
+  VERSION = "4.7.2"
 end

data/lib/appsignal.rb

@@ -644,7 +644,7 @@ require "appsignal/rack/abstract_middleware"
 require "appsignal/rack/instrumentation_middleware"
 require "appsignal/rack/event_handler"
 require "appsignal/rack/event_middleware"
-require "appsignal/integrations/railtie" if defined?(::Rails)
+require "appsignal/integrations/railtie" if defined?(::Rails::Railtie)
 require "appsignal/transaction"
 require "appsignal/version"
 require "appsignal/transmitter"

data/sig/appsignal.rbi

@@ -8,7 +8,7 @@
 module Appsignal
   extend Appsignal::Helpers::Metrics
   extend Appsignal::Helpers::Instrumentation
-  VERSION = T.let("4.7.0", T.untyped)
+  VERSION = T.let("4.7.2", T.untyped)
 
   class << self
     # The loaded AppSignal configuration.

metadata

@@ -1,16 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 4.7.0
+  version: 4.7.2
 platform: ruby
 authors:
 - Robert Beekman
 - Thijs Cadier
 - Tom de Bruijn
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-29 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logger
@@ -217,6 +216,7 @@ files:
 - lib/appsignal/hooks/active_support_notifications.rb
 - lib/appsignal/hooks/at_exit.rb
 - lib/appsignal/hooks/celluloid.rb
+- lib/appsignal/hooks/code_ownership.rb
 - lib/appsignal/hooks/data_mapper.rb
 - lib/appsignal/hooks/delayed_job.rb
 - lib/appsignal/hooks/dry_monitor.rb
@@ -243,6 +243,7 @@ files:
 - lib/appsignal/integrations/active_support_notifications.rb
 - lib/appsignal/integrations/capistrano/appsignal.cap
 - lib/appsignal/integrations/capistrano/capistrano_2_tasks.rb
+- lib/appsignal/integrations/code_ownership.rb
 - lib/appsignal/integrations/data_mapper.rb
 - lib/appsignal/integrations/delayed_job_plugin.rb
 - lib/appsignal/integrations/dry_monitor.rb
@@ -319,7 +320,6 @@ metadata:
   documentation_uri: https://docs.appsignal.com/ruby/
   homepage_uri: https://docs.appsignal.com/ruby/
   source_code_uri: https://github.com/appsignal/appsignal-ruby
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -335,8 +335,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com
 test_files: []