facter 4.0.24 → 4.0.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a87a5fda1ec732928ca868e4df803b7d0b9d68d95f1efd366eeb6497550beff
-  data.tar.gz: b324cde69b928d3a21b43d4f40a973b0086a56eba1a30eb5392f1d515513ee52
+  metadata.gz: e212e7eb5f2af98a17f55977fe98b41d7a804f8efdfe5f7bcc20f9c0627a7334
+  data.tar.gz: 6e6c8734fdf45b0a5d088bbb56c4c489f2b1cde69770ba887a01c8ee973431b3
 SHA512:
-  metadata.gz: bd84d157b62b33d1425d971eed5b94dd3c0b4f10e98755bb417b511c4baf5a8f2b48ee53c2ec2a80c7052023b324303ea965231a8bebaa66ca0c9e4553425d84
-  data.tar.gz: 8f9ff642fd2214cc77e6e8f7c21287b2265f0a604eba6b707a201e9eef2e77215451562e388f9ef017bc9917219bf58d377bf6e79150cbfa52403569e9ecb23c
+  metadata.gz: be1b614104b6f08789b49c73c9948eb6687bee06cb73722a5749d268bb5927bb1bc82ec29944a03ed07f5520318620ce8ee567e32a84d423e124d959a3931354
+  data.tar.gz: 90da2d17a9e80b5aeb4104c51d5bfa3912fa3f3560f04ad965bf4df15f830eef634d92aab71b93a067a8e4257ac733f21bf9b8ba562a28f8c00f0f7d066388f1

data/bin/facter

@@ -2,5 +2,10 @@
 # frozen_string_literal: true
 
 require 'pathname'
-ROOT_DIR = Pathname.new(File.expand_path('..', __dir__)) unless defined?(ROOT_DIR)
-require "#{ROOT_DIR}/lib/framework/cli/cli_launcher.rb"
+require 'facter/framework/cli/cli_launcher.rb'
+
+cli_launcher = CliLauncher.new(ARGV)
+
+cli_launcher.validate_options
+cli_launcher.prepare_arguments
+cli_launcher.start

data/lib/facter.rb

@@ -3,10 +3,9 @@
 require 'pathname'
 require_relative 'util/api_debugger' if ENV['API_DEBUG']
 
-ROOT_DIR = Pathname.new(File.expand_path('..', __dir__)) unless defined?(ROOT_DIR)
-
-require "#{ROOT_DIR}/lib/framework/core/file_loader"
-require "#{ROOT_DIR}/lib/framework/core/options/options_validator"
+require 'facter/version'
+require 'facter/framework/core/file_loader'
+require 'facter/framework/core/options/options_validator'
 
 module Facter
   class ResolveCustomFactError < StandardError; end
@@ -228,8 +227,7 @@ module Facter
     #
     # @api public
     def version
-      version_file = ::File.join(ROOT_DIR, 'VERSION')
-      ::File.read(version_file).strip
+      Facter::VERSION
     end
 
     # Gets a hash mapping fact names to their values
@@ -245,7 +243,7 @@ module Facter
       SessionCache.invalidate_all_caches
       fact_formatter = Facter::FormatterFactory.build(Facter::Options.get)
 
-      status = error_check(args, resolved_facts)
+      status = error_check(resolved_facts)
 
       [fact_formatter.format(resolved_facts), status || 0]
     end
@@ -289,6 +287,9 @@ module Facter
       user_query = user_query.to_s
       resolved_facts = Facter::FactManager.instance.resolve_facts([user_query])
       SessionCache.invalidate_all_caches
+      # we must make a distinction between custom facts that return nil and nil facts
+      # Nil facts should not be packaged as ResolvedFacts! (add_fact_to_searched_facts packages facts)
+      resolved_facts = resolved_facts.reject { |fact| fact.type == :nil }
       fact_collection = FactCollection.new.build_fact_collection!(resolved_facts)
       splitted_user_query = Facter::Utils.split_user_query(user_query)
 
@@ -310,9 +311,10 @@ module Facter
     #  facts that are not found or resolved, otherwise it will return nil
     #
     # @api private
