skylight 5.0.0.beta5 → 5.1.0.beta3

data/lib/skylight/config.rb

@@ -22,111 +22,96 @@ module Skylight
     # Map environment variable keys with Skylight configuration keys
     ENV_TO_KEY = {
       # == Authentication ==
-      -"AUTHENTICATION"               => :authentication,
-
+      -"AUTHENTICATION" => :authentication,
       # == App settings ==
-      -"ROOT"                         => :root,
-      -"HOSTNAME"                     => :hostname,
-      -"SESSION_TOKEN"                => :session_token,
-
+      -"ROOT" => :root,
+      -"HOSTNAME" => :hostname,
+      -"SESSION_TOKEN" => :session_token,
       # == Component settings ==
-      -"ENV"                          => :env,
-      -"COMPONENT"                    => :component,
-      -"REPORT_RAILS_ENV"             => :report_rails_env,
-
+      -"ENV" => :env,
+      -"COMPONENT" => :component,
+      -"REPORT_RAILS_ENV" => :report_rails_env,
       # == Deploy settings ==
-      -"DEPLOY_ID"                    => :'deploy.id',
-      -"DEPLOY_GIT_SHA"               => :'deploy.git_sha',
-      -"DEPLOY_DESCRIPTION"           => :'deploy.description',
-
+      -"DEPLOY_ID" => :'deploy.id',
+      -"DEPLOY_GIT_SHA" => :'deploy.git_sha',
+      -"DEPLOY_DESCRIPTION" => :'deploy.description',
       # == Logging ==
-      -"LOG_FILE"                     => :log_file,
-      -"LOG_LEVEL"                    => :log_level,
-      -"ALERT_LOG_FILE"               => :alert_log_file,
-      -"NATIVE_LOG_FILE"              => :native_log_file,
-      -"LOG_SQL_PARSE_ERRORS"         => :log_sql_parse_errors,
-
+      -"LOG_FILE" => :log_file,
+      -"LOG_LEVEL" => :log_level,
+      -"ALERT_LOG_FILE" => :alert_log_file,
+      -"NATIVE_LOG_FILE" => :native_log_file,
+      -"NATIVE_LOG_LEVEL" => :native_log_level,
+      -"LOG_SQL_PARSE_ERRORS" => :log_sql_parse_errors,
       # == Proxy ==
-      -"PROXY_URL"                    => :proxy_url,
-
+      -"PROXY_URL" => :proxy_url,
       # == Instrumenter ==
-      -"ENABLE_SEGMENTS"              => :enable_segments,
-      -"ENABLE_SIDEKIQ"               => :enable_sidekiq,
-      -"IGNORED_ENDPOINT"             => :ignored_endpoint,
-      -"IGNORED_ENDPOINTS"            => :ignored_endpoints,
-      -"SINATRA_ROUTE_PREFIXES"       => :sinatra_route_prefixes,
-      -"ENABLE_SOURCE_LOCATIONS"      => :enable_source_locations,
-
+      -"ENABLE_SEGMENTS" => :enable_segments,
+      -"ENABLE_SIDEKIQ" => :enable_sidekiq,
+      -"IGNORED_ENDPOINT" => :ignored_endpoint,
+      -"IGNORED_ENDPOINTS" => :ignored_endpoints,
+      -"SINATRA_ROUTE_PREFIXES" => :sinatra_route_prefixes,
+      -"ENABLE_SOURCE_LOCATIONS" => :enable_source_locations,
       # == Max Span Handling ==
-      -"REPORT_MAX_SPANS_EXCEEDED"    => :report_max_spans_exceeded,
-      -"PRUNE_LARGE_TRACES"           => :prune_large_traces,
-
+      -"REPORT_MAX_SPANS_EXCEEDED" => :report_max_spans_exceeded,
+      -"PRUNE_LARGE_TRACES" => :prune_large_traces,
       # == Skylight Remote ==
-      -"AUTH_URL"                     => :auth_url,
-      -"APP_CREATE_URL"               => :app_create_url,
-      -"MERGES_URL"                   => :merges_url,
-      -"VALIDATION_URL"               => :validation_url,
-      -"AUTH_HTTP_DEFLATE"            => :auth_http_deflate,
-      -"AUTH_HTTP_CONNECT_TIMEOUT"    => :auth_http_connect_timeout,
-      -"AUTH_HTTP_READ_TIMEOUT"       => :auth_http_read_timeout,
-      -"REPORT_URL"                   => :report_url,
-      -"REPORT_HTTP_DEFLATE"          => :report_http_deflate,
-      -"REPORT_HTTP_CONNECT_TIMEOUT"  => :report_http_connect_timeout,
-      -"REPORT_HTTP_READ_TIMEOUT"     => :report_http_read_timeout,
-      -"REPORT_HTTP_DISABLED"         => :report_http_disabled,
-
+      -"AUTH_URL" => :auth_url,
+      -"APP_CREATE_URL" => :app_create_url,
+      -"MERGES_URL" => :merges_url,
+      -"VALIDATION_URL" => :validation_url,
+      -"AUTH_HTTP_DEFLATE" => :auth_http_deflate,
+      -"AUTH_HTTP_CONNECT_TIMEOUT" => :auth_http_connect_timeout,
+      -"AUTH_HTTP_READ_TIMEOUT" => :auth_http_read_timeout,
+      -"REPORT_URL" => :report_url,
+      -"REPORT_HTTP_DEFLATE" => :report_http_deflate,
+      -"REPORT_HTTP_CONNECT_TIMEOUT" => :report_http_connect_timeout,
+      -"REPORT_HTTP_READ_TIMEOUT" => :report_http_read_timeout,
+      -"REPORT_HTTP_DISABLED" => :report_http_disabled,
       # == Native agent settings ==
       #
-      -"LAZY_START"                   => :'daemon.lazy_start',
-      -"DAEMON_EXEC_PATH"             => :'daemon.exec_path',
-      -"DAEMON_LIB_PATH"              => :'daemon.lib_path',
-      -"PIDFILE_PATH"                 => :'daemon.pidfile_path',
-      -"SOCKDIR_PATH"                 => :'daemon.sockdir_path',
-      -"BATCH_QUEUE_DEPTH"            => :'daemon.batch_queue_depth',
-      -"BATCH_SAMPLE_SIZE"            => :'daemon.batch_sample_size',
-      -"BATCH_FLUSH_INTERVAL"         => :'daemon.batch_flush_interval',
-      -"DAEMON_TICK_INTERVAL"         => :'daemon.tick_interval',
-      -"DAEMON_LOCK_CHECK_INTERVAL"   => :'daemon.lock_check_interval',
-      -"DAEMON_INACTIVITY_TIMEOUT"    => :'daemon.inactivity_timeout',
-      -"CLIENT_MAX_TRIES"             => :'daemon.max_connect_tries',
-      -"CLIENT_CONN_TRY_WIN"          => :'daemon.connect_try_window',
-      -"MAX_PRESPAWN_JITTER"          => :'daemon.max_prespawn_jitter',
-      -"DAEMON_WAIT_TIMEOUT"          => :'daemon.wait_timeout',
-      -"CLIENT_CHECK_INTERVAL"        => :'daemon.client_check_interval',
-      -"CLIENT_QUEUE_DEPTH"           => :'daemon.client_queue_depth',
-      -"CLIENT_WRITE_TIMEOUT"         => :'daemon.client_write_timeout',
-      -"SSL_CERT_PATH"                => :'daemon.ssl_cert_path',
-      -"SSL_CERT_DIR"                 => :'daemon.ssl_cert_dir',
-
+      -"LAZY_START" => :'daemon.lazy_start',
+      -"DAEMON_EXEC_PATH" => :'daemon.exec_path',
+      -"DAEMON_LIB_PATH" => :'daemon.lib_path',
+      -"PIDFILE_PATH" => :'daemon.pidfile_path',
+      -"SOCKDIR_PATH" => :'daemon.sockdir_path',
+      -"BATCH_QUEUE_DEPTH" => :'daemon.batch_queue_depth',
+      -"BATCH_SAMPLE_SIZE" => :'daemon.batch_sample_size',
+      -"BATCH_FLUSH_INTERVAL" => :'daemon.batch_flush_interval',
+      -"DAEMON_TICK_INTERVAL" => :'daemon.tick_interval',
+      -"DAEMON_LOCK_CHECK_INTERVAL" => :'daemon.lock_check_interval',
+      -"DAEMON_INACTIVITY_TIMEOUT" => :'daemon.inactivity_timeout',
+      -"CLIENT_MAX_TRIES" => :'daemon.max_connect_tries',
+      -"CLIENT_CONN_TRY_WIN" => :'daemon.connect_try_window',
+      -"MAX_PRESPAWN_JITTER" => :'daemon.max_prespawn_jitter',
+      -"DAEMON_WAIT_TIMEOUT" => :'daemon.wait_timeout',
+      -"CLIENT_CHECK_INTERVAL" => :'daemon.client_check_interval',
+      -"CLIENT_QUEUE_DEPTH" => :'daemon.client_queue_depth',
+      -"CLIENT_WRITE_TIMEOUT" => :'daemon.client_write_timeout',
+      -"SSL_CERT_PATH" => :'daemon.ssl_cert_path',
+      -"ENABLE_TCP" => :'daemon.enable_tcp',
+      -"TCP_PORT" => :'daemon.tcp_port',
       # == Legacy env vars ==
       #
-      -"AGENT_LOCKFILE"               => :'agent.lockfile',
-      -"AGENT_SOCKFILE_PATH"          => :'agent.sockfile_path',
-
+      -"AGENT_LOCKFILE" => :'agent.lockfile',
+      -"AGENT_SOCKFILE_PATH" => :'agent.sockfile_path',
       # == User config settings ==
-      -"USER_CONFIG_PATH"             => :user_config_path,
-
+      -"USER_CONFIG_PATH" => :user_config_path,
       # == Heroku settings ==
-      -"HEROKU_DYNO_INFO_PATH"        => :'heroku.dyno_info_path',
-
+      -"HEROKU_DYNO_INFO_PATH" => :'heroku.dyno_info_path',
       # == Source Location ==
       -"SOURCE_LOCATION_IGNORED_GEMS" => :source_location_ignored_gems,
-      -"SOURCE_LOCATION_CACHE_SIZE"   => :source_location_cache_size
+      -"SOURCE_LOCATION_CACHE_SIZE" => :source_location_cache_size
     }.freeze
 
     KEY_TO_NATIVE_ENV = {
       # We use different log files for native and Ruby, but the native code doesn't know this
-      native_log_file:  "LOG_FILE",
+      native_log_file: "LOG_FILE",
       native_log_level: "LOG_LEVEL"
     }.freeze
 
     SERVER_VALIDATE = %i[].freeze
 
