appsignal 3.4.13 → 3.6.1

data/spec/lib/appsignal_spec.rb

@@ -12,7 +12,7 @@ describe Appsignal do
   describe ".config=" do
     it "should set the config" do
       config = project_fixture_config
-      expect(Appsignal.logger).to_not receive(:level=)
+      expect(Appsignal.internal_logger).to_not receive(:level=)
 
       Appsignal.config = config
       expect(Appsignal.config).to eq config
@@ -22,9 +22,9 @@ describe Appsignal do
   describe ".start" do
     context "with no config set beforehand" do
       it "should do nothing when config is not set and there is no valid config in the env" do
-        expect(Appsignal.logger).to receive(:error)
+        expect(Appsignal.internal_logger).to receive(:error)
           .with("Push API key not set after loading config").once
-        expect(Appsignal.logger).to receive(:error)
+        expect(Appsignal.internal_logger).to receive(:error)
           .with("Not starting, no valid config for this environment").once
         expect(Appsignal::Extension).to_not receive(:start)
         Appsignal.start
@@ -33,7 +33,7 @@ describe Appsignal do
       it "should create a config from the env" do
         ENV["APPSIGNAL_PUSH_API_KEY"] = "something"
         expect(Appsignal::Extension).to receive(:start)
-        expect(Appsignal.logger).not_to receive(:error)
+        expect(Appsignal.internal_logger).not_to receive(:error)
         silence { Appsignal.start }
         expect(Appsignal.config[:push_api_key]).to eq("something")
       end
@@ -44,7 +44,7 @@ describe Appsignal do
 
       it "should initialize logging" do
         Appsignal.start
-        expect(Appsignal.logger.level).to eq Logger::INFO
+        expect(Appsignal.internal_logger.level).to eq Logger::INFO
       end
 
       it "should start native" do
@@ -122,7 +122,7 @@ describe Appsignal do
 
       it "should change the log level" do
         Appsignal.start
-        expect(Appsignal.logger.level).to eq Logger::DEBUG
+        expect(Appsignal.internal_logger.level).to eq Logger::DEBUG
       end
     end
   end
@@ -152,7 +152,7 @@ describe Appsignal do
 
   describe ".stop" do
     it "should call stop on the extension" do
-      expect(Appsignal.logger).to receive(:debug).with("Stopping appsignal")
+      expect(Appsignal.internal_logger).to receive(:debug).with("Stopping appsignal")
       expect(Appsignal::Extension).to receive(:stop)
       Appsignal.stop
       expect(Appsignal.active?).to be_falsy
@@ -160,7 +160,7 @@ describe Appsignal do
 
     context "with context specified" do
       it "should log the context" do
-        expect(Appsignal.logger).to receive(:debug).with("Stopping appsignal (something)")
+        expect(Appsignal.internal_logger).to receive(:debug).with("Stopping appsignal (something)")
         expect(Appsignal::Extension).to receive(:stop)
         Appsignal.stop("something")
         expect(Appsignal.active?).to be_falsy
@@ -222,9 +222,9 @@ describe Appsignal do
         Appsignal.config = project_fixture_config("not_active")
         Appsignal.start
         Appsignal.start_logger
-        Appsignal.logger = test_logger(log_stream)
+        Appsignal.internal_logger = test_logger(log_stream)
       end
-      after { Appsignal.logger = nil }
+      after { Appsignal.internal_logger = nil }
 
       it "should do nothing but still yield the block" do
         expect(Appsignal::Transaction).to_not receive(:create)
@@ -308,9 +308,9 @@ describe Appsignal do
       Appsignal.config = project_fixture_config
       Appsignal.start
       Appsignal.start_logger
-      Appsignal.logger = test_logger(log_stream)
+      Appsignal.internal_logger = test_logger(log_stream)
     end
-    after { Appsignal.logger = nil }
+    after { Appsignal.internal_logger = nil }
 
     describe ".monitor_transaction" do
       context "with a successful call" do