-    def error_check(args, resolved_facts)
+    def error_check(resolved_facts)
       if Options[:strict]
-        missing_names = args - resolved_facts.map(&:user_query).uniq
+        missing_names = resolved_facts.select { |fact| fact.type == :nil }.map(&:user_query)
+
         if missing_names.count.positive?
           status = 1
           log_errors(missing_names)

data/lib/{custom_facts → facter/custom_facts}/core/aggregate.rb

@@ -48,6 +48,10 @@ module Facter
         @deps = LegacyFacter::Core::DirectedGraph.new
       end
 
+      def <=>(other)
+        weight <=> other.weight
+      end
+
       def options(options)
         accepted_options = %i[name timeout weight fact_type]
         accepted_options.each do |option_name|

data/lib/{custom_facts → facter/custom_facts}/core/execution/base.rb

@@ -6,6 +6,8 @@ module Facter
       class Base
         STDERR_MESSAGE = 'Command %s resulted with the following stderr message: %s'
 
+        # This is part of the public API. No race condition can happen
+        # here because custom facts are executed sequentially
         def with_env(values)
           old = {}
           values.each do |var, value|
@@ -38,27 +40,22 @@ module Facter
           expand = options.fetch(:expand, true)
           logger = options[:logger]
 
-          # Set LC_ALL and LANG to force i18n to C for the duration of this exec;
-          # this ensures that any code that parses the
-          # output of the command can expect it to be in a consistent / predictable format / locale
-          with_env 'LC_ALL' => 'C', 'LANG' => 'C' do
-            expanded_command = if !expand && builtin_command?(command) || logger
-                                 command
-                               else
-                                 expand_command(command)
-                               end
+          expanded_command = if !expand && builtin_command?(command) || logger
+                               command
+                             else
+                               expand_command(command)
+                             end
 
-            if expanded_command.nil?
-              if on_fail == :raise
-                raise Facter::Core::Execution::ExecutionFailure.new,
-                      "Could not execute '#{command}': command not found"
-              end
-
-              return on_fail
+          if expanded_command.nil?
+            if on_fail == :raise
+              raise Facter::Core::Execution::ExecutionFailure.new,
+                    "Could not execute '#{command}': command not found"
             end
 
-            execute_command(expanded_command, on_fail, logger)
+            return on_fail
           end
+
+          execute_command(expanded_command, on_fail, logger)
         end
 
         private
@@ -82,7 +79,10 @@ module Facter
 
         def execute_command(command, on_fail, logger = nil)
           begin
-            out, stderr, _status_ = Open3.capture3(command.to_s)
+            # Set LC_ALL and LANG to force i18n to C for the duration of this exec;
+            # this ensures that any code that parses the
+            # output of the command can expect it to be in a consistent / predictable format / locale
+            out, stderr, _status_ = Open3.capture3({ 'LC_ALL' => 'C', 'LANG' => 'C' }, command.to_s)
             log_stderr(stderr, command, logger)
           rescue StandardError => e
             return '' if logger

data/lib/facter/custom_facts/core/file_loader.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'facter/custom_facts/core/logging'
+require 'facter/custom_facts/core/legacy_facter'
+require 'facter/custom_facts/util/fact'
+require 'facter/custom_facts/util/collection'
+require 'facter/custom_facts/util/fact'
+require 'facter/custom_facts/util/loader'
+require 'facter/custom_facts/core/execution/base'
+require 'facter/custom_facts/core/execution/windows'
+require 'facter/custom_facts/core/execution/posix'
+require 'facter/custom_facts/util/values'
+require 'facter/custom_facts/util/confine'
+
+require 'facter/custom_facts/util/config'
+
+require 'facter/custom_facts/util/normalization'
+require 'facter/custom_facts/core/execution'
+require 'facter/custom_facts/core/resolvable'
+require 'facter/custom_facts/core/suitable'
+require 'facter/custom_facts/util/resolution'
+require 'facter/custom_facts/core/directed_graph'
+require 'facter/custom_facts/core/resolvable'
+require 'facter/custom_facts/core/aggregate'
+require 'facter/custom_facts/util/composite_loader'
+require 'facter/custom_facts/util/parser'
+require 'facter/custom_facts/util/directory_loader'
+require 'facter/custom_facts/util/nothing_loader'
+require 'facter/custom_facts/util/nothing_loader'