-    DEFAULT_IGNORED_SOURCE_LOCATION_GEMS = [
-      -"skylight",
-      -"activesupport",
-      -"activerecord"
-    ].freeze
+    DEFAULT_IGNORED_SOURCE_LOCATION_GEMS = [-"skylight", -"activesupport", -"activerecord"].freeze
 
     # Default values for Skylight configuration keys
     def self.default_values
@@ -134,38 +119,32 @@ module Skylight
         begin
           ret = {
             # URLs
-            auth_url:                  -"https://auth.skylight.io/agent",
-            app_create_url:            -"https://www.skylight.io/apps",
-            merges_url:                -"https://www.skylight.io/merges",
-            validation_url:            -"https://auth.skylight.io/agent/config",
-
+            auth_url: -"https://auth.skylight.io/agent",
+            app_create_url: -"https://www.skylight.io/apps",
+            merges_url: -"https://www.skylight.io/merges",
+            validation_url: -"https://auth.skylight.io/agent/config",
             # Logging
-            log_file:                  -"-",
-            log_level:                 -"INFO",
-            alert_log_file:            -"-",
-            log_sql_parse_errors:      true,
-
+            log_file: -"-",
+            log_level: -"INFO",
+            alert_log_file: -"-",
+            log_sql_parse_errors: true,
+            native_log_level: -"warn",
             # Features
-            enable_segments:           true,
-            enable_sidekiq:            false,
-            sinatra_route_prefixes:    false,
-            enable_source_locations:   true,
-
+            enable_segments: true,
+            enable_sidekiq: false,
+            sinatra_route_prefixes: false,
+            enable_source_locations: true,
             # Deploys
-            'heroku.dyno_info_path':   -"/etc/heroku/dyno",
-            report_rails_env:          true,
-
+            'heroku.dyno_info_path': -"/etc/heroku/dyno",
+            report_rails_env: true,
             # Daemon
-            'daemon.lazy_start':       true,
-            hostname:                  Util::Hostname.default_hostname,
+            'daemon.lazy_start': true,
+            hostname: Util::Hostname.default_hostname,
             report_max_spans_exceeded: false,
-            prune_large_traces:        true
+            prune_large_traces: true
           }
 