@@ -545,7 +545,8 @@ describe Appsignal do
         it "should not raise an exception when out of range" do
           expect(Appsignal::Extension).to receive(:set_gauge).with("key", 10,
             Appsignal::Extension.data_map_new).and_raise(RangeError)
-          expect(Appsignal.logger).to receive(:warn).with("Gauge value 10 for key 'key' is too big")
+          expect(Appsignal.internal_logger).to receive(:warn)
+            .with("Gauge value 10 for key 'key' is too big")
           expect do
             Appsignal.set_gauge("key", 10)
           end.to_not raise_error
@@ -553,46 +554,46 @@ describe Appsignal do
       end
 
       describe ".set_host_gauge" do
-        it "should call set_host_gauge on the extension with a string key and float" do
-          expect(Appsignal::Extension).to receive(:set_host_gauge).with("key", 0.1)
-          Appsignal.set_host_gauge("key", 0.1)
+        let(:err_stream) { std_stream }
+        let(:stderr) { err_stream.read }
+        let(:log_stream) { StringIO.new }
+        let(:logs) { log_contents(log_stream) }
+        let(:deprecation_message) do
+          "The `set_host_gauge` method has been deprecated. " \
+            "Calling this method has no effect. " \
+            "Please remove method call in the following file to remove " \
+            "this message."
         end
-
-        it "should call set_host_gauge on the extension with a symbol key and int" do
-          expect(Appsignal::Extension).to receive(:set_host_gauge).with("key", 1.0)
-          Appsignal.set_host_gauge(:key, 1)
+        before do
+          Appsignal.internal_logger = test_logger(log_stream)
+          capture_std_streams(std_stream, err_stream) { Appsignal.set_host_gauge("key", 0.1) }
         end
 
-        it "should not raise an exception when out of range" do
-          expect(Appsignal::Extension).to receive(:set_host_gauge).with("key",
-            10).and_raise(RangeError)
-          expect(Appsignal.logger).to receive(:warn)
-            .with("Host gauge value 10 for key 'key' is too big")
-          expect do
-            Appsignal.set_host_gauge("key", 10)
-          end.to_not raise_error
+        it "logs a deprecation warning" do
+          expect(stderr).to include("appsignal WARNING: #{deprecation_message}")
+          expect(logs).to include(deprecation_message)
         end
       end
 
       describe ".set_process_gauge" do
-        it "should call set_process_gauge on the extension with a string key and float" do
-          expect(Appsignal::Extension).to receive(:set_process_gauge).with("key", 0.1)
-          Appsignal.set_process_gauge("key", 0.1)
+        let(:err_stream) { std_stream }
+        let(:stderr) { err_stream.read }
+        let(:log_stream) { StringIO.new }
+        let(:logs) { log_contents(log_stream) }
+        let(:deprecation_message) do
+          "The `set_process_gauge` method has been deprecated. " \
+            "Calling this method has no effect. " \
+            "Please remove method call in the following file to remove " \
+            "this message."
         end
-
-        it "should call set_process_gauge on the extension with a symbol key and int" do
-          expect(Appsignal::Extension).to receive(:set_process_gauge).with("key", 1.0)
-          Appsignal.set_process_gauge(:key, 1)
+        before do
+          Appsignal.internal_logger = test_logger(log_stream)
+          capture_std_streams(std_stream, err_stream) { Appsignal.set_process_gauge("key", 0.1) }
         end
 
-        it "should not raise an exception when out of range" do
-          expect(Appsignal::Extension).to receive(:set_process_gauge).with("key",
-            10).and_raise(RangeError)
-          expect(Appsignal.logger).to receive(:warn)
-            .with("Process gauge value 10 for key 'key' is too big")
-          expect do
-            Appsignal.set_process_gauge("key", 10)
-          end.to_not raise_error
+        it "logs a deprecation warning" do
+          expect(stderr).to include("appsignal WARNING: #{deprecation_message}")
+          expect(logs).to include(deprecation_message)
         end
       end
 
@@ -624,7 +625,7 @@ describe Appsignal do
         it "should not raise an exception when out of range" do
           expect(Appsignal::Extension).to receive(:increment_counter)
             .with("key", 10, Appsignal::Extension.data_map_new).and_raise(RangeError)
