tcell_agent 1.1.3 → 1.1.4

data/lib/tcell_agent/config/unknown_options.rb

@@ -3,27 +3,29 @@ require 'set'
 module TCellAgent
   module Config
     module Validate
-
       def self.get_unknown_options(config_json)
         messages = []
 
-        known_tcell_env_vars = Set.new([
-          "TCELL_AGENT_SERVER", # this is only meant for specs
-          "TCELL_AGENT_APP_ID",
-          "TCELL_AGENT_API_KEY",
-          "TCELL_HMAC_KEY",
-          "TCELL_PASSWORD_HMAC_KEY",
-          "TCELL_AGENT_HOST_IDENTIFIER",
-          "TCELL_API_URL",
-          "TCELL_INPUT_URL",
-          "TCELL_DEMOMODE",
-          "TCELL_AGENT_HOME",
-          "TCELL_AGENT_LOG_DIR",
-          "TCELL_AGENT_CONFIG",
-          "TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS",
-          "TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS",
-          "TCELL_AGENT_ALLOW_PAYLOADS",
-          "TCELL_AGENT_HOME_OWNER"])
+        known_tcell_env_vars = Set.new(
+          [
+            'TCELL_AGENT_SERVER', # this is only meant for specs
+            'TCELL_AGENT_APP_ID',
+            'TCELL_AGENT_API_KEY',
+            'TCELL_HMAC_KEY',
+            'TCELL_PASSWORD_HMAC_KEY',
+            'TCELL_AGENT_HOST_IDENTIFIER',
+            'TCELL_API_URL',
+            'TCELL_INPUT_URL',
+            'TCELL_DEMOMODE',
+            'TCELL_AGENT_HOME',
+            'TCELL_AGENT_LOG_DIR',
+            'TCELL_AGENT_CONFIG',
+            'TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS',
+            'TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS',
+            'TCELL_AGENT_ALLOW_PAYLOADS',
+            'TCELL_AGENT_HOME_OWNER'
+          ]
+        )
 
         ENV.keys.each do |environment_key|
           if environment_key =~ /^TCELL_/ && !known_tcell_env_vars.include?(environment_key)
@@ -35,70 +37,71 @@ module TCellAgent
           key_differences = []
 
           if config_json
-            first_level_keys = ["version", "applications"]
+            first_level_keys = %w[version applications]
 
             key_differences = config_json.keys - first_level_keys
 
-            applications = config_json.fetch("applications", nil)
+            applications = config_json.fetch('applications', nil)
             if applications
 
               if applications.size > 1
-                messages << "Multiple applications detected in config file"
+                messages << 'Multiple applications detected in config file'
 
               elsif applications.size == 1
                 application = applications[0]
 
-                second_level_keys = [
-                  "name",
-                  "app_id",
-                  "api_key",
-                  "fetch_policies_from_tcell",
-                  "preload_policy_filename",
-                  "log_dir",
-                  "tcell_api_url",
-                  "tcell_input_url",
-                  "host_identifier",
-                  "hipaaSafeMode",
-                  "hmac_key",
-                  "password_hmac_key",
-                  "js_agent_api_base_url",
-                  "js_agent_url",
-                  "max_csp_header_bytes",
-                  "event_batch_size_limit",
-                  "allow_unencrypted_appsensor_payloads",
-                  "allow_unencrypted_appfirewall_payloads",
-                  "allow_payloads",
-                  "reverse_proxy",
-                  "reverse_proxy_ip_address_header",
-                  "demomode",
-                  "logging_options",
-                  "data_exposure",
-                  "disable_all",
-                  "enabled",
-                  "enable_event_manager",
-                  "enable_event_consumer",
-                  "enable_policy_polling",
-                  "enable_instrumentation",
-                  "enable_intercept_requests",
-                  "instrument_for_events",
-                  "agent_home_owner",
-                  "enabled_instrumentations"]
-
-                key_differences = key_differences + (application.keys - second_level_keys)
-
-                if application.fetch("logging_options", nil)
-                  logging_options = application["logging_options"]
-                  key_differences = key_differences + (logging_options.keys - ["enabled", "level", "filename"])
+                second_level_keys = %w[
+                  name
+                  app_id
+                  api_key
+                  fetch_policies_from_tcell
+                  preload_policy_filename
+                  log_dir
+                  tcell_api_url
+                  tcell_input_url
+                  host_identifier
+                  hipaaSafeMode
+                  hmac_key
+                  password_hmac_key
+                  js_agent_api_base_url
+                  js_agent_url
+                  max_csp_header_bytes
+                  event_batch_size_limit
+                  allow_unencrypted_appsensor_payloads
+                  allow_unencrypted_appfirewall_payloads
+                  allow_payloads
+                  reverse_proxy
+                  reverse_proxy_ip_address_header
+                  demomode
+                  logging_options
+                  data_exposure
+                  disable_all
+                  enabled
+                  enable_event_manager
+                  enable_event_consumer
+                  enable_policy_polling
+                  enable_instrumentation
+                  enable_intercept_requests
+                  instrument_for_events
+                  agent_home_owner
+                  enabled_instrumentations
+                ]
+
+                key_differences += (application.keys - second_level_keys)
+
+                if application.fetch('logging_options', nil)
+                  logging_options = application['logging_options']
+                  key_differences += (logging_options.keys - %w[enabled level filename])
                 end
 