-          unless Util::Platform::OS == -"darwin"
-            ret[:'daemon.ssl_cert_path'] = Util::SSL.ca_cert_file_or_default
-            ret[:'daemon.ssl_cert_dir'] = Util::SSL.ca_cert_dir
-          end
+          ret[:'daemon.ssl_cert_path'] = Util::SSL.ca_cert_file_or_default unless Util::Platform::OS == -"darwin"
 
           if Skylight.native?
             native_path = Skylight.libskylight_path
@@ -180,65 +159,64 @@ module Skylight
 
     REQUIRED_KEYS = {
       authentication: "authentication token",
-      hostname:       "server hostname",
-      auth_url:       "authentication url",
+      hostname: "server hostname",
+      auth_url: "authentication url",
       validation_url: "config validation url"
     }.freeze
 
     def self.native_env_keys
-      @native_env_keys ||= %i[
-        native_log_level
-        native_log_file
-        log_sql_parse_errors
-        version
-        root
-        proxy_url
-        hostname
-        session_token
-        auth_url
-        auth_http_deflate
-        auth_http_connect_timeout
-        auth_http_read_timeout
-        report_url
-        report_http_deflate
-        report_http_connect_timeout
-        report_http_read_timeout
-        report_http_disabled
-        daemon.lazy_start
-        daemon.exec_path
-        daemon.lib_path
-        daemon.pidfile_path
-        daemon.sockdir_path
-        daemon.batch_queue_depth
-        daemon.batch_sample_size
-        daemon.batch_flush_interval
-        daemon.tick_interval
-        daemon.lock_check_interval
-        daemon.inactivity_timeout
-        daemon.max_connect_tries
-        daemon.connect_try_window
-        daemon.max_prespawn_jitter
-        daemon.wait_timeout
-        daemon.client_check_interval
-        daemon.client_queue_depth
-        daemon.client_write_timeout
-        daemon.ssl_cert_path
-        daemon.ssl_cert_dir
-      ]
+      @native_env_keys ||=
+        %i[
+          native_log_level
+          native_log_file
+          log_sql_parse_errors
+          version
+          root
+          proxy_url
+          hostname
+          session_token
+          auth_url
+          auth_http_deflate
+          auth_http_connect_timeout
+          auth_http_read_timeout
+          report_url
+          report_http_deflate
+          report_http_connect_timeout
+          report_http_read_timeout
+          report_http_disabled
+          daemon.lazy_start
+          daemon.exec_path
+          daemon.lib_path
+          daemon.pidfile_path
+          daemon.sockdir_path
+          daemon.batch_queue_depth
+          daemon.batch_sample_size
+          daemon.batch_flush_interval
+          daemon.tick_interval
+          daemon.lock_check_interval
+          daemon.inactivity_timeout
+          daemon.max_connect_tries
+          daemon.connect_try_window
+          daemon.max_prespawn_jitter
+          daemon.wait_timeout
+          daemon.client_check_interval
+          daemon.client_queue_depth
+          daemon.client_write_timeout
+          daemon.ssl_cert_path
+          daemon.ssl_cert_dir
+          daemon.enable_tcp
+          daemon.tcp_port
+        ]
     end
 
     # Maps legacy config keys to new config keys
     def self.legacy_keys