-          expect(Appsignal.logger).to receive(:warn)
+          expect(Appsignal.internal_logger).to receive(:warn)
             .with("Counter value 10 for key 'key' is too big")
           expect do
             Appsignal.increment_counter("key", 10)
@@ -654,7 +655,7 @@ describe Appsignal do
         it "should not raise an exception when out of range" do
           expect(Appsignal::Extension).to receive(:add_distribution_value)
             .with("key", 10, Appsignal::Extension.data_map_new).and_raise(RangeError)
-          expect(Appsignal.logger).to receive(:warn)
+          expect(Appsignal.internal_logger).to receive(:warn)
             .with("Distribution value 10 for key 'key' is too big")
           expect do
             Appsignal.add_distribution_value("key", 10)
@@ -663,8 +664,8 @@ describe Appsignal do
       end
     end
 
-    describe ".logger" do
-      subject { Appsignal.logger }
+    describe ".internal_logger" do
+      subject { Appsignal.internal_logger }
 
       it { is_expected.to be_a Logger }
     end
@@ -727,7 +728,7 @@ describe Appsignal do
         let(:error) { double }
 
         it "logs an error message" do
-          expect(Appsignal.logger).to receive(:error).with(
+          expect(Appsignal.internal_logger).to receive(:error).with(
             "Appsignal.send_error: Cannot send error. " \
               "The given value is not an exception: #{error.inspect}"
           )
@@ -915,7 +916,7 @@ describe Appsignal do
           let(:error) { Object.new }
 
           it "logs an error" do
-            expect(Appsignal.logger).to receive(:error).with(
+            expect(Appsignal.internal_logger).to receive(:error).with(
               "Appsignal.set_error: Cannot set error. " \
                 "The given value is not an exception: #{error.inspect}"
             )
@@ -1128,7 +1129,7 @@ describe Appsignal do
         "production",
         :log_path => log_path
       )
-      Appsignal.logger.error("Log in memory")
+      Appsignal.internal_logger.error("Log in memory")
     end
 
     context "when the log path is writable" do
@@ -1139,9 +1140,9 @@ describe Appsignal do
           capture_stdout(out_stream) do
             initialize_config
             Appsignal.start_logger
-            Appsignal.logger.error("Log to file")
+            Appsignal.internal_logger.error("Log to file")
           end
-          expect(Appsignal.logger).to be_a(Appsignal::Utils::IntegrationLogger)
+          expect(Appsignal.internal_logger).to be_a(Appsignal::Utils::IntegrationLogger)
         end
 
         it "logs to file" do
@@ -1163,8 +1164,8 @@ describe Appsignal do
           capture_stdout(out_stream) do
             initialize_config
             Appsignal.start_logger
-            Appsignal.logger.error("Log to not writable log file")
-            expect(Appsignal.logger).to be_a(Appsignal::Utils::IntegrationLogger)
+            Appsignal.internal_logger.error("Log to not writable log file")
+            expect(Appsignal.internal_logger).to be_a(Appsignal::Utils::IntegrationLogger)
           end
         end
 
@@ -1178,8 +1179,9 @@ describe Appsignal do
         end
 
         it "outputs a warning" do
+          puts output
           expect(output).to include \
-            "[WARN] appsignal: Unable to start logger with log path '#{log_file}'.",
+            "[WARN] appsignal: Unable to start internal logger with log path '#{log_file}'.",
             "[WARN] appsignal: Permission denied"
         end
       end
@@ -1193,9 +1195,9 @@ describe Appsignal do
         capture_stdout(out_stream) do
           initialize_config
           Appsignal.start_logger
-          Appsignal.logger.error("Log to not writable log path")
+          Appsignal.internal_logger.error("Log to not writable log path")
         end
-        expect(Appsignal.logger).to be_a(Appsignal::Utils::IntegrationLogger)
+        expect(Appsignal.internal_logger).to be_a(Appsignal::Utils::IntegrationLogger)
       end
       after do
         FileUtils.chmod 0o755, Appsignal::Config.system_tmp_dir
@@ -1222,9 +1224,9 @@ describe Appsignal do
         capture_stdout(out_stream) do
           initialize_config
           Appsignal.start_logger
-          Appsignal.logger.error("Log to stdout")
+          Appsignal.internal_logger.error("Log to stdout")
         end
-        expect(Appsignal.logger).to be_a(Appsignal::Utils::IntegrationLogger)
+        expect(Appsignal.internal_logger).to be_a(Appsignal::Utils::IntegrationLogger)
       end
       around { |example| recognize_as_heroku { example.run } }
 
@@ -1238,7 +1240,7 @@ describe Appsignal do
     end
 
     describe "#logger#level" do
-      subject { Appsignal.logger.level }
+      subject { Appsignal.internal_logger.level }
 
       context "when there is no config" do
         before do

data/spec/spec_helper.rb

@@ -165,7 +165,7 @@ RSpec.configure do |config|
   config.after :context do
     FileUtils.rm_f(File.join(project_fixture_path, "log/appsignal.log"))
     Appsignal.config = nil
-    Appsignal.logger = nil
+    Appsignal.internal_logger = nil
   end
 
   def stop_minutely_probes

data/spec/support/fixtures/projects/valid/config/appsignal.yml

@@ -5,7 +5,7 @@ default: &defaults
     "HTTP_ACCEPT", "HTTP_ACCEPT_CHARSET", "HTTP_ACCEPT_ENCODING",
     "HTTP_ACCEPT_LANGUAGE", "HTTP_CACHE_CONTROL", "HTTP_CONNECTION",
     "CONTENT_LENGTH", "PATH_INFO", "HTTP_RANGE", "HTTP_REFERER",
-    "REQUEST_METHOD", "REQUEST_URI", "SERVER_NAME", "SERVER_PORT",
+    "REQUEST_METHOD", "REQUEST_PATH", "SERVER_NAME", "SERVER_PORT",
     "SERVER_PROTOCOL", "HTTP_USER_AGENT"
   ]
   enable_minutely_probes: false
