appsignal 5.0.3-java → 5.0.4-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f511b69fdbdeb226062bbaf05e00fe52eb3bd20854c3eeb202edba0b815b1db5
-  data.tar.gz: e1261b245afbc044c149b030ba2b4f4656c7d56f56e37223006ed21a68d1a8be
+  metadata.gz: 8cbb7e4a2eea9564a48fa144018d1d6b2a3a0071145dbb83667e1f35dd79ced2
+  data.tar.gz: 94ca97dc758b43867456f9ca351085ba0f43945d55c94ef777d83af89a77c2d2
 SHA512:
-  metadata.gz: a18f025051286fc68ff81d27cdc6d4d2fd91b71f79e3d439a9d957e39c98350c9dd46fc7626738c8dfddd13908f13884f9be13155475dedfc8e2844e10f44021
-  data.tar.gz: eb8047d550aaa8afd456483ef40db0d188ba492215e582b6f7ddc5c73b6d7e1d957c39bf0183e2b2d5fa1dbd5531a5064d66ac874a1112193feddf9d90f1cd94
+  metadata.gz: a78c3d8478e1be4f5ba71a555042be872889ceec267b7fa623068059cdf928434141f19ae4635ffd7b0fe8d14ffc075266d1b95db36655dbaebde41bd381c8cb
+  data.tar.gz: c848362c51a845da9b0f5be46180d857b14a58152765664d24abdf89244fb4c7efe9569a18dda2a3549e04cbb21465468eef5d6fd12164c8c5e73954f777921f

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # AppSignal for Ruby gem Changelog
 
+## 5.0.4
+
+_Published on 2026-10-09._
+
+### Fixed
+
+- Fix an issue where transactions would be reused across requests when an error happened before the transaction could be closed, which would lead to errors being returned by the app itself. (patch [65ff0be0](https://github.com/appsignal/appsignal-ruby/commit/65ff0be0c3d4dea2eb9ed5bdb81004e899e6edf4))
+
 ## 5.0.3
 
 _Published on 2026-10-08._

data/lib/appsignal/rack/event_handler.rb

@@ -37,6 +37,15 @@ module Appsignal
         )
       end
 
+      # @api private
+      def self.complete_transaction(transaction)
+        safe_execution("Appsignal::Rack::EventHandler#complete_transaction") do
+          transaction.complete
+        end
+      ensure
+        Appsignal::Transaction.clear_current_transaction!
+      end
+
       # @api private
       attr_reader :id
       attr_writer :using_appsignal_event_middleware
@@ -68,29 +77,7 @@ module Appsignal
             :opentelemetry_context => Appsignal::OpenTelemetry.extract_rack_context(request.env),
             :opentelemetry_scope => ["appsignal-ruby/rack", Appsignal::VERSION]
           )
-          # Describes the transaction's span as an incoming HTTP request.
-          # Together with the SERVER span kind the transaction already carries,
-          # this is what the trace timeline reads to recognize a web request.
-          #
-          # Set before the event below starts, because attributes go on
-          # whichever span is open and these belong on the transaction's own
-          # span. That event stays open until the response finishes, so there is
-          # no later point in the request at which these could be set.
-          transaction.add_opentelemetry_attributes(
-            Appsignal::OpenTelemetry::HttpServerRequest.attributes_for(
-              :method => Appsignal::Rack::Utils.request_method_from(request),
-              :path => Appsignal::Rack::Utils.request_value_from(request, :path),
-              :scheme => Appsignal::Rack::Utils.request_value_from(request, :scheme),
-              :query => Appsignal::Rack::Utils.request_value_from(request, :query_string),
-              :host => Appsignal::Rack::Utils.request_value_from(request, :hostname),
-              :port => Appsignal::Rack::Utils.request_value_from(request, :port),
-              :protocol =>
-                Appsignal::Rack::Utils.request_env_value_from(request, "SERVER_PROTOCOL")
-            )
-          )
-          transaction.start_event(
-            :opentelemetry_scope => ["appsignal-ruby/rack", Appsignal::VERSION]
-          )
+          # Register cleanup before request instrumentation can raise.
           request.env[APPSIGNAL_TRANSACTION] = transaction
 
           request.env[RACK_AFTER_REPLY] ||= []
