puppet 7.3.0-x86-mingw32 → 7.7.0-x86-mingw32

data/lib/puppet/feature/bolt.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:bolt, :libs => ['bolt'])

data/lib/puppet/feature/cfpropertylist.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:cfpropertylist, :libs => ['cfpropertylist'])

data/lib/puppet/feature/eventlog.rb

@@ -1,4 +1,4 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 if Puppet::Util::Platform.windows?
   Puppet.features.add(:eventlog)

data/lib/puppet/feature/hiera_eyaml.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:hiera_eyaml, :libs => ['hiera/backend/eyaml/parser/parser'])

data/lib/puppet/feature/hocon.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:hocon, :libs => ['hocon'])

data/lib/puppet/feature/libuser.rb

@@ -1,5 +1,5 @@
-require 'puppet/util/feature'
-require 'puppet/util/libuser'
+require_relative '../../puppet/util/feature'
+require_relative '../../puppet/util/libuser'
 
 Puppet.features.add(:libuser) {
    File.executable?("/usr/sbin/lgroupadd") and

data/lib/puppet/feature/msgpack.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:msgpack, :libs => ["msgpack"])

data/lib/puppet/feature/pe_license.rb

@@ -1,4 +1,4 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 #Is the pe license library installed providing the ability to read licenses.
 Puppet.features.add(:pe_license, :libs => %{pe_license})

data/lib/puppet/feature/selinux.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:selinux, :libs => ["selinux"])

data/lib/puppet/feature/ssh.rb

@@ -1,3 +1,3 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add(:ssh, :libs => %{net/ssh})

data/lib/puppet/feature/telnet.rb

@@ -1,4 +1,4 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 Puppet.features.add :telnet do
   begin

data/lib/puppet/feature/zlib.rb

@@ -1,4 +1,4 @@
-require 'puppet/util/feature'
+require_relative '../../puppet/util/feature'
 
 # We want this to load if possible, but it's not automatically
 # required.

data/lib/puppet/ffi/posix.rb

@@ -3,8 +3,8 @@ require 'ffi'
 module Puppet
   module FFI
     module POSIX
-      require 'puppet/ffi/posix/functions'
-      require 'puppet/ffi/posix/constants'
+      require_relative 'posix/functions'
+      require_relative 'posix/constants'
     end
   end
 end

data/lib/puppet/ffi/posix/constants.rb

@@ -1,4 +1,4 @@
-require 'puppet/ffi/posix'
+require_relative '../../../puppet/ffi/posix'
 
 module Puppet::FFI::POSIX
   module Constants

data/lib/puppet/ffi/posix/functions.rb

@@ -1,4 +1,4 @@
-require 'puppet/ffi/posix'
+require_relative '../../../puppet/ffi/posix'
 
 module Puppet::FFI::POSIX
   module Functions

data/lib/puppet/ffi/windows.rb

@@ -3,10 +3,10 @@ require 'ffi'
 module Puppet
   module FFI
     module Windows
-      require 'puppet/ffi/windows/api_types'
-      require 'puppet/ffi/windows/constants'
-      require 'puppet/ffi/windows/structs'
-      require 'puppet/ffi/windows/functions'
+      require_relative 'windows/api_types'
+      require_relative 'windows/constants'
+      require_relative 'windows/structs'
+      require_relative 'windows/functions'
     end
   end
 end

data/lib/puppet/ffi/windows/api_types.rb

@@ -1,5 +1,5 @@
-require 'puppet/ffi/windows'
-require 'puppet/util/windows/string'
+require_relative '../../../puppet/ffi/windows'
+require_relative '../../../puppet/util/windows/string'
 
 module Puppet::FFI::Windows
   module APITypes
@@ -20,7 +20,7 @@ module Puppet::FFI::Windows
 
     class ::FFI::Pointer
       NULL_HANDLE = 0
-      WCHAR_NULL = "\0\0".encode('UTF-16LE').freeze
+      WCHAR_NULL = "\0\0".force_encoding('UTF-16LE').freeze
 
       def self.from_string_to_wide_string(str, &block)
         str = Puppet::Util::Windows::String.wide_string(str)