-      @legacy_keys ||= {
-        'agent.sockfile_path': :'daemon.sockdir_path',
-        'agent.lockfile':      :'daemon.pidfile_path'
-      }
+      @legacy_keys ||= { 'agent.sockfile_path': :'daemon.sockdir_path', 'agent.lockfile': :'daemon.pidfile_path' }
     end
 
     def self.validators
-      @validators ||= {
-        'agent.interval': [->(v, _c) { v.is_a?(Integer) && v > 0 }, "must be an integer greater than 0"]
-      }
+      @validators ||=
+        { 'agent.interval': [->(v, _c) { v.is_a?(Integer) && v > 0 }, "must be an integer greater than 0"] }
     end
 
     # @api private
@@ -248,9 +226,7 @@ module Skylight
     def initialize(*args)
       attrs = {}
 
-      if args.last.is_a?(Hash)
-        attrs = args.pop.dup
-      end
+      attrs = args.pop.dup if args.last.is_a?(Hash)
 
       @values = {}
       @priority = {}
@@ -264,13 +240,9 @@ module Skylight
         @priority_regexp = /^#{Regexp.escape(priority_key)}\.(.+)$/
       end
 
-      attrs.each do |k, v|
-        self[k] = v
-      end
+      attrs.each { |k, v| self[k] = v }
 
