appsignal 5.0.2-java → 5.0.3-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 62c71c72cf65e314eed6a65860763d7e5875664e630852702ff77af7dd8cf901
-  data.tar.gz: d28d7b78d58597f8e4ce6f37111110405acd0cdcdc38efc290b6131c57501c86
+  metadata.gz: f511b69fdbdeb226062bbaf05e00fe52eb3bd20854c3eeb202edba0b815b1db5
+  data.tar.gz: e1261b245afbc044c149b030ba2b4f4656c7d56f56e37223006ed21a68d1a8be
 SHA512:
-  metadata.gz: 54f8a659c2178f588165b2c24cdc8d7906d4ce290b4904eb9fb1c8240eda1c314bfa062faa1f718a1996b2b2dbe3c75cdf7d7893f6fce64344c763700cf7451a
-  data.tar.gz: 1dcb25bc71bed601aac8cf9a499182227f8f8b36c2550a203780b57a80f8758a061fb6a7c7b9427ec39b46828abfb6a6c47225aa602f89516d0d6142aa36878c
+  metadata.gz: a18f025051286fc68ff81d27cdc6d4d2fd91b71f79e3d439a9d957e39c98350c9dd46fc7626738c8dfddd13908f13884f9be13155475dedfc8e2844e10f44021
+  data.tar.gz: eb8047d550aaa8afd456483ef40db0d188ba492215e582b6f7ddc5c73b6d7e1d957c39bf0183e2b2d5fa1dbd5531a5064d66ac874a1112193feddf9d90f1cd94

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # AppSignal for Ruby gem Changelog
 
