skylight 7.0.0 → 7.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3776b9d0ac094a7a81dc034b8a0c05d1a9caa55c21c17556a2e9c01e9eeaf3e7
-  data.tar.gz: ca55da2082e52d7ea119cf2c3a6743c28f0d72a475c31768df01f649b6a1c980
+  metadata.gz: fb1aa9e97569dad5f875188d9b9950ce95e2f760cecbf47e56d1a111a3bb3d4b
+  data.tar.gz: 077b6c9517691116e216022f9f5e810edc7173c262a66d99446b49341efbe51a
 SHA512:
-  metadata.gz: b5cda810a0dde1ecb00b676d39400c363ecb3b0c7705b97b096852fbfa74e41b9d52da642e4a5a322874f1e02fb4977563db3ad4042883e3cb3db3a0bec28383
-  data.tar.gz: d466244683b532a00efb3aa48162f12e284be24d6b981e3824deaf473c1f272d0cd731fa3d34ecc0d996b63b600267885dcf130c175638f49e1047d1afbea956
+  metadata.gz: 28a23bdc8e24bd10150a23d917bb669cdebfafb5422046f9c1fe7c0aebee7a12e2f31ca8aee12430e50e32956ef54581b63481f81616fdcf2edecf41ab86ddc2
+  data.tar.gz: c41a8135ee576517a9f1f4f096fc9bb30f024e45e1918a3c19ca21e480f21b9ddecba70527734a660fe9723bc25feffcaae605bc0966d4d19a488b308d66166b

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## 7.1.1 (April 17, 2026)
+- BUGFIX Fix native extension build on GCC 15 by using `append_cflags` instead of mutating `$CFLAGS` directly.
+- BUGFIX Replace K&R-style empty parameter lists (`()`) with `(void)` in C sources to build cleanly under Fedora's `-Werror=old-style-definition`.
+
+## 7.1.0 (January 20, 2025)
+- BREAKING end support for Rails < 7.2.
+- IMPROVEMENT better support for Grape 3.x
+- IMPROVEMENT better ActiveRecord instrumentation for async queries in Rails 8.2
+
 ## 7.0.0 (September 16, 2025)
 
 - IMPROVEMENT Initial (beta) support for AWS Lambda

data/ext/extconf.rb

@@ -228,7 +228,7 @@ end
 #   flag can cause issues for some customers we're turning it off by default. However,
 #   in development and CI, we still have the option of turning it back on to help catch
 #   potential issues.
-$CFLAGS << " -Werror" if SKYLIGHT_EXT_STRICT
+append_cflags(["-Werror"]) if SKYLIGHT_EXT_STRICT
 
 checking_for "fast thread local storage" do
   if try_compile("__thread int foo;")
@@ -238,15 +238,15 @@ checking_for "fast thread local storage" do
 end
 
 # Flag -std=c99 required for older build systems
-$CFLAGS << " -std=c99 -Wall -fno-strict-aliasing"
+append_cflags(["-std=c99", "-Wall", "-fno-strict-aliasing"])
 
 # Allow stricter checks to be turned on for development or debugging
 if SKYLIGHT_EXT_STRICT
-  $CFLAGS << " -Wextra"
+  append_cflags(["-Wextra"])
 
   # Enabling unused-parameter causes failures in Ruby 2.6+
   #   ruby/ruby.h:2186:35: error: unused parameter 'allow_transient'
-  $CFLAGS << " -Wno-error=unused-parameter"
+  append_cflags(["-Wno-error=unused-parameter"])
 end
 
 # TODO: Compute the relative path to the location

data/ext/skylight_memprof.c