-      p&.each do |k, v|
-        @priority[self.class.remap_key(k)] = v
-      end
+      p&.each { |k, v| @priority[self.class.remap_key(k)] = v }
     end
 
     def self.load(opts = {}, env = ENV)
@@ -282,10 +254,8 @@ module Skylight
       if path
         error = nil
         begin
-          attrs = YAML.safe_load(ERB.new(File.read(path)).result,
-                                 [], # permitted_classes
-                                 [], # permitted_symbols
-                                 true) # aliases enabled
+          attrs =
+            YAML.safe_load(ERB.new(File.read(path)).result, permitted_classes: [], permitted_symbols: [], aliases: true)
           error = "empty file" unless attrs
           error = "invalid format" if attrs && !attrs.is_a?(Hash)
         rescue Exception => e
@@ -298,15 +268,11 @@ module Skylight
       # The key-value pairs in this `priority` option are inserted into the
       # config's @priority hash *after* anything listed under priority_key;
       # i.e., ENV takes precendence over priority_key
-      if env
-        attrs[:priority] = remap_env(env)
-      end
+      attrs[:priority] = remap_env(env) if env
 
       config = new(priority_key, attrs)
 
-      opts.each do |k, v|
-        config[k] = v
-      end
+      opts.each { |k, v| config[k] = v }
 
       config
     end
@@ -333,12 +299,18 @@ module Skylight
 
         ret[key] =
           case val
-          when /^false$/i      then false
-          when /^true$/i       then true
-          when /^(nil|null)$/i then nil
-          when /^\d+$/         then val.to_i
-          when /^\d+\.\d+$/    then val.to_f
-          else val
+          when /^false$/i
+            false
+          when /^true$/i
+            true
+          when /^(nil|null)$/i
+            nil
+          when /^\d+$/
+            val.to_i
+          when /^\d+\.\d+$/
+            val.to_f
+          else
+            val
           end
       end
 
@@ -347,11 +319,7 @@ module Skylight
 
     # @api private
     def validate!
-      REQUIRED_KEYS.each do |k, v|
-        unless get(k)
-          raise ConfigError, "#{v} required"
-        end
-      end
+      REQUIRED_KEYS.each { |k, v| raise ConfigError, "#{v} required" unless get(k) }
 
       log_file = self[:log_file]
       alert_log_file = self[:alert_log_file]
@@ -359,6 +327,7 @@ module Skylight
 
       check_logfile_permissions(log_file, "log_file")
       check_logfile_permissions(alert_log_file, "alert_log_file")
+
       # TODO: Support rotation interpolation in this check
       check_logfile_permissions(native_log_file, "native_log_file")
 
@@ -378,15 +347,20 @@ module Skylight
       file_root = File.dirname(file)
 
       # Try to make the directory, don't blow up if we can't. Our writable? check will fail later.
