appsignal 3.5.4 → 3.5.6

data/spec/lib/appsignal/integrations/sidekiq_spec.rb

@@ -4,7 +4,7 @@ describe Appsignal::Integrations::SidekiqErrorHandler do
   let(:log) { StringIO.new }
   before do
     start_agent
-    Appsignal.logger = test_logger(log)
+    Appsignal.internal_logger = test_logger(log)
   end
   around { |example| keep_transactions { example.run } }
 
@@ -86,7 +86,7 @@ describe Appsignal::Integrations::SidekiqMiddleware, :with_yaml_parse_error => f
   let(:log) { StringIO.new }
   before do
     start_agent
-    Appsignal.logger = test_logger(log)
+    Appsignal.internal_logger = test_logger(log)
   end
   around { |example| keep_transactions { example.run } }
   after :with_yaml_parse_error => false do
@@ -423,7 +423,7 @@ if DependencyHelper.active_job_present?
     end
     around do |example|
       start_agent
-      Appsignal.logger = test_logger(log)
+      Appsignal.internal_logger = test_logger(log)
       ActiveJob::Base.queue_adapter = :sidekiq
 
       class ActiveJobSidekiqTestJob < ActiveJob::Base

data/spec/lib/appsignal/integrations/sinatra_spec.rb

@@ -16,8 +16,8 @@ if DependencyHelper.sinatra_present?
     before { allow(Appsignal).to receive(:active?).and_return(true) }
     after { uninstall_sinatra_integration }
 
-    context "Appsignal.logger" do
-      subject { Appsignal.logger }
+    context "Appsignal.internal_logger" do
+      subject { Appsignal.internal_logger }
 
       it "sets a logger" do
         install_sinatra_integration

data/spec/lib/appsignal/minutely_spec.rb

@@ -42,7 +42,7 @@ describe Appsignal::Minutely do
     let(:log_stream) { StringIO.new }
     let(:log) { log_contents(log_stream) }
     before do
-      Appsignal.logger = test_logger(log_stream)
+      Appsignal.internal_logger = test_logger(log_stream)
       # Speed up test time
       allow(Appsignal::Minutely).to receive(:initial_wait_time).and_return(0.001)
       allow(Appsignal::Minutely).to receive(:wait_time).and_return(0.001)
@@ -287,7 +287,7 @@ describe Appsignal::Minutely do
     describe "#register" do
       let(:log_stream) { std_stream }
       let(:log) { log_contents(log_stream) }
-      before { Appsignal.logger = test_logger(log_stream) }
+      before { Appsignal.internal_logger = test_logger(log_stream) }
 
       it "adds the by key probe" do
         probe = lambda {}

data/spec/lib/appsignal/rack/rails_instrumentation_spec.rb

@@ -6,7 +6,7 @@ if DependencyHelper.rails_present?
     let(:log) { StringIO.new }
     before do
       start_agent
-      Appsignal.logger = test_logger(log)
+      Appsignal.internal_logger = test_logger(log)
     end
 
     let(:params) do

data/spec/lib/appsignal/transaction_spec.rb

@@ -580,7 +580,7 @@ describe Appsignal::Transaction do
       it "does not raise an error when the queue start is too big" do
         expect(transaction.ext).to receive(:set_queue_start).and_raise(RangeError)
 
-        expect(Appsignal.logger).to receive(:warn).with("Queue start value 10 is too big")
+        expect(Appsignal.internal_logger).to receive(:warn).with("Queue start value 10 is too big")
 
         expect do
           transaction.set_queue_start(10)
@@ -766,7 +766,7 @@ describe Appsignal::Transaction do
         let(:error) { Object.new }
 
         it "does not add the error" do
-          expect(Appsignal.logger).to receive(:error).with(
+          expect(Appsignal.internal_logger).to receive(:error).with(
             "Appsignal::Transaction#set_error: Cannot set error. " \
               "The given value is not an exception: #{error.inspect}"
           )
@@ -830,7 +830,7 @@ describe Appsignal::Transaction do
             )
           )
 
-          expect(Appsignal.logger).to_not receive(:debug)
+          expect(Appsignal.internal_logger).to_not receive(:debug)
 
           transaction.set_error(error)
         end
@@ -871,7 +871,7 @@ describe Appsignal::Transaction do
             Appsignal::Utils::Data.generate(expected_error_causes)
           )
 
-          expect(Appsignal.logger).to receive(:debug).with(
+          expect(Appsignal.internal_logger).to receive(:debug).with(
             "Appsignal::Transaction#set_error: Error has more " \
               "than 10 error causes. Only the first 10 " \
               "will be reported."

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
@@ -564,7 +565,7 @@ describe Appsignal do
             "this message."
         end
         before do
-          Appsignal.logger = test_logger(log_stream)
+          Appsignal.internal_logger = test_logger(log_stream)
           capture_std_streams(std_stream, err_stream) { Appsignal.set_host_gauge("key", 0.1) }
         end
 
@@ -586,7 +587,7 @@ describe Appsignal do
             "this message."
         end
         before do
-          Appsignal.logger = test_logger(log_stream)
+          Appsignal.internal_logger = test_logger(log_stream)
           capture_std_streams(std_stream, err_stream) { Appsignal.set_process_gauge("key", 0.1) }
         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.5.4
+  version: 3.5.6
 platform: ruby
 authors:
 - Robert Beekman
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-15 00:00:00.000000000 Z
+date: 2024-02-08 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
@@ -335,6 +339,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
@@ -446,7 +451,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.15
+rubygems_version: 3.4.11
 signing_key:
 specification_version: 4
 summary: Logs performance and exception data from your app to appsignal.com