appsignal 4.5.8-java → 4.5.10-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ecc706edb88f6b60fd226d388ef36ccb4d0a3cb8b3cbf98a8eaca1aeace9b22
-  data.tar.gz: eb18c6e7c352ab0e5e44f739190df8a2c671212b97133311070633944597b63f
+  metadata.gz: 95e8aacf7141e9d5c298ecfcb55530ddab73c830c06dbd94ef78090be8c82987
+  data.tar.gz: 23c78bd69fa935df895c98f731cc2b3f12b55c47dc4f7027f953b3d2725103e6
 SHA512:
-  metadata.gz: b77daa7a8e0b80534be9a721cb9cca2f16843e9bef86151345114e66303aaff08ea0e8f94e6314c12384bd17a2e4a3faf50a230f5ef52349c54fb3767d0b7755
-  data.tar.gz: 18efa45cf4d42fd8faacaf5cec5ec93e76d41773d83e9e077a6d4528aeb5019a05ce41b35b1e88e3449ccd8635ce27045c8cca49d46077327d6490e3e6dc31d4
+  metadata.gz: 8cd4da05e0b0f2c71bb3119a6a5e1746e51a6234af8c73923a73432c4cc48139039c2796264f83bcbd736bb5b12236f68bb7d76d65e136fb5ae644114b2877d2
+  data.tar.gz: 0a4594b5f101b790ef24cb66b9badd736a3ed8fd2b5ae49b2d12448fd6390d86ee61979d741cb0f1a92850c3445adf4eedadf925d15360c070f7be02031ffeee

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # AppSignal for Ruby gem Changelog
 