@@ -12,7 +12,7 @@ typedef struct {
 // Use the __thread directive
 static __thread sky_allocations_t sky_allocations;
 
-static inline sky_allocations_t* get_allocations() {
+static inline sky_allocations_t* get_allocations(void) {
   return &sky_allocations;
 }
 
@@ -25,11 +25,11 @@ static pthread_key_t ALLOCATIONS_KEY;
 
 static pthread_once_t KEY_INIT_ONCE = PTHREAD_ONCE_INIT;
 
-static void init_allocations_key() {
+static void init_allocations_key(void) {
   pthread_key_create(&ALLOCATIONS_KEY, free);
 }
 
-static sky_allocations_t* get_allocations() {
+static sky_allocations_t* get_allocations(void) {
   sky_allocations_t* ret;
 
   // Initialize the TLS key
@@ -69,7 +69,7 @@ uint64_t sky_allocation_count(void) {
   return get_allocations()->allocations;
 }
 
-uint64_t sky_consume_allocations() {
+uint64_t sky_consume_allocations(void) {
   uint64_t ret = get_allocations()->allocations;
   sky_clear_allocation_count();
   return ret;
@@ -100,7 +100,7 @@ uint64_t sky_allocation_count(void) {
   return 0;
 }
 
-uint64_t sky_consume_allocations() {
+uint64_t sky_consume_allocations(void) {
   return 0;
 }
 

data/ext/skylight_native.c

@@ -21,8 +21,7 @@
 
 #define CHECK_NUMERIC(VAL)                        \
   do {                                            \
-    if (TYPE(VAL) != T_BIGNUM &&                  \
-        TYPE(VAL) != T_FIXNUM) {                  \
+    if (!RB_INTEGER_TYPE_P(VAL)) {                \
       rb_raise(rb_eArgError, "expected " #VAL " to be numeric but was '%s' (%s [%i])", \
                 TO_S(VAL), rb_obj_classname(VAL), TYPE(VAL)); \
       return Qnil;                                \
@@ -554,7 +553,7 @@ trace_span_set_exception(VALUE self, VALUE span, VALUE exception, VALUE exceptio
   return Qnil;
 }
 
-void Init_skylight_native() {
+void Init_skylight_native(void) {
   rb_mSkylight = rb_define_module("Skylight");
 
   rb_eNativeError = rb_const_get(rb_mSkylight, rb_intern("NativeError"));

data/lib/skylight/config.rb

@@ -19,9 +19,7 @@ module Skylight
     # @api private
     MUTEX = Mutex.new
 
-    DEFAULT_IGNORED_ENDPOINTS = %w[
-      Rails::HealthController#show
-    ].freeze
+    DEFAULT_IGNORED_ENDPOINTS = %w[Rails::HealthController#show].freeze
 
     # Map environment variable keys with Skylight configuration keys
     ENV_TO_KEY = {

data/lib/skylight/native_ext_fetcher.rb

@@ -148,12 +148,25 @@ module Skylight
         p_user, p_pass = uri.userinfo.split(/:/) if uri.userinfo
       end
 
-      opts = {}
-      opts[:use_ssl] = use_ssl
+      opts = { use_ssl: use_ssl }
 
-      opts[:ca_file] = Util::SSL.ca_cert_file_or_default if use_ssl
+      if use_ssl
+        # Create a cert store that doesn't enable CRL checking.
+        # OpenSSL 3.x may enable CRL checking by default, which fails when
+        # the CRL distribution point is unreachable.
+        begin
+          cert_store = OpenSSL::X509::Store.new
+          cert_store.set_default_paths
+          ca_file = Util::SSL.ca_cert_file_or_default
+          cert_store.add_file(ca_file) if ca_file && File.exist?(ca_file)
+          opts[:cert_store] = cert_store
+        rescue OpenSSL::X509::StoreError => e
+          log "failed to configure cert store: #{e.message}, using defaults"
+        end
+        opts[:verify_mode] = OpenSSL::SSL::VERIFY_PEER
+      end
 
-      Net::HTTP.start(host, port, p_host, p_port, p_user, p_pass, use_ssl: use_ssl) do |http|
+      Net::HTTP.start(host, port, p_host, p_port, p_user, p_pass, **opts) do |http|
         http.request_get path do |resp|
           case resp
           when Net::HTTPSuccess

data/lib/skylight/normalizers/active_record/sql.rb

@@ -12,7 +12,7 @@ module Skylight
         register "future_result.active_record"
 
         def normalize(_trace, _name, payload)
-          ["db.future_result", "Async #{payload[:args][1] || "Query"}"]
+          ["db.future_result", "Async #{payload[:name] || "Query"}"]
         end
       end
     end

data/lib/skylight/normalizers/grape/endpoint.rb

@@ -19,8 +19,13 @@ module Skylight
         end
 
         def get_namespace(endpoint)
-          # slice off preceding slash for data continuity
-          ::Grape::Namespace.joined_space_path(endpoint.namespace_stackable(:namespace)).to_s[1..]
+          if endpoint.respond_to?(:namespace_stackable)
+            # Grape 2.x
+            ::Grape::Namespace.joined_space_path(endpoint.namespace_stackable(:namespace)).to_s[1..]
+          else
+            # Grape 3.x
+            endpoint.namespace.to_s[1..]
+          end
         end
       end
     end

data/lib/skylight/normalizers/grape/endpoint_run.rb

@@ -31,7 +31,12 @@ module Skylight
           ep = endpoint.options[:for]
           return ep.name if ep.name
 
-          ep.base.name if ep.respond_to?(:base) && ep.base.respond_to?(:name)
+          if ep.respond_to?(:base) && ep.base.respond_to?(:name)
+            ep.base.name
+          elsif ep.respond_to?(:to_s)
+            # grape >= 3.1 removes the `base` attr_reader but delegates to_s to :@base
+            ep.to_s
+          end
         end
       end
     end

data/lib/skylight/probes/active_record_async.rb

@@ -38,29 +38,27 @@ module Skylight
           end
         end
 
-        # Applied to FutureResult
-        module Instrumentation
+        # Applied to FutureResult (Rails <= 8.1)
+        # Handles both the outer instrumentation and the execute_or_wait hook
+        module FutureResultInstrumentation
           def result(*, **)
-            # This instruments the whole FutureResult so that we know when we were executing (potenially) async.
-            ActiveSupport::Notifications.instrument("future_result.active_record", { args: @args, kwargs: @kwargs }) do
-              super
-            end
+            name = @args&.[](1)
+
+            ActiveSupport::Notifications.instrument("future_result.active_record", { name: name }) { super }
           end
 
           private
 
           def execute_or_wait(*, **)
-            # At this point we're actually waiting for the query to have finished executing.
-
             begin
               # If the query has already started async, the @event_buffer will be defined.
               # We grab the events (currently only the SQL queries), extend them with our
               # special methods and notify Skylight.
-              # We act as if the event has just stared, though the query may already have been
+              # We act as if the event has just started, though the query may already have been
               # running. This means we're essentially just logging blocking time right now.
 
               # Dup here just in case more get added somehow during the super call
-              events = @event_buffer&.instance_variable_get(:@events).dup
+              events = @event_buffer&.instance_variable_get(:@events)&.dup
 
               events&.each do |event|
                 event.singleton_class.prepend(AsyncEventExtensions)
@@ -78,9 +76,48 @@ module Skylight
           end
         end
 
+        # Applied to FutureResult (Rails >= 8.2)
+        module FutureResultInstrumentationRails82
+          def result(*, **)
+            name = @intent&.name
+
+            ActiveSupport::Notifications.instrument("future_result.active_record", { name: name }) { super }
+          end
+        end
+
+        # Applied to QueryIntent (Rails >= 8.2)
+        module QueryIntentInstrumentation
+          private
+
+          def execute_or_wait(*, **)
+            begin
+              # If the query has already started async, the @event_buffer will be defined.
+              events = @event_buffer&.instance_variable_get(:@events)&.dup
+
+              events&.each do |event|
+                event.singleton_class.prepend(AsyncEventExtensions)
+                event.__sk_start!
+              end
+            rescue StandardError => e
+              Skylight.error("Unable to start events for QueryIntent: #{e}")
+            end
+
+            super
+          ensure
+            events&.reverse_each(&:__sk_finish!)
+          end
+        end
+
         class Probe
           def install
-            ::ActiveRecord::FutureResult.prepend(Instrumentation)
+            if defined?(::ActiveRecord::ConnectionAdapters::QueryIntent)
+              # Rails >= 8.2
+              ::ActiveRecord::FutureResult.prepend(FutureResultInstrumentationRails82)
+              ::ActiveRecord::ConnectionAdapters::QueryIntent.prepend(QueryIntentInstrumentation)
+            else
+              # Rails <= 8.1
+              ::ActiveRecord::FutureResult.prepend(FutureResultInstrumentation)
+            end
           end
         end
       end

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 = "7.0.0".freeze
+  VERSION = "7.1.1".freeze
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: skylight
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.1.1
 platform: ruby
 authors:
 - Tilde, Inc.
 bindir: bin
 cert_chain: []
-date: 2025-09-16 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 7.1.0
+        version: 7.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 7.1.0
+        version: 7.2.0
 - !ruby/object:Gem::Dependency
   name: beefcake
   requirement: !ruby/object:Gem::Requirement
@@ -85,14 +85,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 13.0.1
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 13.0.1
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rake-compiler
   requirement: !ruby/object:Gem::Requirement
@@ -205,6 +205,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: ostruct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 email:
 - engineering@tilde.io
 executables:
@@ -398,7 +412,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Skylight is a smart profiler for Rails, Sinatra, and other Ruby apps.
 test_files: []