puppet 7.28.0-x86-mingw32 → 8.0.1-x86-mingw32

data/lib/puppet/parser/ast/top_level_construct.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The base class for AST nodes representing top level things:
 # hostclasses, definitions, and nodes.
 class Puppet::Parser::AST::TopLevelConstruct < Puppet::Parser::AST

data/lib/puppet/parser/ast.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The base class for the 3x "parse tree", now only used by the top level
 # constructs and the compiler.
 # Handles things like file name, line #, and also does the initialization

data/lib/puppet/parser/catalog_compiler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/loaders'
 require_relative '../../puppet/pops'
 

data/lib/puppet/parser/compiler/catalog_validator/relationship_validator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Parser::Compiler
   # Validator that asserts relationship metaparameters refer to valid resources
   class CatalogValidator::RelationshipValidator < CatalogValidator

data/lib/puppet/parser/compiler/catalog_validator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Abstract class for a catalog validator that can be registered with the compiler to run at
 # a certain stage.
 class Puppet::Parser::Compiler

data/lib/puppet/parser/compiler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'forwardable'
 
 require_relative '../../puppet/node'
@@ -579,7 +580,7 @@ class Puppet::Parser::Compiler
     @topscope.set_facts(facts_hash)
   end
 
-  SETTINGS = 'settings'.freeze
+  SETTINGS = 'settings'
 
   def create_settings_scope
     settings_type = create_settings_type

data/lib/puppet/parser/e4_parser_adapter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/pops'
 
 module Puppet

data/lib/puppet/parser/files.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Parser::Files
 
   module_function

data/lib/puppet/parser/functions/assert_type.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :assert_type,
   :type => :rvalue,

data/lib/puppet/parser/functions/binary_file.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :binary_file,
   :type => :rvalue,

data/lib/puppet/parser/functions/break.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :break,
   :arity => 0,

data/lib/puppet/parser/functions/contain.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Called within a class definition, establishes a containment
 # relationship with another class
 

data/lib/puppet/parser/functions/create_resources.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(:create_resources, :arity => -3, :doc => <<-'ENDHEREDOC') do |args|
     Converts a hash into a set of resources and adds them to the catalog.
     

data/lib/puppet/parser/functions/defined.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :defined,
   :type => :rvalue,

data/lib/puppet/parser/functions/dig.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :dig,
   :type => :rvalue,

data/lib/puppet/parser/functions/digest.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/checksums'
 Puppet::Parser::Functions::newfunction(:digest, :type => :rvalue, :arity => 1, :doc => "Returns a hash value from a provided string using the digest_algorithm setting from the Puppet config file.") do |args|
   algo = Puppet[:digest_algorithm]

data/lib/puppet/parser/functions/each.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :each,
   :type => :rvalue,