@@ -33,7 +33,7 @@ old_config:
     "HTTP_ACCEPT", "HTTP_ACCEPT_CHARSET", "HTTP_ACCEPT_ENCODING",
     "HTTP_ACCEPT_LANGUAGE", "HTTP_CACHE_CONTROL", "HTTP_CONNECTION",
     "CONTENT_LENGTH", "PATH_INFO", "HTTP_RANGE", "HTTP_REFERER",
-    "REQUEST_METHOD", "REQUEST_URI", "SERVER_NAME", "SERVER_PORT",
+    "REQUEST_METHOD", "REQUEST_PATH", "SERVER_NAME", "SERVER_PORT",
     "SERVER_PROTOCOL", "HTTP_USER_AGENT"
   ]
 
@@ -49,7 +49,7 @@ old_config_mixed_with_new_config:
     "HTTP_ACCEPT", "HTTP_ACCEPT_CHARSET", "HTTP_ACCEPT_ENCODING",
     "HTTP_ACCEPT_LANGUAGE", "HTTP_CACHE_CONTROL", "HTTP_CONNECTION",
     "CONTENT_LENGTH", "PATH_INFO", "HTTP_RANGE", "HTTP_REFERER",
-    "REQUEST_METHOD", "REQUEST_URI", "SERVER_NAME", "SERVER_PORT",
+    "REQUEST_METHOD", "REQUEST_PATH", "SERVER_NAME", "SERVER_PORT",
     "SERVER_PROTOCOL", "HTTP_USER_AGENT"
   ]
 

data/spec/support/helpers/config_helpers.rb

@@ -5,8 +5,12 @@ module ConfigHelpers
     )
   end
 