-      FileUtils.mkdir_p file_root rescue nil
+      begin
+        FileUtils.mkdir_p file_root
+      rescue StandardError
+        nil
+      end
 
       if File.exist?(file) && !FileTest.writable?(file)
         raise ConfigError, "File `#{file}` is not writable. Please set #{key} in your config to a writable path"
       end
 
       unless FileTest.writable?(file_root)
-        raise ConfigError, "Directory `#{file_root}` is not writable. Please set #{key} in your config to a " \
-                           "writable path"
+        raise ConfigError,
+              "Directory `#{file_root}` is not writable. Please set #{key} in your config to a " \
+                "writable path"
       end
     end
 
@@ -406,7 +380,7 @@ module Skylight
       key = self.class.remap_key(key)
 
       return @priority[key] if @priority.key?(key)
-      return @values[key]   if @values.key?(key)
+      return @values[key] if @values.key?(key)
       return self.class.default_values[key] if self.class.default_values.key?(key)
 
       if default
@@ -421,14 +395,10 @@ module Skylight
     alias [] get
 
     def set(key, val, scope = nil)
-      if scope
-        key = [scope, key].join(".")
-      end
+      key = [scope, key].join(".") if scope
 
       if val.is_a?(Hash)
-        val.each do |k, v|
-          set(k, v, key)
-        end
+        val.each { |k, v| set(k, v, key) }
       else
         k = self.class.remap_key(key)
 
@@ -442,9 +412,7 @@ module Skylight
           end
         end
 
-        if @priority_regexp && k =~ @priority_regexp
-          @priority[$1.to_sym] = val
-        end
+        @priority[$1.to_sym] = val if @priority_regexp && k =~ @priority_regexp
 
         @values[k] = val
       end
@@ -466,7 +434,8 @@ module Skylight
           v / 1_000
         when "nanos"
           v / 1_000_000
-        else # "s", "sec", nil
+        else
+          # "s", "sec", nil
           v * 1000
         end
       else
@@ -477,13 +446,16 @@ module Skylight
     def to_native_env
       ret = []
 
-      self.class.native_env_keys.each do |key|
-        value = send_or_get(key)
-        unless value.nil?
-          env_key = KEY_TO_NATIVE_ENV[key] || ENV_TO_KEY.key(key) || key.upcase
-          ret << "SKYLIGHT_#{env_key}" << cast_for_env(value)
+      self
+        .class
+        .native_env_keys
+        .each do |key|
+          value = send_or_get(key)
+          unless value.nil?
+            env_key = KEY_TO_NATIVE_ENV[key] || ENV_TO_KEY.key(key) || key.upcase
+            ret << "SKYLIGHT_#{env_key}" << cast_for_env(value)
+          end
         end
-      end
 
       ret << "SKYLIGHT_AUTHENTICATION" << authentication_with_meta
       ret << "SKYLIGHT_VALIDATE_AUTHENTICATION" << "false"
@@ -514,9 +486,7 @@ module Skylight
 
           # If, for some odd reason you have a comma in your endpoint name, use the
           # YML config instead.
-          if ignored_endpoints.is_a?(String)
-            ignored_endpoints = ignored_endpoints.split(/\s*,\s*/)
-          end
+          ignored_endpoints = ignored_endpoints.split(/\s*,\s*/) if ignored_endpoints.is_a?(String)
 
           val = Array(get(:ignored_endpoint))
           val.concat(Array(ignored_endpoints))
@@ -529,9 +499,7 @@ module Skylight
       @source_location_ignored_gems ||=
         begin
           ignored_gems = get(:source_location_ignored_gems)
-          if ignored_gems.is_a?(String)
-            ignored_gems = ignored_gems.split(/\s*,\s*/)
-          end
+          ignored_gems = ignored_gems.split(/\s*,\s*/) if ignored_gems.is_a?(String)
 
           Array(ignored_gems) | DEFAULT_IGNORED_SOURCE_LOCATION_GEMS
         end
@@ -547,61 +515,56 @@ module Skylight
           Logger::DEBUG
         else
           case get(:log_level)