data/lib/puppet/parser/functions/epp.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(:epp, :type => :rvalue, :arity => -2, :doc =>
 "Evaluates an Embedded Puppet (EPP) template file and returns the rendered text
 result as a String.

data/lib/puppet/parser/functions/fail.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
     :fail,
     :arity => -1,

data/lib/puppet/parser/functions/file.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/file_system'
 
 Puppet::Parser::Functions::newfunction(

data/lib/puppet/parser/functions/filter.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :filter,
   :type => :rvalue,

data/lib/puppet/parser/functions/find_file.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :find_file,
   :type => :rvalue,

data/lib/puppet/parser/functions/fqdn_rand.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'digest/md5'
 require 'digest/sha2'
 

data/lib/puppet/parser/functions/generate.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Runs an external command and returns the results
 Puppet::Parser::Functions::newfunction(:generate, :arity => -2, :type => :rvalue,
     :doc => "Calls an external command on the Puppet master and returns

data/lib/puppet/parser/functions/hiera.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera_puppet'
 
 module Puppet::Parser::Functions

data/lib/puppet/parser/functions/hiera_array.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera_puppet'
 
 module Puppet::Parser::Functions

data/lib/puppet/parser/functions/hiera_hash.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera_puppet'
 
 module Puppet::Parser::Functions

data/lib/puppet/parser/functions/hiera_include.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'hiera_puppet'
 
 module Puppet::Parser::Functions

data/lib/puppet/parser/functions/include.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Include the specified classes
 Puppet::Parser::Functions::newfunction(:include, :arity => -2, :doc =>
 "Declares one or more classes, causing the resources in them to be

data/lib/puppet/parser/functions/inline_epp.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(:inline_epp, :type => :rvalue, :arity => -2, :doc =>
 "Evaluates an Embedded Puppet (EPP) template string and returns the rendered
 text result as a String.

data/lib/puppet/parser/functions/inline_template.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(:inline_template, :type => :rvalue, :arity => -2, :doc =>
   "Evaluate a template string and return its value.  See
   [the templating docs](https://puppet.com/docs/puppet/latest/lang_template.html) for

data/lib/puppet/parser/functions/lest.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :lest,
   :type => :rvalue,

data/lib/puppet/parser/functions/lookup.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Parser::Functions
   newfunction(:lookup, :type => :rvalue, :arity => -2, :doc => <<-'ENDHEREDOC') do |args|
 Uses the Puppet lookup system to retrieve a value for a given key. By default,

data/lib/puppet/parser/functions/map.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :map,
   :type => :rvalue,

data/lib/puppet/parser/functions/match.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :match,
   :arity => 2,

data/lib/puppet/parser/functions/md5.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'digest/md5'
 
 Puppet::Parser::Functions::newfunction(:md5, :type => :rvalue, :arity => 1, :doc => "Returns a MD5 hash value from a provided string.") do |args|

data/lib/puppet/parser/functions/new.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :new,
   :type => :rvalue,

data/lib/puppet/parser/functions/next.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :next,
   :arity => -2,

data/lib/puppet/parser/functions/realize.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This is just syntactic sugar for a collection, although it will generally
 # be a good bit faster.
 

data/lib/puppet/parser/functions/reduce.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :reduce,
   :type => :rvalue,

data/lib/puppet/parser/functions/regsubst.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (C) 2009 Thomas Bellman
 #
 # Permission is hereby granted, free of charge, to any person obtaining

data/lib/puppet/parser/functions/require.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Requires the specified classes
 
   Puppet::Parser::Functions::newfunction(

data/lib/puppet/parser/functions/return.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :return,
   :arity => -2,

data/lib/puppet/parser/functions/reverse_each.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :reverse_each,
   :type => :rvalue,

data/lib/puppet/parser/functions/scanf.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'scanf'
 
 Puppet::Parser::Functions::newfunction(

data/lib/puppet/parser/functions/sha1.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'digest/sha1'
 
 Puppet::Parser::Functions::newfunction(:sha1, :type => :rvalue, :arity => 1, :doc => "Returns a SHA1 hash value from a provided string.") do |args|

data/lib/puppet/parser/functions/sha256.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'digest/sha2'
 
 Puppet::Parser::Functions::newfunction(:sha256, :type => :rvalue, :arity => 1, :doc => "Returns a SHA256 hash value from a provided string.") do |args|

data/lib/puppet/parser/functions/shellquote.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (C) 2009 Thomas Bellman
 #
 # Permission is hereby granted, free of charge, to any person obtaining

data/lib/puppet/parser/functions/slice.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :slice,
   :type => :rvalue,

data/lib/puppet/parser/functions/split.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Parser::Functions
 
   newfunction(

data/lib/puppet/parser/functions/sprintf.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Copyright (C) 2009 Thomas Bellman
 #
 # Permission is hereby granted, free of charge, to any person obtaining

data/lib/puppet/parser/functions/step.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :step,
   :type => :rvalue,

data/lib/puppet/parser/functions/strftime.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :strftime,
   :type => :rvalue,

data/lib/puppet/parser/functions/tag.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Tag the current scope with each passed name
 Puppet::Parser::Functions::newfunction(:tag, :arity => -2, :doc => "Add the specified tags to the containing class
   or definition.  All contained objects will then acquire that tag, also.

data/lib/puppet/parser/functions/tagged.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Test whether a given tag is set.  This functions as a big OR -- if any of the specified tags are unset, we return false.
 Puppet::Parser::Functions::newfunction(:tagged, :type => :rvalue, :arity => -2, :doc => "A boolean function that
   tells you whether the current container is tagged with the specified tags.

data/lib/puppet/parser/functions/template.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(:template, :type => :rvalue, :arity => -2, :doc =>
   "Loads an ERB template from a module, evaluates it, and returns the resulting
   value as a string.

data/lib/puppet/parser/functions/then.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :then,
   :type => :rvalue,

data/lib/puppet/parser/functions/type.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :type,
   :type => :rvalue,

data/lib/puppet/parser/functions/versioncmp.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/package'
 
 Puppet::Parser::Functions::newfunction( :versioncmp, :type => :rvalue, :arity => 2, :doc =>

data/lib/puppet/parser/functions/with.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Parser::Functions::newfunction(
   :with,
   :type => :rvalue,

data/lib/puppet/parser/functions.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/autoload'
 require_relative '../../puppet/parser/scope'
 require_relative '../../puppet/pops/adaptable'
@@ -257,7 +258,7 @@ module Puppet::Parser::Functions
   def self.functiondocs(environment = Puppet.lookup(:current_environment))
     autoloader.delegatee.loadall(environment)
 
-    ret = ""
+    ret = String.new
 
     merged_functions(environment).sort { |a,b| a[0].to_s <=> b[0].to_s }.each do |name, hash|
       ret << "#{name}\n#{"-" * name.to_s.length}\n"

data/lib/puppet/parser/parser_factory.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet; end
 
 module Puppet::Parser

data/lib/puppet/parser/relationship.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Parser::Relationship
   attr_accessor :source, :target, :type
 

data/lib/puppet/parser/resource/param.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The parameters we stick in Resources.
 class Puppet::Parser::Resource::Param
   include Puppet::Util

data/lib/puppet/parser/resource.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/resource'
 
 # The primary difference between this class and its

data/lib/puppet/parser/scope.rb

@@ -535,7 +535,12 @@ class Puppet::Parser::Scope
         Puppet.warn_once(UNDEFINED_VARIABLES_KIND, _("Variable: %{name}") % { name: name },
         _("Undefined variable '%{name}'; %{reason}") % { name: name, reason: reason } )
       when :error
-        raise ArgumentError, _("Undefined variable '%{name}'; %{reason}") % { name: name, reason: reason }
+        if Puppet.lookup(:avoid_hiera_interpolation_errors){false}
+          Puppet.warn_once(UNDEFINED_VARIABLES_KIND, _("Variable: %{name}") % { name: name },
+          _("Interpolation failed with '%{name}', but compilation continuing; %{reason}") % { name: name, reason: reason } )
+        else
+          raise ArgumentError, _("Undefined variable '%{name}'; %{reason}") % { name: name, reason: reason }
+        end
       end
     end
     nil

data/lib/puppet/parser/script_compiler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/loaders'
 require_relative '../../puppet/pops'
 

data/lib/puppet/parser/type_loader.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'find'
 require 'forwardable'
 require_relative '../../puppet/parser/parser_factory'

data/lib/puppet/parser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # is only needed to create the name space
 module Puppet::Parser; end
 

data/lib/puppet/plugins/configuration.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Configures the Puppet Plugins, by registering extension points
 # and default implementations.
 #

data/lib/puppet/plugins/syntax_checkers.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Plugins
 module SyntaxCheckers
 

data/lib/puppet/plugins.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The Puppet Plugins module defines extension points where plugins can be configured
 # to add or modify puppet's behavior. See the respective classes in this module for more
 # details.

data/lib/puppet/pops/adaptable.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Adaptable is a mix-in module that adds adaptability to a class.
 # This means that an adapter can
 # associate itself with an instance of the class and store additional data/have behavior.

data/lib/puppet/pops/adapters.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # The Adapters module contains adapters for Documentation, Origin, SourcePosition, and Loader.
 #
 module Puppet::Pops

data/lib/puppet/pops/evaluator/access_operator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Evaluator
 # AccessOperator handles operator []
@@ -657,7 +658,7 @@ class AccessOperator
     return result_type_array ? result : result.pop
   end
 
-  NS = '::'.freeze
+  NS = '::'
 
   def access_PClassType(o, scope, keys)
     blamed = keys.size == 0 ? @semantic : @semantic.keys[0]

data/lib/puppet/pops/evaluator/callable_signature.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # CallableSignature
 # ===
 # A CallableSignature describes how something callable expects to be called.

data/lib/puppet/pops/evaluator/closure.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Evaluator
   class Jumper < Exception
@@ -163,7 +164,7 @@ class Closure < CallableSignature
     'unsupported_block'
   end
 
-  CLOSURE_NAME = 'lambda'.freeze
+  CLOSURE_NAME = 'lambda'
 
   # @api public
   def closure_name()

data/lib/puppet/pops/evaluator/collector_transformer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Evaluator
 class CollectorTransformer

data/lib/puppet/pops/evaluator/collectors/abstract_collector.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Pops::Evaluator::Collectors::AbstractCollector
   attr_reader :scope
 

data/lib/puppet/pops/evaluator/collectors/catalog_collector.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Pops::Evaluator::Collectors::CatalogCollector < Puppet::Pops::Evaluator::Collectors::AbstractCollector
 
   # Creates a CatalogCollector using the AbstractCollector's 

data/lib/puppet/pops/evaluator/collectors/exported_collector.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Pops::Evaluator::Collectors::ExportedCollector < Puppet::Pops::Evaluator::Collectors::AbstractCollector
 
   # Creates an ExportedCollector using the AbstractCollector's

data/lib/puppet/pops/evaluator/collectors/fixed_set_collector.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Pops::Evaluator::Collectors::FixedSetCollector < Puppet::Pops::Evaluator::Collectors::AbstractCollector
 
   # Creates a FixedSetCollector using the AbstractCollector constructor

data/lib/puppet/pops/evaluator/compare_operator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Pops
 module Evaluator
 # Compares the puppet DSL way

data/lib/puppet/pops/evaluator/deferred_resolver.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/parser/script_compiler'
 
 module Puppet::Pops
@@ -9,21 +10,15 @@ class DeferredValue
   end
 
   def resolve
-    val = @proc.call
-    # Deferred sensitive values will be marked as such in resolve_futures()
-    if val.is_a?(Puppet::Pops::Types::PSensitiveType::Sensitive)
-      val.unwrap
-    else
-      val
-    end
+    @proc.call
   end
 end
 
 # Utility class to help resolve instances of Puppet::Pops::Types::PDeferredType::Deferred
 #
 class DeferredResolver
-  DOLLAR = '$'.freeze
-  DIG    = 'dig'.freeze
+  DOLLAR = '$'
+  DIG    = 'dig'
 
   # Resolves and replaces all Deferred values in a catalog's resource attributes
   # found as direct values or nested inside Array, Hash or Sensitive values.
@@ -93,12 +88,8 @@ class DeferredResolver
         #
         if resolved.is_a?(Puppet::Pops::Types::PSensitiveType::Sensitive)
           resolved = resolved.unwrap
-          mark_sensitive_parameters(r, k)
-        # If the value is a DeferredValue and it has an argument of type PSensitiveType, mark it as sensitive
-        # The DeferredValue.resolve method will unwrap it during catalog application
-        elsif resolved.is_a?(Puppet::Pops::Evaluator::DeferredValue)
-          if v.arguments.any? {|arg| arg.is_a?(Puppet::Pops::Types::PSensitiveType)}
-            mark_sensitive_parameters(r, k)
+          unless r.sensitive_parameters.include?(k.to_sym)
+            r.sensitive_parameters = (r.sensitive_parameters + [k.to_sym]).freeze
           end
         end
         overrides[ k ] = resolved
@@ -107,13 +98,6 @@ class DeferredResolver
     end
   end
 
-  def mark_sensitive_parameters(r, k)
-    unless r.sensitive_parameters.include?(k.to_sym)
-      r.sensitive_parameters = (r.sensitive_parameters + [k.to_sym]).freeze
-    end
-  end
-  private :mark_sensitive_parameters
-
   def resolve(x)
     if x.class == @deferred_class
       resolve_future(x)

data/lib/puppet/pops/evaluator/epp_evaluator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Handler of Epp call/evaluation from the epp and inline_epp functions
 #
 class Puppet::Pops::Evaluator::EppEvaluator