-  def project_fixture_config(env = "production", initial_config = {}, logger = Appsignal.logger, # rubocop:disable Metrics/ParameterLists
-    config_file = nil)
+  def project_fixture_config( # rubocop:disable Metrics/ParameterLists
+    env = "production",
+    initial_config = {},
+    logger = Appsignal.internal_logger,
+    config_file = nil
+  )
     Appsignal::Config.new(
       project_fixture_path,
       env,

data/spec/support/helpers/dependency_helper.rb

@@ -1,4 +1,4 @@
-module DependencyHelper
+module DependencyHelper # rubocop:disable Metrics/ModuleLength
   module_function
 
   def ruby_version
@@ -53,6 +53,14 @@ module DependencyHelper
     dependency_present? "resque"
   end
 
+  def redis_client_present?
+    dependency_present? "redis-client"
+  end
+
+  def hiredis_client_present?
+    dependency_present? "hiredis-client"
+  end
+
   def redis_present?
     dependency_present? "redis"
   end

data/spec/support/helpers/log_helpers.rb

@@ -6,9 +6,9 @@ module LogHelpers
   end
 
   def use_logger_with(log)
-    Appsignal.logger = test_logger(log)
+    Appsignal.internal_logger = test_logger(log)
     yield
-    Appsignal.logger = nil
+    Appsignal.internal_logger = nil
   end
 
   def test_logger(log)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appsignal
 version: !ruby/object:Gem::Version
-  version: 3.4.13
+  version: 3.6.1
 platform: ruby
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-10 00:00:00.000000000 Z
+date: 2024-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -174,6 +174,8 @@ files:
 - gemfiles/rails-6.1.gemfile
 - gemfiles/rails-7.0.gemfile
 - gemfiles/rails-7.1.gemfile
+- gemfiles/redis-4.gemfile
+- gemfiles/redis-5.gemfile
 - gemfiles/resque-2.gemfile
 - gemfiles/sequel.gemfile
 - gemfiles/sinatra.gemfile
@@ -226,6 +228,7 @@ files:
 - lib/appsignal/hooks/que.rb
 - lib/appsignal/hooks/rake.rb
 - lib/appsignal/hooks/redis.rb
+- lib/appsignal/hooks/redis_client.rb
 - lib/appsignal/hooks/resque.rb
 - lib/appsignal/hooks/sequel.rb
 - lib/appsignal/hooks/shoryuken.rb
@@ -251,6 +254,7 @@ files:
 - lib/appsignal/integrations/railtie.rb
 - lib/appsignal/integrations/rake.rb
 - lib/appsignal/integrations/redis.rb
+- lib/appsignal/integrations/redis_client.rb
 - lib/appsignal/integrations/resque.rb
 - lib/appsignal/integrations/sidekiq.rb
 - lib/appsignal/integrations/sinatra.rb
@@ -264,6 +268,7 @@ files:
 - lib/appsignal/probes/helpers.rb
 - lib/appsignal/probes/mri.rb
 - lib/appsignal/probes/sidekiq.rb
+- lib/appsignal/rack/body_wrapper.rb
 - lib/appsignal/rack/generic_instrumentation.rb
 - lib/appsignal/rack/rails_instrumentation.rb
 - lib/appsignal/rack/sinatra_instrumentation.rb
@@ -335,6 +340,7 @@ files:
 - spec/lib/appsignal/hooks/puma_spec.rb
 - spec/lib/appsignal/hooks/que_spec.rb
 - spec/lib/appsignal/hooks/rake_spec.rb
+- spec/lib/appsignal/hooks/redis_client_spec.rb
 - spec/lib/appsignal/hooks/redis_spec.rb
 - spec/lib/appsignal/hooks/resque_spec.rb
 - spec/lib/appsignal/hooks/sequel_spec.rb
@@ -361,6 +367,7 @@ files:
 - spec/lib/appsignal/probes/gvl_spec.rb
 - spec/lib/appsignal/probes/mri_spec.rb
 - spec/lib/appsignal/probes/sidekiq_spec.rb
+- spec/lib/appsignal/rack/body_wrapper_spec.rb
 - spec/lib/appsignal/rack/generic_instrumentation_spec.rb
 - spec/lib/appsignal/rack/rails_instrumentation_spec.rb
 - spec/lib/appsignal/rack/sinatra_instrumentation_spec.rb