skylight 6.0.0.beta → 6.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff8e9cf759364d5b113994e6b12046ff5664fbf5d35bcaf54171b82d813c2458
-  data.tar.gz: cd673dab2a53c62e1103cdc5bba16ccecc7b2fe185247bd9593b774738de2341
+  metadata.gz: 98bbb52dcc3879f40677c3cb97ea2b17e6a64604b8ff4bc4d79dafdb63a4664b
+  data.tar.gz: 6be670d8453346d3f4b6a72de3debfd03741059d4373e12ca4cdb8bc0272727e
 SHA512:
-  metadata.gz: 841c15efa8d3a99bc248058a01f6c644d5a16302fb964317576bc6fb2e71ad0aa47fc907961821144b5d168ba4dfdfd85d30701a8650532d04fbfdf917e6cf4b
-  data.tar.gz: 2471c1d1c366136312aac9c3b83a1e669400157da6a3e11437966e151f53d0ae5b449b5f4a1e4b513cd802da07d6ab2a92417cc544e9ab93a798e67f59f7e73a
+  metadata.gz: 8820d77b664d80a35e01f2277a8e89535ccda401bad9b9ac2a143bf0810439da4512d2fe35a46a5c500a7ce87eb8399406e5a28d399073e608d40c666b9c4033
+  data.tar.gz: af18513abf5c96509ece88f289ee6d5ffa0380036f9137cc08e6f67850f9e76584a8eaa1a79228be26a603e9681571ec67b8ba934a33ac24fa0181a59e68052a

data/CHANGELOG.md

@@ -1,4 +1,4 @@
-## 6.0.0-beta (prerelease)
+## 6.0.0 (September 11, 2023)
 - [BREAKING] End support for Ruby 2.6
 - The following libraries are no longer tested and are not guaranteed to work with Skylight 6:
   - Sinatra 1.x
@@ -11,6 +11,9 @@
   responder (like a Rails router), Skylight will not set the endpoint name after exception processing has started.
   This means that error traces will now be aggregated under the original endpoint that generated the error (but with
   the 'error' segment), rather than under the exception handler's controller and action name.
+- [IMPROVEMENT] (Once again) Provide native support for FreeBSD.
+- [BUGFIX] Fix an issue in which the daemon could time out if the processes file descriptor count was set very high (e.g. on Heroku's Performance dynos)
+- [IMPROVEMENT] Better support for GraphQL versions >= 2.0.18.
 
 ## 5.3.4 (October 17, 2022)
 

data/lib/skylight/probes/graphql.rb

@@ -11,20 +11,67 @@ module Skylight
 
           return unless defined?(@tracers)
 
+          # This is the legacy tracing used in graphql =< 2.0.17
           unless @tracers.include?(::GraphQL::Tracing::ActiveSupportNotificationsTracing)
             @tracers << ::GraphQL::Tracing::ActiveSupportNotificationsTracing
           end
         end
       end
 
+      module InstrumentationV2
+        def self.included(base)
+          base.singleton_class.prepend ClassMethods
+        end
+
+        # GraphQL versions 2.0.18 - 2.0.21 (or higher?) were missing this notification
+        module ExecuteMultiplexNotification
+          def execute_multiplex(**metadata, &blk)
+            if @notifications_engine
+              @notifications_engine.instrument("execute_multiplex.graphql", metadata, &blk)
+            else
+              # safety fallback in case graphql's authors unexpectedly rename @notifications_engine
+              super
+            end
+          end
+        end
+
+        module ClassMethods
+          def new_trace(*, **)
+            unless @__sk_instrumentation_installed
+              trace_with(::GraphQL::Tracing::ActiveSupportNotificationsTrace)
+
+              unless ::GraphQL::Tracing::ActiveSupportNotificationsTrace.instance_methods.include?(:execute_multiplex)
+                trace_with(ExecuteMultiplexNotification)
+              end
+
+              @__sk_instrumentation_installed = true
+            end
+
+            super
+          end
+        end
+      end
+
       class Probe
         def install
-          tracing_klass_name = "::GraphQL::Tracing::ActiveSupportNotificationsTracing"
-          klasses_to_probe = %w[::GraphQL::Execution::Multiplex ::GraphQL::Query]
+          new_tracing = false
+          begin
+            require "graphql/tracing/active_support_notifications_trace"
+            new_tracing = true
+          rescue LoadError # rubocop:disable Lint/SuppressedException
+          end
+
+          if new_tracing
+            # GraphQL >= 2.0.18
+            ::GraphQL::Schema.include(InstrumentationV2)
+          else
+            tracing_klass_name = "::GraphQL::Tracing::ActiveSupportNotificationsTracing"
+            klasses_to_probe = %w[::GraphQL::Execution::Multiplex ::GraphQL::Query]
 
-          return unless ([tracing_klass_name] + klasses_to_probe).all?(&method(:safe_constantize))
+            return unless ([tracing_klass_name] + klasses_to_probe).all?(&method(:safe_constantize))
 
-          klasses_to_probe.each { |klass_name| safe_constantize(klass_name).prepend(Instrumentation) }
+            klasses_to_probe.each { |klass_name| safe_constantize(klass_name).prepend(Instrumentation) }
+          end
         end
 
         def safe_constantize(klass_name)

data/lib/skylight/version.rb

@@ -3,5 +3,5 @@ module Skylight
   # for compatibility with semver when it is parsed by the rust agent.
   # This string will be transformed in the gemspec to "5.0.0.alpha"
   # to conform with rubygems.
-  VERSION = "6.0.0-beta".freeze
+  VERSION = "6.0.0".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: skylight
 version: !ruby/object:Gem::Version
-  version: 6.0.0.beta
+  version: 6.0.0
 platform: ruby
 authors:
 - Tilde, Inc.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-03 00:00:00.000000000 Z
+date: 2023-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -396,9 +396,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.7'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.4.3
 signing_key: