RubyGems - newrelic_rpm - Versions diffs - 8.9.0 → 8.10.0 - Mend

newrelic_rpm 8.9.0 → 8.10.0

Files changed (361) hide show

data/lib/new_relic/rack/browser_monitoring.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 require 'rack'
 require 'new_relic/rack/agent_middleware'
@@ -107,7 +108,7 @@ module NewRelic::Rack
             js_to_inject <<
             source[insertion_index..-1]
         else
-          NewRelic::Agent.logger.debug "Skipping RUM instrumentation. Could not properly determine location to inject script."
+          NewRelic::Agent.logger.debug("Skipping RUM instrumentation. Could not properly determine location to inject script.")
         end
       else
         msg = "Skipping RUM instrumentation. Unable to find <body> tag in first #{SCAN_LIMIT} bytes of document."
@@ -117,7 +118,7 @@ module NewRelic::Rack
       source
     rescue => e
-      NewRelic::Agent.logger.debug "Skipping RUM instrumentation on exception.", e
+      NewRelic::Agent.logger.debug("Skipping RUM instrumentation on exception.", e)
       nil
     end

data/lib/new_relic/rack.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 # @api public
 module NewRelic

data/lib/new_relic/recipes/capistrano3.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 require 'capistrano/framework'
@@ -52,14 +53,14 @@ namespace :newrelic do
         :license_key => license_key
       }
-      debug "Uploading deployment to New Relic"
-      deployment = NewRelic::Cli::Deployments.new deploy_options
+      debug("Uploading deployment to New Relic")
+      deployment = NewRelic::Cli::Deployments.new(deploy_options)
       deployment.run
-      info "Uploaded deployment information to New Relic"
+      info("Uploaded deployment information to New Relic")
     rescue NewRelic::Cli::Command::CommandFailure => e
-      info e.message
+      info(e.message)
     rescue => e
-      info "Error creating New Relic deployment (#{e})\n#{e.backtrace.join("\n")}"
+      info("Error creating New Relic deployment (#{e})\n#{e.backtrace.join("\n")}")
     end
   end
@@ -68,7 +69,7 @@ namespace :newrelic do
     current_revision = fetch(:current_revision)
     return unless current_revision && previous_revision
-    debug "Retrieving changelog for New Relic Deployment details"
+    debug("Retrieving changelog for New Relic Deployment details")
     if Rake::Task.task_defined?("git:check")
       log_command = "git --no-pager log --no-color --pretty=format:'  * %an: %s' " +

data/lib/new_relic/recipes/capistrano_legacy.rb CHANGED Viewed

@@ -1,12 +1,13 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 make_notify_task = Proc.new do
-  namespace :newrelic do
+  namespace(:newrelic) do
     # on all deployments, notify New Relic
-    desc "Record a deployment in New Relic (newrelic.com)"
-    task :notice_deployment, :roles => :app, :except => {:no_release => true} do
+    desc("Record a deployment in New Relic (newrelic.com)")
+    task(:notice_deployment, :roles => :app, :except => {:no_release => true}) do
       rails_env = fetch(:newrelic_rails_env, fetch(:rails_env, "production"))
       require 'new_relic/cli/command'
@@ -36,22 +37,22 @@ make_notify_task = Proc.new do
           :license_key => license_key
         }
-        logger.debug "Uploading deployment to New Relic"
-        deployment = NewRelic::Cli::Deployments.new deploy_options
+        logger.debug("Uploading deployment to New Relic")
+        deployment = NewRelic::Cli::Deployments.new(deploy_options)
         deployment.run
-        logger.info "Uploaded deployment information to New Relic"
+        logger.info("Uploaded deployment information to New Relic")
       rescue NewRelic::Cli::Command::CommandFailure => e
-        logger.info e.message
+        logger.info(e.message)
       rescue Capistrano::CommandError
-        logger.info "Unable to notify New Relic of the deployment... skipping"
+        logger.info("Unable to notify New Relic of the deployment... skipping")
       rescue => e
-        logger.info "Error creating New Relic deployment (#{e})\n#{e.backtrace.join("\n")}"
+        logger.info("Error creating New Relic deployment (#{e})\n#{e.backtrace.join("\n")}")
       end
     end
     def lookup_changelog(changelog)
       if !changelog
-        logger.debug "Getting log of changes for New Relic Deployment details"
+        logger.debug("Getting log of changes for New Relic Deployment details")
         from_revision = source.next_revision(current_revision)
         if scm == :git
@@ -69,7 +70,7 @@ make_notify_task = Proc.new do
     def lookup_rev(rev)
       if rev.nil?
         rev = source.query_revision(source.head()) do |cmd|
-          logger.debug "executing locally: '#{cmd}'"
+          logger.debug("executing locally: '#{cmd}'")
           `#{cmd}`
         end

data/lib/new_relic/recipes.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 # When installed as a plugin, this is loaded automatically.
 #

data/lib/new_relic/supportability_helper.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 module NewRelic
   module SupportabilityHelper
@@ -62,13 +63,13 @@ module NewRelic
     def record_api_supportability_metric(method_name)
       agent = NewRelic::Agent.agent or return
       if metric = API_SUPPORTABILITY_METRICS[method_name]
-        agent.stats_engine.tl_record_unscoped_metrics metric, &:increment_count
+        agent.stats_engine.tl_record_unscoped_metrics(metric, &:increment_count)
       else
-        NewRelic::Agent.logger.debug "API supportability metric not found for :#{method_name}"
+        NewRelic::Agent.logger.debug("API supportability metric not found for :#{method_name}")
       end
     end
-    def valid_api_argument_class? arg, name, klass
+    def valid_api_argument_class?(arg, name, klass)
       return true if arg.is_a?(klass)
       caller_location = caller_locations.first.label
@@ -76,7 +77,7 @@ module NewRelic
       message = "Bad argument passed to ##{caller_location}. " \
         "Expected #{klass} for `#{name}` but got #{arg.class}"
-      NewRelic::Agent.logger.warn message
+      NewRelic::Agent.logger.warn(message)
       nil
     end
   end

data/lib/new_relic/version.rb CHANGED Viewed

@@ -2,11 +2,12 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 module NewRelic
   module VERSION # :nodoc:
     MAJOR = 8
-    MINOR = 9
+    MINOR = 10
     TINY = 0
     STRING = "#{MAJOR}.#{MINOR}.#{TINY}"

data/lib/newrelic_rpm.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 # == New Relic Initialization
 #

data/lib/sequel/extensions/newrelic_instrumentation.rb CHANGED Viewed

@@ -2,6 +2,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 require 'sequel' unless defined?(Sequel)
 require 'newrelic_rpm' unless defined?(NewRelic)
@@ -64,16 +65,16 @@ module Sequel
     # the statements that come from the disable_all_tracing block into this node's
     # statement, otherwise we would end up ovewriting the sql statement with the
     # last one executed.
-    def notice_sql sql
+    def notice_sql(sql)
       return unless txn = NewRelic::Agent::Tracer.current_transaction
       current_segment = txn.current_segment
       return unless current_segment.is_a?(NewRelic::Agent::Transaction::DatastoreSegment)
       if current_segment.sql_statement
-        current_segment.sql_statement.append_sql sql
+        current_segment.sql_statement.append_sql(sql)
       else
-        current_segment._notice_sql sql, self.opts, explainer_for(sql)
+        current_segment._notice_sql(sql, self.opts, explainer_for(sql))
       end
     end
@@ -81,7 +82,7 @@ module Sequel
       (defined?(::Sequel::ThreadedConnectionPool) && ::Sequel::ThreadedConnectionPool)
     ].freeze
-    def explainer_for sql
+    def explainer_for(sql)
       Proc.new do |*|
         if THREAD_SAFE_CONNECTION_POOL_CLASSES.include?(self.pool.class)
           self[sql].explain
@@ -93,6 +94,6 @@ module Sequel
     end
   end # module NewRelicInstrumentation
-  NewRelic::Agent.logger.debug "Registering the :newrelic_instrumentation extension."
+  NewRelic::Agent.logger.debug("Registering the :newrelic_instrumentation extension.")
   Database.register_extension(:newrelic_instrumentation, NewRelicInstrumentation)
 end # module Sequel

data/lib/sequel/plugins/newrelic_instrumentation.rb CHANGED Viewed

@@ -2,6 +2,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 require 'sequel' unless defined?(Sequel)
 require 'newrelic_rpm' unless defined?(NewRelic)

data/lib/tasks/all.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 # This is required to load in task definitions
 Dir.glob(File.join(File.dirname(__FILE__), '*.rake')) do |file|

data/lib/tasks/config.rake CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 namespace :newrelic do
   namespace :config do
@@ -122,7 +123,7 @@ namespace :newrelic do
     end
     def format_env_var(key)
-      return "None" if NON_ENV_CONFIGS.include? key
+      return "None" if NON_ENV_CONFIGS.include?(key)
       "NEW_RELIC_#{key.gsub(".", "_").upcase}"
     end

data/lib/tasks/coverage_report.rake CHANGED Viewed

@@ -1,4 +1,5 @@
 # encoding: utf-8
+# frozen_string_literal: true
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
@@ -11,7 +12,7 @@ namespace :coverage do
       puts 'This task is intended to be run only on the CI.'
       return
     end
-    SimpleCov.collate Dir["coverage*/.resultset.json"] do
+    SimpleCov.collate(Dir["coverage*/.resultset.json"]) do
       formatter SimpleCov::Formatter::MultiFormatter.new([
         SimpleCov::Formatter::JSONFormatter,
         SimpleCov::Formatter::HTMLFormatter

data/lib/tasks/install.rake CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 namespace :newrelic do
   desc "Install a default config/newrelic.yml file"

data/lib/tasks/multiverse.rake CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 # Guts of task are in a .rb file to allow requiring from external gems
 require "#{File.join(File.dirname(__FILE__))}/multiverse"

data/lib/tasks/multiverse.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 #
 # Rake task for running Ruby agent multiverse tests. This file may be required
 # from third party gems. It is also used by the agent itself to run multiverse.
@@ -55,18 +56,18 @@ namespace :test do
     end
     def remove_generated_gemfiles
-      file_path = File.expand_path "test/multiverse/suites"
-      Dir.glob(File.join file_path, "**", "Gemfile*").each do |fn|
+      file_path = File.expand_path("test/multiverse/suites")
+      Dir.glob(File.join(file_path, "**", "Gemfile*")).each do |fn|
         puts "Removing #{fn.gsub(file_path, '.../suites')}"
-        FileUtils.rm fn
+        FileUtils.rm(fn)
       end
     end
     def remove_generated_gemfile_lockfiles
-      file_path = File.expand_path "test/environments"
-      Dir.glob(File.join file_path, "**", "Gemfile.lock").each do |fn|
+      file_path = File.expand_path("test/environments")
+      Dir.glob(File.join(file_path, "**", "Gemfile.lock")).each do |fn|
         puts "Removing #{fn.gsub(file_path, '.../environments')}"
-        FileUtils.rm fn
+        FileUtils.rm(fn)
       end
     end
@@ -83,7 +84,7 @@ namespace :test do
     desc "Clean cached gemfiles from Bundler.bundle_path"
     task :clean_gemfile_cache do
-      glob = File.expand_path 'multiverse-cache/Gemfile.*.lock', Bundler.bundle_path
+      glob = File.expand_path('multiverse-cache/Gemfile.*.lock', Bundler.bundle_path)
       File.delete(*Dir[glob])
     end

data/lib/tasks/newrelic.rb CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 #
 # If you are looking to instrument rake tasks in a Rails environment, but the
 # task doesn't depend on :environment, this task may be included to ensure that

data/lib/tasks/tests.rake CHANGED Viewed

@@ -1,6 +1,7 @@
 # encoding: utf-8
 # This file is distributed under New Relic's license terms.
 # See https://github.com/newrelic/newrelic-ruby-agent/blob/main/LICENSE for complete details.
+# frozen_string_literal: true
 begin
   require 'rake/testtask'