-          when /^debug$/i then Logger::DEBUG
-          when /^info$/i  then Logger::INFO
-          when /^warn$/i  then Logger::WARN
-          when /^error$/i then Logger::ERROR
-          when /^fatal$/i then Logger::FATAL
-          else Logger::ERROR # rubocop:disable Lint/DuplicateBranch
+          when /^debug$/i
+            Logger::DEBUG
+          when /^info$/i
+            Logger::INFO
+          when /^warn$/i
+            Logger::WARN
+          when /^error$/i
+            Logger::ERROR
+          when /^fatal$/i
+            Logger::FATAL
+          else
+            Logger::ERROR
           end
         end
     end
 
     def native_log_level
-      @native_log_level ||=
-        if trace?
-          "trace"
-        else
-          case log_level
-          when Logger::DEBUG then "debug"
-          when Logger::INFO  then "info"
-          when Logger::WARN  then "warn"
-          else "error"
-          end
-        end
+      get(:native_log_level).to_s.downcase
     end
 
     def logger
-      @logger ||=
-        MUTEX.synchronize do
-          load_logger
-        end
+      @logger ||= MUTEX.synchronize { load_logger }
     end
 
     def native_log_file
-      @native_log_file ||= get("native_log_file") do
-        log_file = self["log_file"]
-        return "-" if log_file == "-"
-
-        parts = log_file.to_s.split(".")
-        parts.insert(-2, "native")
-        parts.join(".")
-      end
+      @native_log_file ||=
+        get("native_log_file") do
+          log_file = self["log_file"]
+          return "-" if log_file == "-"
+
+          parts = log_file.to_s.split(".")
+          parts.insert(-2, "native")
+          parts.join(".")
+        end
     end
 
     attr_writer :logger, :alert_logger
 
     def alert_logger
-      @alert_logger ||= MUTEX.synchronize do
-        unless (l = @alert_logger)
-          out = get(:alert_log_file)
-          out = Util::AlertLogger.new(load_logger) if out == "-"
+      @alert_logger ||=
+        MUTEX.synchronize do
+          unless (l = @alert_logger)
+            out = get(:alert_log_file)
+            out = Util::AlertLogger.new(load_logger) if out == "-"
 
-          l = create_logger(out, level: Logger::DEBUG)
-        end
+            l = create_logger(out, level: Logger::DEBUG)
+          end
 
-        l
-      end
+          l
+        end
     end
 
     def enable_segments?
@@ -630,36 +593,41 @@ module Skylight
 
     private
 
-      def create_logger(out, level: :info)
-        if out.is_a?(String)
-          out = File.expand_path(out, root)
-          # May be redundant since we also do this in the permissions check
-          FileUtils.mkdir_p(File.dirname(out))
-        end
+    def create_logger(out, level: :info)
+      if out.is_a?(String)
+        out = File.expand_path(out, root)
 
-        Logger.new(out, progname: "Skylight", level: level)
-      rescue
-        Logger.new($stdout, progname: "Skylight", level: level)
+        # May be redundant since we also do this in the permissions check
+        FileUtils.mkdir_p(File.dirname(out))
       end
 
-      def load_logger
-        unless (l = @logger)
-          out = get(:log_file)
-          out = $stdout if out == "-"
-          l = create_logger(out, level: log_level)
-        end
+      Logger.new(out, progname: "Skylight", level: level)
+    rescue StandardError
+      Logger.new($stdout, progname: "Skylight", level: level)
+    end
 
-        l
+    def load_logger
+      unless (l = @logger)
+        out = get(:log_file)
+        out = $stdout if out == "-"
+        l = create_logger(out, level: log_level)
       end
 
-      def cast_for_env(val)
-        case val
-        when true  then "true"
-        when false then "false"
-        when nil   then "nil"
-        else val.to_s
-        end
+      l
+    end
+
+    def cast_for_env(val)
+      case val
+      when true
+        "true"
+      when false
+        "false"
+      when nil
+        "nil"
+      else
+        val.to_s
       end
+    end
 
     public
 
@@ -676,22 +644,18 @@ module Skylight
         return false
       end
 
-      if res.error_response?
-        warn("Unable to reach server for config validation")
-      end
+      warn("Unable to reach server for config validation") if res.error_response?
 
       unless res.config_valid?
         warn("Invalid configuration") unless res.error_response?