+## 5.0.3
+
+_Published on 2026-10-08._
+
+### Fixed
+
+- In collector mode, report counter metrics correctly from processes forked after AppSignal starts, such as Puma workers when using `preload_app!`. Before this change, applications running several forked processes would lose data points or report incorrect values for them. (patch [4e5a3e78](https://github.com/appsignal/appsignal-ruby/commit/4e5a3e781b4ce2b2620b172a15198a2a59e0ebee))
+
 ## 5.0.2
 
 _Published on 2026-10-06._

data/lib/appsignal/opentelemetry/process_resource.rb

@@ -0,0 +1,137 @@
+# frozen_string_literal: true
+
+require "securerandom"
+
+module Appsignal
+  module OpenTelemetry
+    # @!visibility private
+    #
+    # The resource attributes that identify the current process:
+    # `service.instance.id` and `process.pid`. The OpenTelemetry providers are
+    # built once, so a process forked after that inherits its parent's values
+    # until {.attach_fork_hook} refreshes them in the child.
+    module ProcessResource
+      SERVICE_INSTANCE_ID = "service.instance.id"
+      PROCESS_PID = "process.pid"
+      REFRESH_FAILED = "Could not refresh the OpenTelemetry resource after fork"
+
+      class << self
+        # A random UUIDv4, generated again when the process ID changes.
+        def service_instance_id
+          pid = Process.pid
+          unless @service_instance_id_pid == pid
+            @service_instance_id = SecureRandom.uuid
+            @service_instance_id_pid = pid
+          end
+          @service_instance_id
+        end
+
+        def resource
+          ::OpenTelemetry::SDK::Resources::Resource.create(
+            SERVICE_INSTANCE_ID => service_instance_id,
+            PROCESS_PID => Process.pid
+          )
+        end
+
+        def attach_fork_hook
+          return if @fork_hook_attached
+
+          Process.singleton_class.prepend(ForkHook)
+          @fork_hook_attached = true
+        end
+
+        def before_fork
+          return unless Appsignal::OpenTelemetry.started?
+
+          providers.transform_values do |provider|
+            current = current_resource(provider)
+            current && attribute(current, SERVICE_INSTANCE_ID)
+          end
+        rescue => e
+          log_failure("#{e.class}: #{e.message}")
+          nil
+        end
+
+        def after_fork(instance_ids_before_fork)
+          return unless instance_ids_before_fork
+
+          update = resource
+          providers.each do |name, provider|
+            refresh(name, provider, update, instance_ids_before_fork[name])
+          end
+        rescue => e
+          log_failure("#{e.class}: #{e.message}")
+        end
+
+        private
+
+        def providers
+          {
+            "tracer" => ::OpenTelemetry.tracer_provider,
+            "meter" => ::OpenTelemetry.meter_provider,
+            "logger" => ::OpenTelemetry.logger_provider
+          }
+        end
+
+        def refresh(name, provider, update, instance_id_before_fork)
+          current = current_resource(provider)
+          unless current
+            log_failure("the #{name} provider's resource could not be read")
+            return
+          end
+          return unless attribute(current, SERVICE_INSTANCE_ID) == instance_id_before_fork
+
+          refreshed = current.merge(update)
+          write_resource(provider, refreshed)
+          return if current_resource(provider).equal?(refreshed)
+
+          log_failure("the #{name} provider did not keep the new resource")
+        end
+
+        # The SDK providers take their resource only in their constructors, and
+        # read it from `@resource` when they export.
+        def current_resource(provider)
+          if provider.respond_to?(:resource)
+            resource = provider.resource
+            unless provider.respond_to?(:resource=)
+              # Without a setter, only `@resource` can be written, and it isn't there.
+              return unless provider.instance_variable_defined?(:@resource)
+              # The reader returns something else, so writing `@resource` would change nothing.
+              return unless provider.instance_variable_get(:@resource).equal?(resource)
+            end
+          elsif provider.instance_variable_defined?(:@resource)
+            resource = provider.instance_variable_get(:@resource)
+          end
+          # If it's not a resource, it isn't what the provider exports.
+          resource if resource.is_a?(::OpenTelemetry::SDK::Resources::Resource)
+        end
+
+        def write_resource(provider, resource)
+          if provider.respond_to?(:resource=)
+            provider.resource = resource
+          else
+            provider.instance_variable_set(:@resource, resource)
+          end
+        end
+
+        def attribute(resource, key)
+          resource.attribute_enumerator.each { |k, v| return v if k == key }
+          nil
+        end
+
+        def log_failure(reason)
+          Appsignal.internal_logger.warn("#{REFRESH_FAILED}: #{reason}")
+        end
+      end
+
+      module ForkHook
+        def _fork
+          instance_ids = ProcessResource.before_fork
+          pid = super
+          ProcessResource.after_fork(instance_ids) if pid.zero?
+          pid
+        end
+      end
+    end
+  end
+end

data/lib/appsignal/opentelemetry.rb

@@ -8,6 +8,7 @@ require "appsignal/opentelemetry/http_method"
 require "appsignal/opentelemetry/http_response"
 require "appsignal/opentelemetry/http_server_request"
 require "appsignal/opentelemetry/messaging"
+require "appsignal/opentelemetry/process_resource"
 require "appsignal/opentelemetry/proxied_exporter"
 require "appsignal/opentelemetry/rendering"
 require "appsignal/opentelemetry/sql_db_system"
@@ -59,7 +60,9 @@ module Appsignal
         # `LoggerProvider` take a `resource:` kwarg that replaces (not
         # merges), so we do the merge ourselves and use the same merged
         # resource for the tracer provider to keep all three in sync.
-        resource = ::OpenTelemetry::SDK::Resources::Resource.default.merge(build_resource(config))
+        resource = ProcessResource.resource
+          .merge(::OpenTelemetry::SDK::Resources::Resource.default)
+          .merge(build_resource(config))
 
         span_exporter = build_exporter(
           ::OpenTelemetry::Exporter::OTLP::Exporter,
@@ -103,6 +106,7 @@ module Appsignal
         )
 
         @started = true
+        ProcessResource.attach_fork_hook
       rescue LoadError => e
         @started = false
         Appsignal::Utils::StdoutAndLoggerMessage.error(

data/lib/appsignal/version.rb

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

data/sig/appsignal.rbi

@@ -8,7 +8,7 @@
 module Appsignal
   extend Appsignal::Helpers::Metrics
   extend Appsignal::Helpers::Instrumentation
-  VERSION = T.let("5.0.2", T.untyped)
+  VERSION = T.let("5.0.3", 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.2
+  version: 5.0.3
 platform: java
 authors:
 - Robert Beekman
@@ -306,6 +306,7 @@ files:
 - lib/appsignal/opentelemetry/http_response.rb
 - lib/appsignal/opentelemetry/http_server_request.rb
 - lib/appsignal/opentelemetry/messaging.rb
+- lib/appsignal/opentelemetry/process_resource.rb
 - lib/appsignal/opentelemetry/proxied_exporter.rb
 - lib/appsignal/opentelemetry/rendering.rb
 - lib/appsignal/opentelemetry/sql_db_system.rb