data/lib/{custom_facts → facter/custom_facts}/core/legacy_facter.rb

@@ -18,8 +18,7 @@
 
 require 'pathname'
 
-ROOT_DIR = Pathname.new(File.expand_path('../../../', __dir__)) unless defined?(ROOT_DIR)
-require "#{ROOT_DIR}/lib/custom_facts/core/file_loader"
+require 'facter/custom_facts/core/file_loader'
 
 # Functions as a hash of 'facts' about your system system, such as MAC
 # address, IP address, architecture, etc.

data/lib/{custom_facts → facter/custom_facts}/util/collection.rb

@@ -95,8 +95,7 @@ module LegacyFacter
         return @external_facts unless @external_facts.nil?
 
         load_external_facts
-        external_facts = @facts.select { |_k, v| v.options[:fact_type] == :external }
-        @external_facts = Facter::Utils.deep_copy(external_facts.keys)
+        @external_facts = @facts.select { |_k, v| v.options[:fact_type] == :external }
       end
 
       def invalidate_custom_facts
@@ -148,11 +147,7 @@ module LegacyFacter
 
       def value(name)
         fact = fact(name)
-        fact_value = fact&.value
-        return Facter.core_value(name) if fact_value.nil?
-
-        core_value = Facter.core_value(name) if fact.used_resolution_weight <= 0
-        core_value.nil? ? fact_value : core_value
+        fact&.value
       end
 
       private

data/lib/{custom_facts → facter/custom_facts}/util/config.rb

@@ -8,7 +8,7 @@ module LegacyFacter
   module Util
     module Config
       def self.ext_fact_loader
-        @ext_fact_loader || LegacyFacter::Util::DirectoryLoader.default_loader
+        @ext_fact_loader || LegacyFacter::Util::DirectoryLoader.new
       end
 
       def self.ext_fact_loader=(loader)

data/lib/facter/custom_facts/util/directory_loader.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+# A Facter plugin that loads external facts.
+#
+# Default Unix Directories:
+# /opt/puppetlabs/custom_facts/facts.d, /etc/custom_facts/facts.d, /etc/puppetlabs/custom_facts/facts.d
+#
+# Beginning with Facter 3, only /opt/puppetlabs/custom_facts/facts.d will be a default external fact
+# directory in Unix.
+#
+# Default Windows Direcotires:
+# C:\ProgramData\Puppetlabs\custom_facts\facts.d (2008)
+# C:\Documents and Settings\All Users\Application Data\Puppetlabs\custom_facts\facts.d (2003)
+#
+# Can also load from command-line specified directory
+#
+# Facts can be in the form of JSON, YAML or Text files
+# and any executable that returns key=value pairs.
+
+require 'yaml'
+
+module LegacyFacter
+  module Util
+    class DirectoryLoader
+      class NoSuchDirectoryError < RuntimeError
+      end
+
+      # This value makes it highly likely that external facts will take
+      # precedence over all other facts
+      EXTERNAL_FACT_WEIGHT = 10_000
+
+      # Directory for fact loading
+      attr_reader :directories
+
+      def initialize(dir = LegacyFacter::Util::Config.external_facts_dirs, weight = EXTERNAL_FACT_WEIGHT)
+        @directories = [dir].flatten
+        @weight = weight
+      end
+
+      # Load facts from files in fact directory using the relevant parser classes to
+      # parse them.
+      def load(collection)
+        weight = @weight
+
+        searched_facts, cached_facts = load_directory_entries(collection)
+
+        load_cached_facts(collection, cached_facts, weight)
+
+        load_searched_facts(collection, searched_facts, weight)
+      end
+
+      private
+
+      def load_directory_entries(_collection)
+        cm = Facter::CacheManager.new
+        facts = []
+        entries.each do |file|
+          basename = File.basename(file)
+          if facts.find { |f| f.name == basename } && cm.group_cached?(basename)
+            Facter.log_exception(Exception.new("Caching is enabled for group \"#{basename}\" while "\
+              'there are at least two external facts files with the same filename'))
+          else
+            searched_fact = Facter::SearchedFact.new(basename, nil, [], nil, :file)
+            searched_fact.file = file
+            facts << searched_fact
+          end
+        end
+
+        cm.resolve_facts(facts)
+      end
+
+      def load_cached_facts(collection, cached_facts, weight)
+        cached_facts.each do |cached_fact|
+          collection.add(cached_fact.name, value: cached_fact.value, fact_type: :external,
+                                           file: cached_fact.file) { has_weight(weight) }
+        end
+      end
+
+      def load_searched_facts(collection, searched_facts, weight)
+        searched_facts.each do |fact|
+          parser = LegacyFacter::Util::Parser.parser_for(fact.file)
+          next if parser.nil?
+
+          data = parser.results
+          if data == false
+            LegacyFacter.warn "Could not interpret fact file #{fact.file}"
+          elsif (data == {}) || data.nil?
+            LegacyFacter.warn "Fact file #{fact.file} was parsed but returned an empty data set"
+          else
+            data.each do |p, v|
+              collection.add(p, value: v, fact_type: :external,
+                                file: fact.file) { has_weight(weight) }
+            end
+          end
+        end
+      end
+
+      def entries
+        dirs = @directories.select { |directory| File.directory?(directory) }.map do |directory|
+          Dir.entries(directory).map { |directory_entry| File.join(directory, directory_entry) }
+        end
+        dirs.flatten.select { |f| should_parse?(f) }
+      rescue Errno::ENOENT
+        []
+      end
+
+      def should_parse?(file)
+        File.basename(file) !~ /^\./
+      end
+    end
+  end
+end