-                if application.fetch("data_exposure", nil)
-                  data_exposure = application["data_exposure"]
-                  key_differences = key_differences + (data_exposure.keys - ["max_data_ex_db_records_per_request"])
+                if application.fetch('data_exposure', nil)
+                  data_exposure = application['data_exposure']
+                  key_differences += (data_exposure.keys - ['max_data_ex_db_records_per_request'])
                 end
 
-                if application.fetch("enabled_instrumentations", nil)
-                  enabled_instrumentations = application["enabled_instrumentations"]
-                  key_differences = key_differences + (enabled_instrumentations.keys - ["doorkeeper", "devise", "authlogic"])
+                if application.fetch('enabled_instrumentations', nil)
+                  enabled_instrumentations = application['enabled_instrumentations']
+                  key_differences += (enabled_instrumentations.keys - %w[doorkeeper devise authlogic])
                 end
               end
             end
@@ -114,7 +117,6 @@ module TCellAgent
 
         messages
       end
-
     end
   end
 end

data/lib/tcell_agent/configuration.rb

@@ -21,46 +21,46 @@ module TCellAgent
     yield(configuration)
   end
 
-  class Configuration
+  class Configuration # rubocop:disable Metrics/ClassLength
     attr_accessor :version,
-      :app_id,
-      :api_key,
-      :hmac_key,
-      :tcell_api_url,
-      :tcell_input_url,
-      :logging_options,
-      :logger,
-      :appfirewall_payloads_logger, # appfirewall_payloads_logger can be specified from initializers
-      :fetch_policies_from_tcell, :instrument_for_events,
-      :preload_policy_filename,
-      :host_identifier,
-      :uuid,
-      :event_batch_size_limit, :event_time_limit_seconds,
-      :base_dir,
-      :cache_filename,
-      :js_agent_api_base_url,
-      :js_agent_url,
-      :config_filename,
-      :agent_log_dir,
-      :max_data_ex_db_records_per_request,
-      :agent_home_dir,
-      :agent_home_owner,
-      :reverse_proxy,
-      :reverse_proxy_ip_address_header,
-      :log_file_name,
-      :log_tag,
-      :max_csp_header_bytes,
-      :demomode,
-      :allow_payloads,
-      :password_hmac_key
+                  :app_id,
+                  :api_key,
+                  :hmac_key,
+                  :tcell_api_url,
+                  :tcell_input_url,
+                  :logging_options,
+                  :logger,
+                  :appfirewall_payloads_logger, # appfirewall_payloads_logger can be specified from initializers
+                  :fetch_policies_from_tcell, :instrument_for_events,
+                  :preload_policy_filename,
+                  :host_identifier,
+                  :uuid,
+                  :event_batch_size_limit, :event_time_limit_seconds,
+                  :base_dir,
+                  :cache_filename,
+                  :js_agent_api_base_url,
+                  :js_agent_url,
+                  :config_filename,
+                  :agent_log_dir,
+                  :max_data_ex_db_records_per_request,
+                  :agent_home_dir,
+                  :agent_home_owner,
+                  :reverse_proxy,
+                  :reverse_proxy_ip_address_header,
+                  :log_file_name,
+                  :log_tag,
+                  :max_csp_header_bytes,
+                  :demomode,
+                  :allow_payloads,
+                  :password_hmac_key
 
     attr_accessor :disable_all,