@@ -113,9 +100,32 @@ module Appsignal
             # just a fallback if that doesn't get called.
             #
             # One such scenario is when a Puma "lowlevel_error" occurs.
-            transaction.complete
-            Appsignal::Transaction.clear_current_transaction!
+            Appsignal::Rack::EventHandler.complete_transaction(transaction)
           end
+
+          # Describes the transaction's span as an incoming HTTP request.
+          # Together with the SERVER span kind the transaction already carries,
+          # this is what the trace timeline reads to recognize a web request.
+          #
+          # Set before the event below starts, because attributes go on
+          # whichever span is open and these belong on the transaction's own
+          # span. That event stays open until the response finishes, so there is
+          # no later point in the request at which these could be set.
+          transaction.add_opentelemetry_attributes(
+            Appsignal::OpenTelemetry::HttpServerRequest.attributes_for(
+              :method => Appsignal::Rack::Utils.request_method_from(request),
+              :path => Appsignal::Rack::Utils.request_value_from(request, :path),
+              :scheme => Appsignal::Rack::Utils.request_value_from(request, :scheme),
+              :query => Appsignal::Rack::Utils.request_value_from(request, :query_string),
+              :host => Appsignal::Rack::Utils.request_value_from(request, :hostname),
+              :port => Appsignal::Rack::Utils.request_value_from(request, :port),
+              :protocol =>
+                Appsignal::Rack::Utils.request_env_value_from(request, "SERVER_PROTOCOL")
+            )
+          )
+          transaction.start_event(
+            :opentelemetry_scope => ["appsignal-ruby/rack", Appsignal::VERSION]
+          )
         end
       end
 
@@ -188,8 +198,7 @@ module Appsignal
 
         # Make sure the current transaction is always closed when the request
         # is finished
-        transaction.complete
-        Appsignal::Transaction.clear_current_transaction!
+        self.class.complete_transaction(transaction)
       end
 
       private

data/lib/appsignal/transaction/opentelemetry_backend.rb

@@ -446,6 +446,7 @@ module Appsignal
           emit_queue_duration_metric if should_report?
           report_allocation_count
         end
+      ensure
         teardown
       end
 

data/lib/appsignal/transaction.rb

@@ -316,26 +316,30 @@ module Appsignal
         return
       end
 
-      # If the transaction is a duplicate, we don't want to finish it,
-      # because we want its finish time to be the finish time of the
-      # original transaction.
-      # Duplicate transactions should always be sampled, as we only
-      # create duplicates for errors, which are always sampled.
-      should_sample = true
-
-      unless duplicate?
-        self.class.last_errors = @errors.to_a
-        should_sample = @backend.finish
-      end
-
-      report_errors
+      begin
+        # If the transaction is a duplicate, we don't want to finish it,
+        # because we want its finish time to be the finish time of the
+        # original transaction.
+        # Duplicate transactions should always be sampled, as we only
+        # create duplicates for errors, which are always sampled.
+        should_sample = true
+
+        unless duplicate?
+          self.class.last_errors = @errors.to_a
+          should_sample = @backend.finish
+        end
 
-      run_before_complete_hooks
+        report_errors
 
-      sample_data if should_sample
+        run_before_complete_hooks
 
-      @completed = true
-      @backend.complete
+        sample_data if should_sample
+      ensure
+        # Release backend resources even when sample data cannot be collected
+        # or serialized. In collector mode this also detaches the span context.
+        @completed = true
+        @backend.complete
+      end
     end
 
     # @!visibility private

data/lib/appsignal/version.rb

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

data/sig/appsignal.rbi

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 5.0.3
+  version: 5.0.4
 platform: java
 authors:
 - Robert Beekman