data/lib/{custom_facts → facter/custom_facts}/util/fact.rb

@@ -123,8 +123,10 @@ module Facter
           announce_when_no_suitable_resolution(suitable_resolutions)
           announce_when_no_value_found(@value)
 
-          @value
+          @value = resolve_value
         end
+
+        @value
       end
 
       # @api private
@@ -138,6 +140,13 @@ module Facter
 
       private
 
+      def resolve_value
+        return Facter.core_value(name) if @value.nil?
+
+        core_value = Facter.core_value(name) if @used_resolution_weight <= 0
+        core_value.nil? ? @value : core_value
+      end
+
       # Are we in the midst of a search?
       def searching?
         @searching

data/lib/{custom_facts → facter/custom_facts}/util/loader.rb

@@ -39,6 +39,8 @@ module LegacyFacter
 
         paths = search_path
         paths&.each do |dir|
+          # clean the search path of wrong slashes and backslashes
+          dir = dir.gsub(%r{[\/\\]+}, File::SEPARATOR)
           # dir is already an absolute path
           Dir.glob(File.join(dir, '*.rb')).each do |path|
             # exclude dirs that end with .rb
@@ -141,7 +143,7 @@ module LegacyFacter
           # match it.
           next if fact && (env_name != fact)
 
-          LegacyFacter.add(Regexp.last_match(1)) do
+          LegacyFacter.add(Regexp.last_match(1), fact_type: :external) do
             has_weight 1_000_000
             setcode { value }
           end

data/lib/{custom_facts → facter/custom_facts}/util/resolution.rb

@@ -86,7 +86,7 @@ module Facter
       end
 
       def options(options)
-        accepted_options = %i[name value timeout weight fact_type]
+        accepted_options = %i[name value timeout weight fact_type file]
 
         accepted_options.each do |option_name|
           instance_variable_set("@#{option_name}", options.delete(option_name)) if options.key?(option_name)

data/lib/facter/facts/aix/aio_agent_version.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Facts
+  module Aix
+    class AioAgentVersion
+      FACT_NAME = 'aio_agent_version'
+
+      def call_the_resolver
+        fact_value = Facter::Resolvers::AioAgentVersion.resolve(:aio_agent_version)
+        Facter::ResolvedFact.new(FACT_NAME, fact_value)
+      end
+    end
+  end
+end