-      :enabled,
-      :enable_event_manager,       # false = Do not start the even manager
-      :enable_event_consumer,      # false = Do not consume events, drop them
-      :enable_policy_polling,      # false = Do not poll for policies
-      :enable_instrumentation,     # false = Do not add instrumentation
-      :enable_intercept_requests   # false = Do not insert middleware
+                  :enabled,
+                  :enable_event_manager,       # false = Do not start the even manager
+                  :enable_event_consumer,      # false = Do not consume events, drop them
+                  :enable_policy_polling,      # false = Do not poll for policies
+                  :enable_instrumentation,     # false = Do not add instrumentation
+                  :enable_intercept_requests   # false = Do not insert middleware
 
     attr_accessor :enabled_instrumentations
 
@@ -87,30 +87,30 @@ module TCellAgent
     end
 
     def should_instrument_doorkeeper?
-      if @enabled_instrumentations.has_key?('doorkeeper') || @enabled_instrumentations.has_key?(:doorkeeper)
-        !!(@enabled_instrumentations['doorkeeper'] || @enabled_instrumentations[:doorkeeper])
+      if @enabled_instrumentations.key?('doorkeeper') || @enabled_instrumentations.key?(:doorkeeper)
+        !!(@enabled_instrumentations['doorkeeper'] || @enabled_instrumentations[:doorkeeper]) # rubocop:disable Style/DoubleNegation
       else
         true
       end
     end
 
     def should_instrument_devise?
-      if @enabled_instrumentations.has_key?('devise') || @enabled_instrumentations.has_key?(:devise)
-        !!(@enabled_instrumentations['devise'] || @enabled_instrumentations[:devise])
+      if @enabled_instrumentations.key?('devise') || @enabled_instrumentations.key?(:devise)
+        !!(@enabled_instrumentations['devise'] || @enabled_instrumentations[:devise]) # rubocop:disable Style/DoubleNegation
       else
         true
       end
     end
 
     def should_instrument_authlogic?
-      if @enabled_instrumentations.has_key?('authlogic') || @enabled_instrumentations.has_key?(:authlogic)
-        !!(@enabled_instrumentations['authlogic'] || @enabled_instrumentations[:authlogic])
+      if @enabled_instrumentations.key?('authlogic') || @enabled_instrumentations.key?(:authlogic)
+        !!(@enabled_instrumentations['authlogic'] || @enabled_instrumentations[:authlogic]) # rubocop:disable Style/DoubleNegation
       else
         true
       end
     end
 
-    def initialize(filename="config/tcell_agent.config", useapp=nil)
+    def initialize(filename = 'config/tcell_agent.config', _useapp = nil)
       # These will be set when the agent starts up, to give rails initializers
       # a chance to run
       @cache_filename = nil
@@ -141,7 +141,7 @@ module TCellAgent
         :authlogic => true
       }
 
-      @log_file_name = "tcell_agent.log"
+      @log_file_name = 'tcell_agent.log'
 
       @event_batch_size_limit = 50
       @event_time_limit_seconds = 15
@@ -154,8 +154,8 @@ module TCellAgent
       @max_csp_header_bytes = nil
       @password_hmac_key = nil
 
-      @agent_home_dir = ENV["TCELL_AGENT_HOME"] || File.join(Dir.getwd, "tcell")
-      @config_filename = ENV["TCELL_AGENT_CONFIG"] || File.join(Dir.getwd, filename)
+      @agent_home_dir = ENV['TCELL_AGENT_HOME'] || File.join(Dir.getwd, 'tcell')
+      @config_filename = ENV['TCELL_AGENT_CONFIG'] || File.join(Dir.getwd, filename)
 
       read_config_from_file(@config_filename)
       read_config_using_env