-        res.validation_errors.each do |k, v|
-          warn("  #{k}: #{v}")
-        end
+        res.validation_errors.each { |k, v| warn("  #{k}: #{v}") }
 
         return false if res.forbidden?
 
         corrected_config = res.corrected_config
 
         # Use defaults if no corrected config is available. This will happen if the request failed.
-        corrected_config ||= Hash[SERVER_VALIDATE.map { |k| [k, self.class.default_values.fetch(k)] }]
+        corrected_config ||= SERVER_VALIDATE.map { |k| [k, self.class.default_values.fetch(k)] }.to_h
 
         config_to_update = corrected_config.reject { |k, v| get(k) == v }
         unless config_to_update.empty?
@@ -713,29 +677,33 @@ module Skylight
 
     def check_sockdir_permissions(sockdir_path)
       # Try to make the directory, don't blow up if we can't. Our writable? check will fail later.
-      FileUtils.mkdir_p sockdir_path rescue nil
+      begin
+        FileUtils.mkdir_p sockdir_path
+      rescue StandardError
+        nil
+      end
 
       unless FileTest.writable?(sockdir_path)
-        raise ConfigError, "Directory `#{sockdir_path}` is not writable. Please set daemon.sockdir_path in " \
-                           "your config to a writable path"
+        raise ConfigError,
+              "Directory `#{sockdir_path}` is not writable. Please set daemon.sockdir_path in " \
+                "your config to a writable path"
       end
 
       if check_nfs(sockdir_path)
-        raise ConfigError, "Directory `#{sockdir_path}` is an NFS mount and will not allow sockets. Please set " \
-                           "daemon.sockdir_path in your config to a non-NFS path."
+        raise ConfigError,
+              "Directory `#{sockdir_path}` is an NFS mount and will not allow sockets. Please set " \
+                "daemon.sockdir_path in your config to a non-NFS path."
       end
     end
 
     def write(path)
       FileUtils.mkdir_p(File.dirname(path))
 
-      File.open(path, "w") do |f|
-        f.puts <<~YAML
+      File.open(path, "w") { |f| f.puts <<~YAML }
           ---
           # The authentication token for the application.
           authentication: #{self[:authentication]}
         YAML
-      end
     end
 
     #
@@ -765,17 +733,11 @@ module Skylight
     end
 
     def components
-      @components ||= {
-        web:    Util::Component.new(
-          get(:env),
-          Util::Component::DEFAULT_NAME
-        ),
-        worker: Util::Component.new(
-          get(:env),
-          get(:component) || get(:worker_component),
-          force_worker: true
-        )
-      }
+      @components ||=
+        {
+          web: Util::Component.new(get(:env), Util::Component::DEFAULT_NAME),
+          worker: Util::Component.new(get(:env), get(:component) || get(:worker_component), force_worker: true)
+        }
     rescue ArgumentError => e
       raise ConfigError, e.message
     end
@@ -789,25 +751,20 @@ module Skylight
     end
 
     def as_json(*)
-      {
-        config: {
-          priority: @priority.merge(component.as_json),
-          values:   @values
-        }
-      }
+      { config: { priority: @priority.merge(component.as_json), values: @values } }
     end
 
     private
 
-      def check_nfs(path)
-        # Should work on most *nix, though not on OS X
-        `stat -f -L -c %T #{path} 2>&1`.strip == "nfs"
-      end
+    def check_nfs(path)
+      # Should work on most *nix, though not on OS X
+      `stat -f -L -c %T #{path} 2>&1`.strip == "nfs"
+    end
 
-      def reporting_env?
-        # true if env was explicitly set,
-        # or if we are auto-detecting via the opt-in SKYLIGHT_REPORT_RAILS_ENV=true
-        !!(get(:report_rails_env) || get(:env))
-      end
+    def reporting_env?
+      # true if env was explicitly set,
+      # or if we are auto-detecting via the opt-in SKYLIGHT_REPORT_RAILS_ENV=true
+      !!(get(:report_rails_env) || get(:env))
+    end
   end
 end