+## 4.5.10
+
+_Published on 2025-05-05._
+
+### Changed
+
+- Remove redundant cron check-in pairs. When more than one pair of start and finish cron check-in events is reported for the same identifier in the same period, only one of them will be reported to AppSignal. (patch [803a0389](https://github.com/appsignal/appsignal-ruby/commit/803a03893aaf83391129b0b30cda169c8922a870))
+
+## 4.5.9
+
+_Published on 2025-04-09._
+
+### Fixed
+
+- Ensure request transactions are always closed in the `Rack::EventHandler`. A problem with Fibers changing during a request would cause transactions transactions to be left open and the data from requests to not be sent to our servers. (patch [52a72395](https://github.com/appsignal/appsignal-ruby/commit/52a72395b99a44d451fdc4026acfeb0cc2a5c23b))
+
 ## 4.5.8
 
 _Published on 2025-04-04._

data/README.md

@@ -9,9 +9,6 @@ issues.
 - [Ruby code documentation][ruby-doc]
 - [Support][contact]
 
-[![Gem Version](https://badge.fury.io/rb/appsignal.svg)](http://badge.fury.io/rb/appsignal)
-[![Code Climate](https://codeclimate.com/github/appsignal/appsignal.png)](https://codeclimate.com/github/appsignal/appsignal)
-
 ## Description
 
 The AppSignal gem collects exceptions and performance data from your Ruby

data/build_matrix.yml

@@ -88,6 +88,7 @@ matrix:
       - "rails-8.0"
 
   ruby:
+    - ruby: "3.5.0-preview1"
     - ruby: "3.4.1"
     - ruby: "3.3.4"
     - ruby: "3.2.5"
@@ -103,6 +104,7 @@ matrix:
     - gem: "dry-monitor"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -112,6 +114,7 @@ matrix:
     - gem: "hanami-2.0"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -120,6 +123,7 @@ matrix:
     - gem: "hanami-2.1"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -128,6 +132,7 @@ matrix:
     - gem: "hanami-2.2"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -138,6 +143,7 @@ matrix:
     - gem: "psych-3"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -147,6 +153,7 @@ matrix:
     - gem: "psych-4"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -173,6 +180,7 @@ matrix:
     - gem: "rails-7.0"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -183,6 +191,7 @@ matrix:
     - gem: "rails-7.1"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -192,6 +201,7 @@ matrix:
     - gem: "rails-7.2"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -200,6 +210,7 @@ matrix:
     - gem: "rails-8.0"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
           - "3.3.4"
           - "3.2.5"
@@ -211,8 +222,10 @@ matrix:
     - gem: "sidekiq-7"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"
     - gem: "sidekiq-8"
       only:
         ruby:
+          - "3.5.0-preview1"
           - "3.4.1"

data/lib/appsignal/check_in/event.rb

@@ -66,6 +66,61 @@ module Appsignal
             end
           end
         end
+
+        def deduplicate_cron!(events) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity
+          # Remove redundant cron check-in events from the given list of events.
+          # This is done by removing redundant *pairs* of events -- that is,
+          # for each identifier, only send one complete pair of start and
+          # finish events. Remove all other complete pairs of start and finish
+          # events for that identifier, but keep any other start or finish events
+          # that don't have a matching pair.
+          #
+          # Note that this method assumes that the events in this list have already
+          # been rejected based on `Event.redundant?`, so we don't check to remove
+          # check-in events that are functionally identical.
+          start_digests = Hash.new { |h, k| h[k] = Set.new }
+          finish_digests = Hash.new { |h, k| h[k] = Set.new }
+          complete_digests = Hash.new { |h, k| h[k] = Set.new }
+          keep_digest = {}
+
+          # Compute a list of complete digests for each identifier, that is, digests
+          # for which both a start and finish cron check-in event exist. Store the
+          # last seen digest for each identifier as the one to keep.
+          events.each do |event|
+            if event[:check_in_type] == "cron"
+              if event[:kind] == "start"
+                start_digests[event[:identifier]] << event[:digest]
+                if finish_digests[event[:identifier]].include?(event[:digest])
+                  complete_digests[event[:identifier]] << event[:digest]
+                  keep_digest[event[:identifier]] = event[:digest]
+                end
+              elsif event[:kind] == "finish"
+                finish_digests[event[:identifier]] << event[:digest]
+                if start_digests[event[:identifier]].include?(event[:digest])
+                  complete_digests[event[:identifier]] << event[:digest]
+                  keep_digest[event[:identifier]] = event[:digest]
+                end
+              end
+            end
+          end
+
+          start_digests = nil
+          finish_digests = nil
+
+          events.reject! do |event|
+            # Do not remove events that are not cron check-in events or that
+            # have an unknown kind.
+            return false unless
+              event[:check_in_type] == "cron" && (
+                event[:kind] == "start" ||
+                event[:kind] == "finish")
+
+            # Remove any event that is part of a complete digest pair, except
+            # for the one digest that should be kept.
+            keep_digest[event[:identifier]] != event[:digest] &&
+              complete_digests[event[:identifier]].include?(event[:digest])
+          end
+        end
       end
     end
   end

data/lib/appsignal/check_in/scheduler.rb

@@ -161,7 +161,9 @@ module Appsignal
         # Push a copy of the events to the queue, and clear the events array.
         # This ensures that `@events` always contains events that have not
         # yet been pushed to the queue.
-        @queue.push(@events.dup)
+        events = @events.dup
+        Event.deduplicate_cron!(events)
+        @queue.push(events)
         @events.clear
 
         start_waker(BETWEEN_TRANSMISSIONS_DEBOUNCE_SECONDS)

data/lib/appsignal/rack/event_handler.rb

@@ -82,7 +82,8 @@ module Appsignal
             # just a fallback if that doesn't get called.
             #
             # One such scenario is when a Puma "lowlevel_error" occurs.
-            Appsignal::Transaction.complete_current!
+            transaction.complete
+            Appsignal::Transaction.clear_current_transaction!
           end
         end
       end
@@ -138,7 +139,8 @@ module Appsignal
 
         # Make sure the current transaction is always closed when the request
         # is finished
-        Appsignal::Transaction.complete_current!
+        transaction.complete
+        Appsignal::Transaction.clear_current_transaction!
       end
 
       private

data/lib/appsignal/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Appsignal
-  VERSION = "4.5.8"
+  VERSION = "4.5.10"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 4.5.8
+  version: 4.5.10
 platform: java
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-04 00:00:00.000000000 Z
+date: 2025-05-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logger
@@ -331,7 +331,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.23
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com