@@ -165,33 +165,33 @@ module TCellAgent
         @event_time_limit_seconds = 2
       end
 
-      if ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS"]
-        puts "tCell.io Agent: [DEPRECATED] TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS is deprecated and will be removed in a future release. Please switch to TCELL_AGENT_ALLOW_PAYLOADS."
+      if ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS']
+        puts 'tCell.io Agent: [DEPRECATED] TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS is deprecated and will be removed in a future release. Please switch to TCELL_AGENT_ALLOW_PAYLOADS.'
       end
 
-      if (ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS"])
-        puts "tCell.io Agent: [DEPRECATED] TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS is deprecated and will be removed in a future release. Please switch to TCELL_AGENT_ALLOW_PAYLOADS."
+      if ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS']
+        puts 'tCell.io Agent: [DEPRECATED] TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS is deprecated and will be removed in a future release. Please switch to TCELL_AGENT_ALLOW_PAYLOADS.'
       end
 
-      if (ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS"] != nil)
-        @allow_payloads = [true, "true", "yes", "1"].include?(ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS"])
+      unless ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS'].nil?
+        @allow_payloads = [true, 'true', 'yes', '1'].include?(ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPSENSOR_PAYLOADS'])
       end
-      if (ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS"] != nil)
-        @allow_payloads = [true, "true", "yes", "1"].include?(ENV["TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS"])
+      unless ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS'].nil?
+        @allow_payloads = [true, 'true', 'yes', '1'].include?(ENV['TCELL_AGENT_ALLOW_UNENCRYPTED_APPFIREWALL_PAYLOADS'])
       end
-      if (ENV["TCELL_AGENT_ALLOW_PAYLOADS"] != nil)
-        @allow_payloads = [true, "true", "yes", "1"].include?(ENV["TCELL_AGENT_ALLOW_PAYLOADS"])
+      unless ENV['TCELL_AGENT_ALLOW_PAYLOADS'].nil?
+        @allow_payloads = [true, 'true', 'yes', '1'].include?(ENV['TCELL_AGENT_ALLOW_PAYLOADS'])
       end
 
       @tcell_api_url = compose_api_url!
-      @tcell_input_url ||= "https://input.tcell.io/api/v1"
-      @js_agent_url ||= "https://jsagent.tcell.io/tcellagent.min.js"
+      @tcell_input_url ||= 'https://input.tcell.io/api/v1'
+      @js_agent_url ||= 'https://jsagent.tcell.io/tcellagent.min.js'
 
-      if (@host_identifier == nil)
+      if @host_identifier.nil?
         begin
-          @host_identifier = (Socket.gethostname() || "localhost")
+          @host_identifier = (Socket.gethostname || 'localhost')
         rescue StandardError
-          @host_identifier = "host_identifier_not_found"
+          @host_identifier = 'host_identifier_not_found'
         end
       end
 
@@ -199,12 +199,12 @@ module TCellAgent
     end
 
     def compose_api_url!
-      @tcell_api_url ||= "https://api.tcell.io"
+      @tcell_api_url ||= 'https://api.tcell.io'
       parsed_uri = URI.parse(@tcell_api_url)
 
       api_url = [
         parsed_uri.scheme,
-        "://",
+        '://',
         parsed_uri.host
       ]
 
@@ -214,26 +214,26 @@ module TCellAgent
 
       [
         api_url.join(''),
-        "/agents/api/v1/apps/",
-        "{app_id}",
-        "/policies/latest",
-        "?",
-        "type=jsagentinjection:v1",
-        "&type=http-redirect:v1",
-        "&type=clickjacking:v1",
-        "&type=secure-headers:v1",
-        "&type=cmdi:v1",
-        "&type=csp-headers:v1",
-        "&type=dlp:v1",
-        "&type=login:v1",
-        "&type=regex:v1",
-        "&type=appsensor:v2",
-        "&type=patches:v1"
+        '/agents/api/v1/apps/',
+        '{app_id}',
+        '/policies/latest',
+        '?',
+        'type=jsagentinjection:v1',
+        '&type=http-redirect:v1',
+        '&type=clickjacking:v1',
+        '&type=secure-headers:v1',
+        '&type=cmdi:v1',
+        '&type=csp-headers:v1',
+        '&type=dlp:v1',
+        '&type=login:v1',
+        '&type=regex:v1',
+        '&type=appsensor:v2',
+        '&type=patches:v1'
       ].join('')
     end
 
     def cache_filename_with_app_id
-      @cache_filename ||= File.join(@agent_home_dir, "cache", "tcell_agent.cache")
+      @cache_filename ||= File.join(@agent_home_dir, 'cache', 'tcell_agent.cache')
 
       if @app_id
         "#{@cache_filename}.#{@app_id}"
@@ -243,140 +243,136 @@ module TCellAgent
     end
 
     def read_config_using_env
-      @app_id = ENV["TCELL_AGENT_APP_ID"] || @app_id
-      @api_key = ENV["TCELL_AGENT_API_KEY"] || @api_key
-      @hmac_key = ENV["TCELL_HMAC_KEY"] || @hmac_key
-      @password_hmac_key = ENV["TCELL_PASSWORD_HMAC_KEY"] || @password_hmac_key
-      @host_identifier = ENV["TCELL_AGENT_HOST_IDENTIFIER"] || @host_identifier
-      @tcell_api_url = ENV["TCELL_API_URL"] || @tcell_api_url
-      @tcell_input_url = ENV["TCELL_INPUT_URL"] || @tcell_input_url
-      @demomode = ENV["TCELL_DEMOMODE"] || @demomode
-
-      @agent_home_owner = ENV["TCELL_AGENT_HOME_OWNER"] || @agent_home_owner
-      @agent_log_dir = ENV["TCELL_AGENT_LOG_DIR"] || @agent_log_dir
+      @app_id = ENV['TCELL_AGENT_APP_ID'] || @app_id
+      @api_key = ENV['TCELL_AGENT_API_KEY'] || @api_key
+      @hmac_key = ENV['TCELL_HMAC_KEY'] || @hmac_key
+      @password_hmac_key = ENV['TCELL_PASSWORD_HMAC_KEY'] || @password_hmac_key
+      @host_identifier = ENV['TCELL_AGENT_HOST_IDENTIFIER'] || @host_identifier
+      @tcell_api_url = ENV['TCELL_API_URL'] || @tcell_api_url
+      @tcell_input_url = ENV['TCELL_INPUT_URL'] || @tcell_input_url
+      @demomode = ENV['TCELL_DEMOMODE'] || @demomode
+
+      @agent_home_owner = ENV['TCELL_AGENT_HOME_OWNER'] || @agent_home_owner
+      @agent_log_dir = ENV['TCELL_AGENT_LOG_DIR'] || @agent_log_dir
     end
 
     def read_config_from_file(filename)
-      if File.file?(filename)
-        #puts "tCell.io: Loading from file"
-        begin
-          config_text = File.open(filename).read
-          config = JSON.parse(config_text)
+      return unless File.file?(filename)
 
-          messages = TCellAgent::Config::Validate.get_unknown_options(config)
-          messages.each do |message|
-            puts message
-          end
+      begin
+        config_text = File.open(filename).read
+        config = JSON.parse(config_text)
+
+        messages = TCellAgent::Config::Validate.get_unknown_options(config)
+        messages.each do |message|
+          puts message
+        end
+
+        if config['version'] == 1
+          # Required
+          app_data = config['applications'][0] # Default
+          @version = 1
+          @app_id = app_data['app_id']
+          @api_key = app_data['api_key']
+
+          # Optional
+          @preload_policy_filename = app_data.fetch('preload_policy_filename', nil)
+
+          @disable_all = app_data.fetch('disable_all', @disable_all)
+          @enabled = app_data.fetch('enabled', @enabled)
+
+          @enable_event_manager = app_data.fetch('enable_event_manager', @enable_event_manager)
+          @enable_event_consumer = app_data.fetch('enable_event_consumer', @enable_event_consumer)
+          @enable_policy_polling = app_data.fetch('enable_policy_polling', @enable_policy_polling)
+          @enable_instrumentation = app_data.fetch('enable_instrumentation', @enable_instrumentation)
+          @enable_intercept_requests = app_data.fetch('enable_intercept_requests', @enable_intercept_requests)
+          @fetch_policies_from_tcell = app_data.fetch('fetch_policies_from_tcell', @fetch_policies_from_tcell)
+          @instrument_for_events = app_data.fetch('instrument_for_events', @instrument_for_events)
+
+          @agent_home_owner = app_data.fetch('agent_home_owner', @agent_home_owner)
+
+          @logging_options = app_data.fetch('logging_options', {})
+          @agent_log_dir = app_data.fetch('log_dir', @agent_log_dir)
+          @log_file_name = @logging_options.fetch('filename', @log_file_name)
+
+          @tcell_api_url = app_data.fetch('tcell_api_url', @tcell_api_url)
+          @tcell_input_url = app_data.fetch('tcell_input_url', @tcell_input_url)
+
+          @max_csp_header_bytes = app_data.fetch('max_csp_header_bytes', @max_csp_header_bytes)
+
+          @allow_payloads = app_data.fetch(
+            'allow_unencrypted_appsensor_payloads',
+            @allow_payloads
+          )
+          @allow_payloads = app_data.fetch(
+            'allow_unencrypted_appfirewall_payloads',
+            @allow_payloads
+          )
+          @allow_payloads = app_data.fetch(
+            'allow_payloads',
+            @allow_payloads
+          )
+
+          data_exposure = app_data.fetch('data_exposure', {})
+          @max_data_ex_db_records_per_request = data_exposure.fetch('max_data_ex_db_records_per_request', @max_data_ex_db_records_per_request)
 
-          if (config["version"] == 1)
-            # Required
-            app_data = config["applications"][0] #Default
-            @version = 1
-            @app_id = app_data["app_id"]
-            @api_key = app_data["api_key"]
-
-            # Optional
-            @preload_policy_filename = app_data.fetch("preload_policy_filename", nil)
-
-            @disable_all = app_data.fetch("disable_all", @disable_all)
-            @enabled = app_data.fetch("enabled", @enabled)
-
-            @enable_event_manager = app_data.fetch("enable_event_manager", @enable_event_manager)
-            @enable_event_consumer = app_data.fetch("enable_event_consumer", @enable_event_consumer)
-            @enable_policy_polling = app_data.fetch("enable_policy_polling", @enable_policy_polling)
-            @enable_instrumentation = app_data.fetch("enable_instrumentation", @enable_instrumentation)
-            @enable_intercept_requests = app_data.fetch("enable_intercept_requests", @enable_intercept_requests)
-            @fetch_policies_from_tcell = app_data.fetch("fetch_policies_from_tcell", @fetch_policies_from_tcell)
-            @instrument_for_events = app_data.fetch("instrument_for_events", @instrument_for_events)
-
-            @agent_home_owner = app_data.fetch("agent_home_owner", @agent_home_owner)
-
-            @logging_options = app_data.fetch("logging_options", {})
-            @agent_log_dir = app_data.fetch("log_dir", @agent_log_dir)
-            @log_file_name = @logging_options.fetch("filename", @log_file_name)
-
-            @tcell_api_url = app_data.fetch("tcell_api_url", @tcell_api_url)
-            @tcell_input_url = app_data.fetch("tcell_input_url", @tcell_input_url)
-
-            @max_csp_header_bytes = app_data.fetch("max_csp_header_bytes", @max_csp_header_bytes)
-
-            @allow_payloads = app_data.fetch(
-              'allow_unencrypted_appsensor_payloads',
-              @allow_payloads
-            )
-            @allow_payloads = app_data.fetch(
-              'allow_unencrypted_appfirewall_payloads',
-              @allow_payloads
-            )
-            @allow_payloads = app_data.fetch(
-              'allow_payloads',
-              @allow_payloads
-            )
-
-            data_exposure = app_data.fetch('data_exposure', {})
-            @max_data_ex_db_records_per_request = data_exposure.fetch('max_data_ex_db_records_per_request', @max_data_ex_db_records_per_request)
-
-            @enabled_instrumentations = app_data.fetch('enabled_instrumentations', @enabled_instrumentations)
-
-            @reverse_proxy = app_data.fetch('reverse_proxy', @reverse_proxy)
-            @reverse_proxy_ip_address_header = app_data.fetch('reverse_proxy_ip_address_header', @reverse_proxy_ip_address_header)
-
-            @host_identifier = app_data.fetch("host_identifier", @host_identifier)
-            @hmac_key = app_data.fetch("hmac_key", @hmac_key)
-
-            @password_hmac_key = app_data.fetch("password_hmac_key", @password_hmac_key)
-
-            @uuid = SecureRandom.uuid
-            if (@uuid == nil)
-              @uuid = "secure-random-failed"
-            end
-
-            if app_data.key?("js_agent_api_base_url")
-              @js_agent_api_base_url = app_data["js_agent_api_base_url"]
-            end
-            if app_data.key?("js_agent_url")
-              @js_agent_url = app_data["js_agent_url"]
-            end
-
-            @demomode = app_data.fetch('demomode', @demomode)
-          else
-            puts " ********* ********* ********* *********"
-            puts "* tCell.io                               *"
-            puts "* Unsupported config file version        *"
-            puts " ********* ********* ********* *********"
+          @enabled_instrumentations = app_data.fetch('enabled_instrumentations', @enabled_instrumentations)
+
+          @reverse_proxy = app_data.fetch('reverse_proxy', @reverse_proxy)
+          @reverse_proxy_ip_address_header = app_data.fetch('reverse_proxy_ip_address_header', @reverse_proxy_ip_address_header)
+
+          @host_identifier = app_data.fetch('host_identifier', @host_identifier)
+          @hmac_key = app_data.fetch('hmac_key', @hmac_key)
+
+          @password_hmac_key = app_data.fetch('password_hmac_key', @password_hmac_key)
+
+          @uuid = SecureRandom.uuid
+          @uuid = 'secure-random-failed' if @uuid.nil?
+
+          if app_data.key?('js_agent_api_base_url')
+            @js_agent_api_base_url = app_data['js_agent_api_base_url']
+          end
+          if app_data.key?('js_agent_url')
+            @js_agent_url = app_data['js_agent_url']
           end
-        rescue StandardError => e
-          puts " ********* ********* ********* *********"
-          puts "* tCell.io                               *"
-          puts "* Could not load config file             *"
-          puts " ********* ********* ********* *********"
-          puts e
-        end #begin
-      end # filename exist
-    end #def read
+
+          @demomode = app_data.fetch('demomode', @demomode)
+        else
+          puts ' ********* ********* ********* *********'
+          puts '* tCell.io                               *'
+          puts '* Unsupported config file version        *'
+          puts ' ********* ********* ********* *********'
+        end
+      rescue StandardError => e
+        puts ' ********* ********* ********* *********'
+        puts '* tCell.io                               *'
+        puts '* Could not load config file             *'
+        puts ' ********* ********* ********* *********'
+        puts e
+      end
+    end
 
     # old value could be set via initializers, this makes sure those initializers still work
     # properly
     def allow_unencrypted_appfirewall_payloads=(val)
       @allow_payloads = val
     end
+
     # keep this around in case the value was read as well
     def allow_unencrypted_appfirewall_payloads
       @allow_payloads
     end
 
     def log_filename
-      @agent_log_dir ||= File.join(@agent_home_dir, "logs")
+      @agent_log_dir ||= File.join(@agent_home_dir, 'logs')
       File.join(@agent_log_dir, @log_file_name)
     end
 
     def appfirewall_payloads_log_filename
-      @agent_log_dir ||= File.join(@agent_home_dir, "logs")
-      File.join(@agent_log_dir, "tcell_agent_payloads.log")
+      @agent_log_dir ||= File.join(@agent_home_dir, 'logs')
+      File.join(@agent_log_dir, 'tcell_agent_payloads.log')
     end
-
-  end # class
+  end
 
   TCellAgent.configuration ||= TCellAgent::Configuration.new
-
 end