inspec-core 4.3.2 → 4.6.3

data/lib/fetchers/local.rb

@@ -1,12 +1,8 @@
-# encoding: utf-8
-# author: Dominik Richter
-# author: Christoph Hartmann
-
-require 'openssl'
+require "openssl"
 
 module Fetchers
   class Local < Inspec.fetcher(1)
-    name 'local'
+    name "local"
     priority 0
 
     def self.resolve(target)
@@ -31,11 +27,11 @@ module Fetchers
 
     def self.resolve_from_string(target)
       # Support "urls" in the form of file://
-      if target.start_with?('file://')
-        target = target.gsub(%r{^file://}, '')
+      if target.start_with?("file://")
+        target = target.gsub(%r{^file://}, "")
       else
         # support for windows paths
-        target = target.tr('\\', '/')
+        target = target.tr('\\', "/")
       end
 
       target if File.exist?(File.expand_path(target))
@@ -105,7 +101,7 @@ module Fetchers
     end
 
     def perform_shasum(target)
-      @archive_shasum ||= OpenSSL::Digest::SHA256.digest(File.read(target)).unpack('H*')[0]
+      @archive_shasum ||= OpenSSL::Digest::SHA256.digest(File.read(target)).unpack("H*")[0]
     end
 
     def resolved_source

data/lib/fetchers/mock.rb

@@ -1,10 +1,8 @@
-# encoding: utf-8
-# author: Dominik Richter
-# author: Christoph Hartmann
+require "inspec/fetcher"
 
 module Fetchers
   class Mock < Inspec.fetcher(1)
-    name 'mock'
+    name "mock"
     priority 0
 
     def self.resolve(target)
@@ -29,7 +27,7 @@ module Fetchers
     end
 
     def cache_key
-      ''
+      ""
     end
   end
 end

data/lib/fetchers/url.rb

@@ -1,22 +1,18 @@
-# encoding: utf-8
-# author: Dominik Richter
-# author: Christoph Hartmann
-
-require 'uri'
-require 'openssl'
-require 'tempfile'
-require 'open-uri'
+require "uri"
+require "openssl"
+require "tempfile"
+require "open-uri"
 
 module Fetchers
   class Url < Inspec.fetcher(1)
     MIME_TYPES = {
-      'application/x-zip-compressed' => '.zip',
-      'application/zip' => '.zip',
-      'application/x-gzip' => '.tar.gz',
-      'application/gzip' => '.tar.gz',
+      "application/x-zip-compressed" => ".zip",
+      "application/zip" => ".zip",
+      "application/x-gzip" => ".tar.gz",
+      "application/gzip" => ".tar.gz",
     }.freeze
 
-    name 'url'
+    name "url"
     priority 200
 
     def self.resolve(target, opts = {})
@@ -31,7 +27,7 @@ module Fetchers
 
     def self.resolve_from_string(target, opts, username = nil, password = nil)
       uri = URI.parse(target)
-      return nil if uri.nil? or uri.scheme.nil?
+      return nil if uri.nil? || uri.scheme.nil?
       return nil unless %{ http https }.include? uri.scheme
       target = transform(target)
       opts[:username] = username if username
@@ -66,11 +62,11 @@ module Fetchers
     # https://bitbucket.org/username/repo/commits/95ce1f83d5bbe9eec34c5973f6894617e8d6d8cc is transformed to
     # https://bitbucket.org/username/repo/get/95ce1f83d5bbe9eec34c5973f6894617e8d6d8cc.tar.gz
 
-    GITHUB_URL_REGEX = %r{^https?://(www\.)?github\.com/(?<user>[\w-]+)/(?<repo>[\w-]+)(\.git)?(/)?$}
-    GITHUB_URL_WITH_TREE_REGEX = %r{^https?://(www\.)?github\.com/(?<user>[\w-]+)/(?<repo>[\w-]+)/tree/(?<commit>[\w\.]+)(/)?$}
-    BITBUCKET_URL_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)(\.git)?(/)?$}
-    BITBUCKET_URL_BRANCH_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)/branch/(?<branch>[\w\.]+)(/)?$}
-    BITBUCKET_URL_COMMIT_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)/commits/(?<commit>[\w\.]+)(/)?$}
+    GITHUB_URL_REGEX = %r{^https?://(www\.)?github\.com/(?<user>[\w-]+)/(?<repo>[\w.-]+?)(\.git)?(/)?$}.freeze
+    GITHUB_URL_WITH_TREE_REGEX = %r{^https?://(www\.)?github\.com/(?<user>[\w-]+)/(?<repo>[\w.-]+)/tree/(?<commit>[\w\.]+)(/)?$}.freeze
+    BITBUCKET_URL_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)(\.git)?(/)?$}.freeze
+    BITBUCKET_URL_BRANCH_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)/branch/(?<branch>[\w\.]+)(/)?$}.freeze
+    BITBUCKET_URL_COMMIT_REGEX = %r{^https?://(www\.)?bitbucket\.org/(?<user>[\w-]+)/(?<repo>[\w-]+)/commits/(?<commit>[\w\.]+)(/)?$}.freeze
 
     def self.transform(target)
       transformed_target = if m = GITHUB_URL_REGEX.match(target) # rubocop:disable Lint/AssignmentInCondition
@@ -98,8 +94,8 @@ module Fetchers
     def initialize(url, opts)
       @target = url.to_s
       @target_uri = url.is_a?(URI) ? url : parse_uri(url)
-      @insecure = opts['insecure']
-      @token = opts['token']
+      @insecure = opts["insecure"]
+      @token = opts["token"]
       @config = opts
       @archive_path = nil
       @temp_archive_path = nil
@@ -130,23 +126,23 @@ module Fetchers
 
     def sha256
       file = @archive_path || temp_archive_path
-      OpenSSL::Digest::SHA256.digest(File.read(file)).unpack('H*')[0]
+      OpenSSL::Digest::SHA256.digest(File.read(file)).unpack("H*")[0]
     end
 
     def file_type_from_remote(remote)
-      content_type = remote.meta['content-type']
+      content_type = remote.meta["content-type"]
       file_type = MIME_TYPES[content_type]
 
       if file_type.nil?
         Inspec::Log.warn("Unrecognized content type: #{content_type}. Assuming tar.gz")
-        file_type = '.tar.gz'
+        file_type = ".tar.gz"
       end
 
       file_type
     end
 
     def temp_archive_path
-      @temp_archive_path ||= if @config['server_type'] == 'automate2'
+      @temp_archive_path ||= if @config["server_type"] == "automate2"
                                download_automate2_archive_to_temp
                              else
                                download_archive_to_temp
@@ -158,13 +154,13 @@ module Fetchers
 
       Inspec::Log.debug("Fetching URL: #{@target}")
       json = {
-        owner: @config['profile'][0],
-        name: @config['profile'][1],
-        version: @config['profile'][2],
+        owner: @config["profile"][0],
+        name: @config["profile"][1],
+        version: @config["profile"][2],
       }.to_json
 
       opts = http_opts
-      opts[:use_ssl] = @target_uri.scheme == 'https'
+      opts[:use_ssl] = @target_uri.scheme == "https"
 
       if @insecure
         opts[:verify_mode] = OpenSSL::SSL::VERIFY_NONE
@@ -177,12 +173,12 @@ module Fetchers
         req.add_field(key, value)
       end
       req.body = json
-      res = Net::HTTP.start(@target_uri.host, @target_uri.port, opts) { |http|
+      res = Net::HTTP.start(@target_uri.host, @target_uri.port, opts) do |http|
         http.request(req)
-      }
+      end
 
-      @archive_type = '.tar.gz'
-      archive = Tempfile.new(['inspec-dl-', @archive_type])
+      @archive_type = ".tar.gz"
+      archive = Tempfile.new(["inspec-dl-", @archive_type])
       archive.binmode
       archive.write(res.body)
       archive.rewind
@@ -197,7 +193,7 @@ module Fetchers
       Inspec::Log.debug("Fetching URL: #{@target}")
       remote = open_via_uri(@target)
       @archive_type = file_type_from_remote(remote) # side effect :(
-      archive = Tempfile.new(['inspec-dl-', @archive_type])
+      archive = Tempfile.new(["inspec-dl-", @archive_type])
       archive.binmode
       archive.write(remote.read)
       archive.rewind
@@ -211,7 +207,7 @@ module Fetchers
 
       if opts[:http_basic_authentication]
         # OpenURI does not support userinfo so we need to remove it
-        open(target.sub("#{@target_uri.userinfo}@", ''), opts)
+        open(target.sub("#{@target_uri.userinfo}@", ""), opts)
       else
         open(target, opts)
       end
@@ -231,16 +227,16 @@ module Fetchers
       opts = {}
       opts[:ssl_verify_mode] = OpenSSL::SSL::VERIFY_NONE if @insecure
 
-      if @config['server_type'] =~ /automate/
-        opts['chef-delivery-enterprise'] = @config['automate']['ent']
-        if @config['automate']['token_type'] == 'dctoken'
-          opts['x-data-collector-token'] = @config['token']
+      if @config["server_type"] =~ /automate/
+        opts["chef-delivery-enterprise"] = @config["automate"]["ent"]
+        if @config["automate"]["token_type"] == "dctoken"
+          opts["x-data-collector-token"] = @config["token"]
         else
-          opts['chef-delivery-user'] = @config['user']
-          opts['chef-delivery-token'] = @config['token']
+          opts["chef-delivery-user"] = @config["user"]
+          opts["chef-delivery-token"] = @config["token"]
         end
       elsif @token
-        opts['Authorization'] = "Bearer #{@token}"
+        opts["Authorization"] = "Bearer #{@token}"
       end
 
       username = @config[:username] || @target_uri.user
@@ -264,8 +260,10 @@ module Fetchers
           true
         end
       end
-      raise 'Unable to fetch profile - the following HTTP headers have no value: ' \
-        "#{keys_missing_values.join(', ')}" unless keys_missing_values.empty?
+      unless keys_missing_values.empty?
+        raise "Unable to fetch profile - the following HTTP headers have no value: " \
+          "#{keys_missing_values.join(', ')}"
+      end
     end
   end
 end

data/lib/inspec.rb

@@ -1,32 +1,31 @@
-# encoding: utf-8
 # copyright: 2015, Dominik Richter
-# author: Dominik Richter
-# author: Christoph Hartmann
 
 libdir = File.dirname(__FILE__)
 $LOAD_PATH.unshift(libdir) unless $LOAD_PATH.include?(libdir)
 
-require 'inspec/version'
-require 'inspec/exceptions'
-require 'utils/deprecation'
-require 'inspec/profile'
-require 'inspec/rule'
-require 'matchers/matchers'
-require 'inspec/runner'
-require 'inspec/shell'
-require 'inspec/formatters'
-require 'inspec/reporters'
-require 'inspec/input_registry'
-require 'inspec/rspec_extensions'
-require 'inspec/globals'
-require 'inspec/impact'
-require 'inspec/utils/telemetry'
+require "inspec/version"
+require "inspec/exceptions"
+require "inspec/utils/deprecation"
+require "inspec/profile"
+require "inspec/rule"
+require "matchers/matchers"
+require "inspec/runner"
+require "inspec/shell"
+require "inspec/formatters"
+require "inspec/reporters"
+require "inspec/input_registry"
+require "inspec/rspec_extensions"
+require "inspec/globals"
+require "inspec/impact"
+require "inspec/utils/telemetry"
+require "inspec/utils/telemetry/global_methods"
 
-require 'inspec/plugin/v2'
-require 'inspec/plugin/v1'
+require "inspec/plugin/v2"
+require "inspec/plugin/v1"
 
 # all utils that may be required by legacy plugins
-require 'inspec/base_cli'
-require 'inspec/fetcher'
-require 'inspec/source_reader'
-require 'inspec/resource'
+require "inspec/base_cli"
+require "inspec/fetcher"
+require "inspec/source_reader"
+require "inspec/resource"
+require "inspec/resources"

data/lib/inspec/archive/tar.rb

@@ -1,13 +1,9 @@
-# encoding: utf-8
-# author: Christoph Hartmann
-# author: Dominik Richter
-
-require 'rubygems/package'
+require "rubygems/package"
 
 module Inspec::Archive
   class TarArchiveGenerator
     def archive(base_dir, files, archive)
-      File.open(archive, 'wb') do |file|
+      File.open(archive, "wb") do |file|
         Zlib::GzipWriter.wrap(file) do |gz|
           Gem::Package::TarWriter.new(gz) do |tar|
             files.each do |input_filename|

data/lib/inspec/archive/zip.rb

@@ -1,10 +1,6 @@
-# encoding: utf-8
-# author: Christoph Hartmann
-# author: Dominik Richter
-
-require 'rubygems'
-require 'zip'
-require 'pathname'
+require "rubygems"
+require "zip"
+require "pathname"
 
 module Inspec::Archive
   class ZipArchiveGenerator

data/lib/inspec/backend.rb

@@ -1,10 +1,9 @@
-# encoding: utf-8
 # copyright: 2015, Dominik Richter
-# author: Dominik Richter
-# author: Christoph Hartmann
 
-require 'train'
-require 'inspec/config'
+require "train"
+require "inspec/config"
+require "inspec/version"
+require "inspec/resource"
 
 module Inspec
   module Backend
@@ -28,7 +27,7 @@ module Inspec
 
       # Ruby internal for printing a nice name for this class
       def to_s
-        'Inspec::Backend::Class'
+        "Inspec::Backend::Class"
       end
 
       # Ruby internal for pretty-printing a summary for this class
@@ -57,15 +56,15 @@ module Inspec
       # Set caching settings. We always want to enable caching for
       # the Mock transport for testing.
       if config[:backend_cache] || config[:backend] == :mock
-        Inspec::Log.debug 'Option backend_cache is enabled'
+        Inspec::Log.debug "Option backend_cache is enabled"
         connection.enable_cache(:file)
         connection.enable_cache(:command)
       elsif config[:debug_shell]
-        Inspec::Log.debug 'Option backend_cache is disabled'
+        Inspec::Log.debug "Option backend_cache is disabled"
         connection.disable_cache(:file)
         connection.disable_cache(:command)
       else
-        Inspec::Log.debug 'Option backend_cache is disabled'
+        Inspec::Log.debug "Option backend_cache is disabled"
         connection.disable_cache(:file)
         connection.disable_cache(:command)
       end

data/lib/inspec/base_cli.rb

@@ -1,11 +1,8 @@
-# encoding: utf-8
-# author: Christoph Hartmann
-# author: Dominik Richter
-
-require 'thor'
-require 'inspec/log'
-require 'inspec/profile_vendor'
-require 'inspec/ui'
+require "thor"
+require "inspec/log"
+require "inspec/ui"
+require "inspec/config"
+require "inspec/utils/deprecation/global_method"
 
 # Allow end of options during array type parsing
 # https://github.com/erikhuda/thor/issues/631
@@ -34,107 +31,107 @@ module Inspec
 
     def self.target_options # rubocop:disable MethodLength
       option :target, aliases: :t, type: :string,
-        desc: 'Simple targeting option using URIs, e.g. ssh://user:pass@host:port'
+        desc: "Simple targeting option using URIs, e.g. ssh://user:pass@host:port"
       option :backend, aliases: :b, type: :string,
-        desc: 'Choose a backend: local, ssh, winrm, docker.'
+        desc: "Choose a backend: local, ssh, winrm, docker."
       option :host, type: :string,
-        desc: 'Specify a remote host which is tested.'
+        desc: "Specify a remote host which is tested."
       option :port, aliases: :p, type: :numeric,
-        desc: 'Specify the login port for a remote scan.'
+        desc: "Specify the login port for a remote scan."
       option :user, type: :string,
-        desc: 'The login user for a remote scan.'
+        desc: "The login user for a remote scan."
       option :password, type: :string, lazy_default: -1,
-        desc: 'Login password for a remote scan, if required.'
+        desc: "Login password for a remote scan, if required."
       option :enable_password, type: :string, lazy_default: -1,
-        desc: 'Password for enable mode on Cisco IOS devices.'
+        desc: "Password for enable mode on Cisco IOS devices."
       option :key_files, aliases: :i, type: :array,
-        desc: 'Login key or certificate file for a remote scan.'
+        desc: "Login key or certificate file for a remote scan."
       option :path, type: :string,
-        desc: 'Login path to use when connecting to the target (WinRM).'
+        desc: "Login path to use when connecting to the target (WinRM)."
       option :sudo, type: :boolean,
-        desc: 'Run scans with sudo. Only activates on Unix and non-root user.'
+        desc: "Run scans with sudo. Only activates on Unix and non-root user."
       option :sudo_password, type: :string, lazy_default: -1,
-        desc: 'Specify a sudo password, if it is required.'
+        desc: "Specify a sudo password, if it is required."
       option :sudo_options, type: :string,
-        desc: 'Additional sudo options for a remote scan.'
+        desc: "Additional sudo options for a remote scan."
       option :sudo_command, type: :string,
-        desc: 'Alternate command for sudo.'
+        desc: "Alternate command for sudo."
       option :shell, type: :boolean,
-        desc: 'Run scans in a subshell. Only activates on Unix.'
+        desc: "Run scans in a subshell. Only activates on Unix."
       option :shell_options, type: :string,
-        desc: 'Additional shell options.'
+        desc: "Additional shell options."
       option :shell_command, type: :string,
-        desc: 'Specify a particular shell to use.'
+        desc: "Specify a particular shell to use."
       option :ssl, type: :boolean,
-        desc: 'Use SSL for transport layer encryption (WinRM).'
+        desc: "Use SSL for transport layer encryption (WinRM)."
       option :self_signed, type: :boolean,
-        desc: 'Allow remote scans with self-signed certificates (WinRM).'
-      option :winrm_transport, type: :string, default: 'negotiate',
-        desc: 'Specify which transport to use, defaults to negotiate (WinRM).'
+        desc: "Allow remote scans with self-signed certificates (WinRM)."
+      option :winrm_transport, type: :string, default: "negotiate",
+        desc: "Specify which transport to use, defaults to negotiate (WinRM)."
       option :winrm_disable_sspi, type: :boolean,
-        desc: 'Whether to use disable sspi authentication, defaults to false (WinRM).'
+        desc: "Whether to use disable sspi authentication, defaults to false (WinRM)."
       option :winrm_basic_auth, type: :boolean,
-        desc: 'Whether to use basic authentication, defaults to false (WinRM).'
+        desc: "Whether to use basic authentication, defaults to false (WinRM)."
       option :config, type: :string,
-        desc: 'Read configuration from JSON file (`-` reads from stdin).'
+        desc: "Read configuration from JSON file (`-` reads from stdin)."
       option :json_config, type: :string, hide: true
       option :proxy_command, type: :string,
-        desc: 'Specifies the command to use to connect to the server'
+        desc: "Specifies the command to use to connect to the server"
       option :bastion_host, type: :string,
-        desc: 'Specifies the bastion host if applicable'
+        desc: "Specifies the bastion host if applicable"
       option :bastion_user, type: :string,
-        desc: 'Specifies the bastion user if applicable'
+        desc: "Specifies the bastion user if applicable"
       option :bastion_port, type: :string,
-        desc: 'Specifies the bastion port if applicable'
+        desc: "Specifies the bastion port if applicable"
       option :insecure, type: :boolean, default: false,
-        desc: 'Disable SSL verification on select targets'
+        desc: "Disable SSL verification on select targets"
       option :target_id, type: :string,
-        desc: 'Provide a ID which will be included on reports'
+        desc: "Provide a ID which will be included on reports"
     end
 
     def self.profile_options
       option :profiles_path, type: :string,
-        desc: 'Folder which contains referenced profiles.'
+        desc: "Folder which contains referenced profiles."
       option :vendor_cache, type: :string,
-        desc: 'Use the given path for caching dependencies. (default: ~/.inspec/cache)'
+        desc: "Use the given path for caching dependencies. (default: ~/.inspec/cache)"
     end
 
     def self.exec_options
       target_options
       profile_options
       option :controls, type: :array,
-        desc: 'A list of control names to run, or a list of /regexes/ to match against control names. Ignore all other tests.'
+        desc: "A list of control names to run, or a list of /regexes/ to match against control names. Ignore all other tests."
       option :reporter, type: :array,
-        banner: 'one two:/output/file/path',
-        desc: 'Enable one or more output reporters: cli, documentation, html, progress, json, json-min, json-rspec, junit, yaml'
+        banner: "one two:/output/file/path",
+        desc: "Enable one or more output reporters: cli, documentation, html, progress, json, json-min, json-rspec, junit, yaml"
       option :input_file, type: :array,
-        desc: 'Load one or more input files, a YAML file with values for the profile to use'
+        desc: "Load one or more input files, a YAML file with values for the profile to use"
       option :attrs, type: :array,
-        desc: 'Legacy name for --input-file - deprecated.'
+        desc: "Legacy name for --input-file - deprecated."
       option :create_lockfile, type: :boolean,
-        desc: 'Write out a lockfile based on this execution (unless one already exists)'
+        desc: "Write out a lockfile based on this execution (unless one already exists)"
       option :backend_cache, type: :boolean,
-        desc: 'Allow caching for backend command output. (default: true)'
+        desc: "Allow caching for backend command output. (default: true)"
       option :show_progress, type: :boolean,
-        desc: 'Show progress while executing tests.'
+        desc: "Show progress while executing tests."
       option :distinct_exit, type: :boolean, default: true,
-        desc: 'Exit with code 101 if any tests fail, and 100 if any are skipped (default).  If disabled, exit 0 on skips and 1 for failures.'
+        desc: "Exit with code 101 if any tests fail, and 100 if any are skipped (default).  If disabled, exit 0 on skips and 1 for failures."
     end
 
     def self.format_platform_info(params: {}, indent: 0, color: 39)
-      str = ''
-      params.each { |item, info|
+      str = ""
+      params.each do |item, info|
         data = info
 
         # Format Array for better output if applicable
-        data = data.join(', ') if data.is_a?(Array)
+        data = data.join(", ") if data.is_a?(Array)
 
         # Do not output fields of data is missing ('unknown' is fine)
         next if data.nil?
 
         data = "\e[1m\e[#{color}m#{data}\e[0m"
-        str << format("#{' ' * indent}%-10s %s\n", item.to_s.capitalize + ':', data)
-      }
+        str << format("#{' ' * indent}%-10s %s\n", item.to_s.capitalize + ":", data)
+      end
       str
     end
 
@@ -193,12 +190,12 @@ module Inspec
     private
 
     def suppress_log_output?(opts)
-      return false if opts['reporter'].nil?
-      match = %w{json json-min json-rspec json-automate junit html yaml documentation progress} & opts['reporter'].keys
+      return false if opts["reporter"].nil?
+      match = %w{json json-min json-rspec json-automate junit html yaml documentation progress} & opts["reporter"].keys
       unless match.empty?
         match.each do |m|
           # check to see if we are outputting to stdout
-          return true if opts['reporter'][m]['stdout'] == true
+          return true if opts["reporter"][m]["stdout"] == true
         end
       end
       false
@@ -220,7 +217,7 @@ module Inspec
       if valid.include?(level)
         l = level
       else
-        l = 'info'
+        l = "info"
       end
 
       Logger.const_get(l.upcase)
@@ -237,11 +234,13 @@ module Inspec
     end
 
     def vendor_deps(path, opts)
+      require "inspec/profile_vendor"
+
       profile_path = path || Dir.pwd
       profile_vendor = Inspec::ProfileVendor.new(profile_path)
 
       if (profile_vendor.cache_path.exist? || profile_vendor.lockfile.exist?) && !opts[:overwrite]
-        puts 'Profile is already vendored. Use --overwrite.'
+        puts "Profile is already vendored. Use --overwrite."
         return false
       end
 
@@ -257,23 +256,23 @@ module Inspec
       # logging singleton Inspec::Log. Eventually it would be nice to
       # move internal debug logging to use this logging singleton.
       #
-      loc = if o['log_location']
-              o['log_location']
+      loc = if o["log_location"]
+              o["log_location"]
             elsif suppress_log_output?(o)
-              STDERR
+              $stderr
             else
-              STDOUT
+              $stdout
             end
 
       Inspec::Log.init(loc)
-      Inspec::Log.level = get_log_level(o['log_level'])
+      Inspec::Log.level = get_log_level(o["log_level"])
 
       o[:logger] = Logger.new(loc)
       # output json if we have activated the json formatter
-      if o['log-format'] == 'json'
+      if o["log-format"] == "json"
         o[:logger].formatter = Logger::JSONFormatter.new
       end
-      o[:logger].level = get_log_level(o['log_level'])
+      o[:logger].level = get_log_level(o["log_level"])
     end
   end
 end