data/lib/puppet/ffi/windows/constants.rb

@@ -1,4 +1,4 @@
-require 'puppet/ffi/windows'
+require_relative '../../../puppet/ffi/windows'
 
 module Puppet::FFI::Windows
   module Constants
@@ -375,7 +375,7 @@ module Puppet::FFI::Windows
     SERVICE_CONFIG_PRESHUTDOWN_INFO         = 0x00000007
     SERVICE_CONFIG_TRIGGER_INFO             = 0x00000008
     SERVICE_CONFIG_PREFERRED_NODE           = 0x00000009
-    SERVICE_CONFIG_LAUNCH_PROTECTED         = 0x00000012
+    SERVICE_CONFIG_LAUNCH_PROTECTED         = 0x0000000C
     SERVICE_NO_CHANGE                       = 0xffffffff
     SERVICE_CONFIG_TYPES = {
       SERVICE_CONFIG_DESCRIPTION => :SERVICE_CONFIG_DESCRIPTION,

data/lib/puppet/ffi/windows/functions.rb

@@ -1,4 +1,4 @@
-require 'puppet/ffi/windows'
+require_relative '../../../puppet/ffi/windows'
 
 module Puppet::FFI::Windows
   module Functions

data/lib/puppet/ffi/windows/structs.rb

@@ -1,5 +1,5 @@
 # coding: utf-8
-require 'puppet/ffi/windows'
+require_relative '../../../puppet/ffi/windows'
 
 module Puppet::FFI::Windows
   module Structs

data/lib/puppet/file_bucket/dipper.rb

@@ -1,8 +1,8 @@
 require 'pathname'
-require 'puppet/file_bucket'
-require 'puppet/file_bucket/file'
-require 'puppet/indirector/request'
-require 'puppet/util/diff'
+require_relative '../../puppet/file_bucket'
+require_relative '../../puppet/file_bucket/file'
+require_relative '../../puppet/indirector/request'
+require_relative '../../puppet/util/diff'
 require 'tempfile'
 
 class Puppet::FileBucket::Dipper

data/lib/puppet/file_bucket/file.rb

@@ -1,6 +1,6 @@
-require 'puppet/file_bucket'
-require 'puppet/indirector'
-require 'puppet/util/checksums'
+require_relative '../../puppet/file_bucket'
+require_relative '../../puppet/indirector'
+require_relative '../../puppet/util/checksums'
 require 'digest/md5'
 require 'stringio'
 

data/lib/puppet/file_serving/base.rb

@@ -1,5 +1,5 @@
-require 'puppet/file_serving'
-require 'puppet/util'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/util'
 
 # The base class for Content and Metadata; provides common
 # functionality like the behaviour around links.

data/lib/puppet/file_serving/configuration.rb

@@ -1,15 +1,15 @@
-require 'puppet'
-require 'puppet/file_serving'
-require 'puppet/file_serving/mount'
-require 'puppet/file_serving/mount/file'
-require 'puppet/file_serving/mount/modules'
-require 'puppet/file_serving/mount/plugins'
-require 'puppet/file_serving/mount/locales'
-require 'puppet/file_serving/mount/pluginfacts'
-require 'puppet/file_serving/mount/tasks'
+require_relative '../../puppet'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/mount'
+require_relative '../../puppet/file_serving/mount/file'
+require_relative '../../puppet/file_serving/mount/modules'
+require_relative '../../puppet/file_serving/mount/plugins'
+require_relative '../../puppet/file_serving/mount/locales'
+require_relative '../../puppet/file_serving/mount/pluginfacts'
+require_relative '../../puppet/file_serving/mount/tasks'
 
 class Puppet::FileServing::Configuration
-  require 'puppet/file_serving/configuration/parser'
+  require_relative 'configuration/parser'
 
   def self.configuration
     @configuration ||= new

data/lib/puppet/file_serving/configuration/parser.rb

@@ -1,5 +1,5 @@
-require 'puppet/file_serving/configuration'
-require 'puppet/util/watched_file'
+require_relative '../../../puppet/file_serving/configuration'
+require_relative '../../../puppet/util/watched_file'
 
 class Puppet::FileServing::Configuration::Parser
   Mount = Puppet::FileServing::Mount

data/lib/puppet/file_serving/content.rb

@@ -1,6 +1,6 @@
-require 'puppet/indirector'
-require 'puppet/file_serving'
-require 'puppet/file_serving/base'
+require_relative '../../puppet/indirector'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/base'
 
 # A class that handles retrieving file contents.
 # It only reads the file when its content is specifically

data/lib/puppet/file_serving/fileset.rb

@@ -1,11 +1,11 @@
 require 'find'
-require 'puppet/file_serving'
-require 'puppet/file_serving/metadata'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/metadata'
 
 # Operate recursively on a path, returning a set of file paths.
 class Puppet::FileServing::Fileset
   attr_reader :path, :ignore, :links
-  attr_accessor :recurse, :recurselimit, :checksum_type
+  attr_accessor :recurse, :recurselimit, :max_files, :checksum_type
 
   # Produce a hash of files, with merged so that earlier files
   # with the same postfix win.  E.g., /dir1/subfile beats /dir2/subfile.
@@ -40,6 +40,7 @@ class Puppet::FileServing::Fileset
     self.links = :manage
     @recurse = false
     @recurselimit = :infinite
+    @max_files = 0
 
     if options.is_a?(Puppet::Indirector::Request)
       initialize_from_request(options)
@@ -58,6 +59,17 @@ class Puppet::FileServing::Fileset
   # level deep, which Find doesn't do.
   def files
     files = perform_recursion
+    soft_max_files = 1000
+
+    # munged_max_files is needed since puppet http handler is keeping negative numbers as strings
+    # https://github.com/puppetlabs/puppet/blob/main/lib/puppet/network/http/handler.rb#L196-L197
+    munged_max_files = max_files == '-1' ? -1 : max_files
+
+    if munged_max_files > 0 && files.size > munged_max_files
+      raise Puppet::Error.new _("The directory '%{path}' contains %{entries} entries, which exceeds the limit of %{munged_max_files} specified by the max_files parameter for this resource. The limit may be increased, but be aware that large number of file resources can result in excessive resource consumption and degraded performance. Consider using an alternate method to manage large directory trees") % { path: path, entries: files.size, munged_max_files: munged_max_files }
+    elsif munged_max_files == 0 && files.size > soft_max_files
+      Puppet.warning _("The directory '%{path}' contains %{entries} entries, which exceeds the default soft limit %{soft_max_files} and may cause excessive resource consumption and degraded performance. To remove this warning set a value for `max_files` parameter or consider using an alternate method to manage large directory trees") % { path: path, entries: files.size, soft_max_files: soft_max_files }
+    end
 
     # Now strip off the leading path, so each file becomes relative, and remove
     # any slashes that might end up at the beginning of the path.
@@ -96,7 +108,7 @@ class Puppet::FileServing::Fileset
   end
 
   def initialize_from_request(request)
-    [:links, :ignore, :recurse, :recurselimit, :checksum_type].each do |param|
+    [:links, :ignore, :recurse, :recurselimit, :max_files, :checksum_type].each do |param|
       if request.options.include?(param) # use 'include?' so the values can be false
         value = request.options[param]
       elsif request.options.include?(param.to_s)

data/lib/puppet/file_serving/http_metadata.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/metadata'
+require_relative '../../puppet/file_serving/metadata'
 
 # Simplified metadata representation, suitable for the information
 # that is available from HTTP headers.

data/lib/puppet/file_serving/metadata.rb

@@ -1,8 +1,8 @@
-require 'puppet'
-require 'puppet/indirector'
-require 'puppet/file_serving'
-require 'puppet/file_serving/base'
-require 'puppet/util/checksums'
+require_relative '../../puppet'
+require_relative '../../puppet/indirector'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/base'
+require_relative '../../puppet/util/checksums'
 require 'uri'
 
 # A class that handles retrieving file metadata.
@@ -66,7 +66,7 @@ class Puppet::FileServing::Metadata < Puppet::FileServing::Base
 
   class WindowsStat < MetaStat
     if Puppet::Util::Platform.windows?
-      require 'puppet/util/windows/security'
+      require_relative '../../puppet/util/windows/security'
     end
 
     def initialize(stat, path, source_permissions)

data/lib/puppet/file_serving/mount.rb

@@ -1,7 +1,7 @@
-require 'puppet/util/logging'
-require 'puppet/file_serving'
-require 'puppet/file_serving/metadata'
-require 'puppet/file_serving/content'
+require_relative '../../puppet/util/logging'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/metadata'
+require_relative '../../puppet/file_serving/content'
 
 # Broker access to the filesystem, converting local URIs into metadata
 # or content objects.

data/lib/puppet/file_serving/mount/file.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 class Puppet::FileServing::Mount::File < Puppet::FileServing::Mount
   def self.localmap

data/lib/puppet/file_serving/mount/locales.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 # Find files in the modules' locales directories.
 # This is a very strange mount because it merges

data/lib/puppet/file_serving/mount/modules.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 # This is the modules-specific mount: it knows how to search through
 # modules for files.  Yay.

data/lib/puppet/file_serving/mount/pluginfacts.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 # Find files in the modules' pluginfacts directories.
 # This is a very strange mount because it merges

data/lib/puppet/file_serving/mount/plugins.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 # Find files in the modules' plugins directories.
 # This is a very strange mount because it merges

data/lib/puppet/file_serving/mount/tasks.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving/mount'
+require_relative '../../../puppet/file_serving/mount'
 
 class Puppet::FileServing::Mount::Tasks < Puppet::FileServing::Mount
   def find(path, request)

data/lib/puppet/file_serving/terminus_helper.rb

@@ -1,5 +1,5 @@
-require 'puppet/file_serving'
-require 'puppet/file_serving/fileset'
+require_relative '../../puppet/file_serving'
+require_relative '../../puppet/file_serving/fileset'
 
 # Define some common methods for FileServing termini.
 

data/lib/puppet/file_serving/terminus_selector.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_serving'
+require_relative '../../puppet/file_serving'
 
 # This module is used to pick the appropriate terminus
 # in file-serving indirections.  This is necessary because

data/lib/puppet/file_system.rb

@@ -1,20 +1,20 @@
 module Puppet::FileSystem
-  require 'puppet/util'
-  require 'puppet/file_system/path_pattern'
-  require 'puppet/file_system/file_impl'
-  require 'puppet/file_system/memory_file'
-  require 'puppet/file_system/memory_impl'
-  require 'puppet/file_system/uniquefile'
+  require_relative '../puppet/util'
+  require_relative 'file_system/path_pattern'
+  require_relative 'file_system/file_impl'
+  require_relative 'file_system/memory_file'
+  require_relative 'file_system/memory_impl'
+  require_relative 'file_system/uniquefile'
 
   # create instance of the file system implementation to use for the current platform
   @impl = if Puppet::Util::Platform.windows?
-           require 'puppet/file_system/windows'
+           require_relative 'file_system/windows'
            Puppet::FileSystem::Windows
           elsif Puppet::Util::Platform.jruby?
-            require 'puppet/file_system/jruby'
+            require_relative 'file_system/jruby'
             Puppet::FileSystem::JRuby
           else
-            require 'puppet/file_system/posix'
+            require_relative 'file_system/posix'
             Puppet::FileSystem::Posix
           end.new()
 

data/lib/puppet/file_system/jruby.rb

@@ -1,4 +1,4 @@
-require 'puppet/file_system/posix'
+require_relative '../../puppet/file_system/posix'
 
 class Puppet::FileSystem::JRuby < Puppet::FileSystem::Posix
   def unlink(*paths)