puppet 7.25.0-universal-darwin → 8.0.1-universal-darwin

data/lib/puppet/util/execution.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'timeout'
 require_relative '../../puppet/file_system/uniquefile'
 
@@ -161,7 +162,7 @@ module Puppet::Util::Execution
     # do this after processing 'command' array or string
     command_str = '[redacted]' if options[:sensitive]
 
-    user_log_s = ''
+    user_log_s = String.new
     if options[:uid]
       user_log_s << " uid=#{options[:uid]}"
     end
@@ -201,7 +202,7 @@ module Puppet::Util::Execution
       stderr = options[:combine] ? stdout : Puppet::FileSystem.open(null_file, nil, 'w')
 
       exec_args = [command, options, stdin, stdout, stderr]
-      output = ''
+      output = String.new
 
       # We close stdin/stdout/stderr immediately after fork/exec as they're no longer needed by
       # this process. In most cases they could be closed later, but when `stdout` is the "writer"
@@ -379,7 +380,7 @@ module Puppet::Util::Execution
     end.join(" ") if command.is_a?(Array)
 
     options[:custom_environment] ||= {}
-    Puppet::Util.withenv(options[:custom_environment], :windows) do
+    Puppet::Util.withenv(options[:custom_environment]) do
       Puppet::Util::Windows::Process.execute(command, options, stdin, stdout, stderr)
     end
   end

data/lib/puppet/util/execution_stub.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util
   class ExecutionStub
     class << self

data/lib/puppet/util/feature.rb

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

data/lib/puppet/util/file_watcher.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Util::FileWatcher
   include Enumerable
 

data/lib/puppet/util/fileparsing.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # A mini-language for parsing files.  This is only used file the ParsedFile
 # provider, but it makes more sense to split it out so it's easy to maintain
 # in one place.

data/lib/puppet/util/filetype.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Basic classes for reading, writing, and emptying files.  Not much
 # to see here.
 
@@ -215,7 +216,7 @@ class Puppet::Util::FileType
     # Remove a specific @path's cron tab.
     def remove
       cmd = "#{cmdbase} -r"
-      if %w{Darwin FreeBSD DragonFly}.include?(Puppet.runtime[:facter].value("operatingsystem"))
+      if %w{Darwin FreeBSD DragonFly}.include?(Puppet.runtime[:facter].value('os.name'))
         cmd = "/bin/echo yes | #{cmd}"
       end
 
@@ -244,7 +245,7 @@ class Puppet::Util::FileType
     # Only add the -u flag when the @path is different.  Fedora apparently
     # does not think I should be allowed to set the @path to my own user name
     def cmdbase
-      if @uid == Puppet::Util::SUIDManager.uid || Puppet.runtime[:facter].value(:operatingsystem) == "HP-UX"
+      if @uid == Puppet::Util::SUIDManager.uid || Puppet.runtime[:facter].value('os.name') == "HP-UX"
         return "crontab"
       else
         return "crontab -u #{@path}"

data/lib/puppet/util/http_proxy.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/http'
 
 # for backwards compatibility

data/lib/puppet/util/inifile.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Module Puppet::IniConfig
 # A generic way to parse .ini style files and manipulate them in memory
 # One 'file' can be made up of several physical files. Changes to sections
@@ -78,7 +79,7 @@ module Puppet::Util::IniConfig
     # written to file
     def format
       if @destroy
-        text = ""
+        text = String.new
       else
         text = "[#{name}]\n"
         @entries.each do |entry|
@@ -207,7 +208,7 @@ module Puppet::Util::IniConfig
     end
 
     def format
-      text = ""
+      text = String.new
 
       @contents.each do |content|
         if content.is_a? Section

data/lib/puppet/util/instance_loader.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/autoload'
 require_relative '../../puppet/util'
 require_relative '../../puppet/concurrent/lock'

data/lib/puppet/util/json.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util
   module Json
     class ParseError < StandardError

data/lib/puppet/util/json_lockfile.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/lockfile'
 
 # This class provides a simple API for managing a lock file

data/lib/puppet/util/ldap/connection.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/ldap'
 
 class Puppet::Util::Ldap::Connection

data/lib/puppet/util/ldap/generator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/ldap'
 
 class Puppet::Util::Ldap::Generator

data/lib/puppet/util/ldap/manager.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/ldap'
 require_relative '../../../puppet/util/ldap/connection'
 require_relative '../../../puppet/util/ldap/generator'

data/lib/puppet/util/ldap.rb

@@ -1,2 +1,3 @@
+# frozen_string_literal: true
 module Puppet::Util::Ldap
 end

data/lib/puppet/util/libuser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::Libuser
    def self.getconf
      File.expand_path("../libuser.conf", __FILE__)

data/lib/puppet/util/limits.rb

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

data/lib/puppet/util/lockfile.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This class provides a simple API for managing a lock file
 # whose contents are an (optional) String.  In addition
 # to querying the basic state (#locked?) of the lock, managing

data/lib/puppet/util/log/destination.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # A type of log destination.
 class Puppet::Util::Log::Destination
   class << self

data/lib/puppet/util/log/destinations.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 Puppet::Util::Log.newdesttype :syslog do
   def self.suitable?(obj)
     Puppet.features.syslog?

data/lib/puppet/util/log.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/tagging'
 require_relative '../../puppet/util/classgen'
 require_relative '../../puppet/util/psych_support'

data/lib/puppet/util/logging.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # A module to make logging a bit easier.
 require_relative '../../puppet/util/log'
 require_relative '../../puppet/error'

data/lib/puppet/util/metaid.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Object
   # The hidden singleton lurks behind everyone
   def singleton_class; class << self; self; end; end

data/lib/puppet/util/metric.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # included so we can test object types
 require_relative '../../puppet'
 require_relative '../../puppet/network/format_support'

data/lib/puppet/util/monkey_patches.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/platform'
 
 module Puppet::Util::MonkeyPatches
@@ -29,39 +30,13 @@ class Object
   end
 end
 
-if RUBY_VERSION.to_f < 3.0
-  # absolute/relative were optimized to avoid chop_basename in ruby 3
-  # see https://github.com/ruby/ruby/commit/39312cf4d6c2ab3f07d688ad1a467c8f84b58db0
-  require 'pathname'
-  class Pathname
-    if File.dirname('A:') == 'A:.' # DOSish drive letter
-      ABSOLUTE_PATH = /\A(?:[A-Za-z]:|#{SEPARATOR_PAT})/o
-    else
-      ABSOLUTE_PATH = /\A#{SEPARATOR_PAT}/o
-    end
-    private_constant :ABSOLUTE_PATH
-
-    def absolute?
-      ABSOLUTE_PATH.match? @path
-    end
-
-    def relative?
-      !absolute?
-    end
-  end
-end
-
-# (#19151) Reject all SSLv2 ciphers and handshakes
 require_relative '../../puppet/ssl/openssl_loader'
 unless Puppet::Util::Platform.jruby_fips?
   class OpenSSL::SSL::SSLContext
     if DEFAULT_PARAMS[:options]
-      DEFAULT_PARAMS[:options] |= OpenSSL::SSL::OP_NO_SSLv2 | OpenSSL::SSL::OP_NO_SSLv3
+      DEFAULT_PARAMS[:options] |= OpenSSL::SSL::OP_NO_SSLv3
     else
-      DEFAULT_PARAMS[:options] = OpenSSL::SSL::OP_NO_SSLv2 | OpenSSL::SSL::OP_NO_SSLv3
-    end
-    if DEFAULT_PARAMS[:ciphers]
-      DEFAULT_PARAMS[:ciphers] << ':!SSLv2'
+      DEFAULT_PARAMS[:options] = OpenSSL::SSL::OP_NO_SSLv3
     end
 
     alias __original_initialize initialize

data/lib/puppet/util/multi_match.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # MultiMatch allows multiple values to be tested at once in a case expression.
 # This class is needed since Array does not implement the === operator to mean
 # "each v === other.each v". 

data/lib/puppet/util/network_device/base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/autoload'
 require 'uri'
 require_relative '../../../puppet/util/network_device/transport'

data/lib/puppet/util/network_device/config.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'ostruct'
 require_relative '../../../puppet/util/watched_file'
 require_relative '../../../puppet/util/network_device'

data/lib/puppet/util/network_device/transport/base.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 
 require_relative '../../../../puppet/util/network_device'
 require_relative '../../../../puppet/util/network_device/transport'

data/lib/puppet/util/network_device/transport.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/network_device'
 
 # stub

data/lib/puppet/util/network_device.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Util::NetworkDevice
   class << self
     attr_reader :current

data/lib/puppet/util/package/version/debian.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::Package::Version
   class Debian < Numeric
     include Comparable

data/lib/puppet/util/package/version/gem.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::Package::Version
   class Gem < ::Gem::Version
     def self.parse(version)

data/lib/puppet/util/package/version/pip.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::Package::Version
   class Pip
     include Comparable
@@ -31,7 +32,7 @@ module Puppet::Util::Package::Version
         )?
       )
       (?:\\+(?<local>[a-z0-9]+(?:[-_\\.][a-z0-9]+)*))?      # local version
-    ".freeze
+    "
 
     def self.parse(version)
       raise ValidationFailure, version.to_s unless version.is_a? String

data/lib/puppet/util/package/version/range/eq.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range/simple'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/gt.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range/simple'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/gt_eq.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range/simple'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/lt.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range/simple'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/lt_eq.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range/simple'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/min_max.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range/simple.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../../puppet/util/package/version/range'
 
 module Puppet::Util::Package::Version

data/lib/puppet/util/package/version/range.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative 'range/lt'
 require_relative 'range/lt_eq'
 require_relative 'range/gt'

data/lib/puppet/util/package/version/rpm.rb

@@ -19,7 +19,7 @@ module Puppet::Util::Package::Version
     end
 
     def to_s
-      version_found = ''
+      version_found = String.new
       version_found += "#{@epoch}:"   if @epoch
       version_found += @version
       version_found += "-#{@release}" if @release

data/lib/puppet/util/pidlock.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'fileutils'
 require_relative '../../puppet/util/lockfile'
 

data/lib/puppet/util/platform.rb

@@ -1,8 +1,9 @@
+# frozen_string_literal: true
 module Puppet
   module Util
     module Platform
-      FIPS_STATUS_FILE = "/proc/sys/crypto/fips_enabled".freeze
-      WINDOWS_FIPS_REGISTRY_KEY = 'System\\CurrentControlSet\\Control\\Lsa\\FipsAlgorithmPolicy'.freeze
+      FIPS_STATUS_FILE = "/proc/sys/crypto/fips_enabled"
+      WINDOWS_FIPS_REGISTRY_KEY = 'System\\CurrentControlSet\\Control\\Lsa\\FipsAlgorithmPolicy'
 
       def windows?
         # Ruby only sets File::ALT_SEPARATOR on Windows and the Ruby standard

data/lib/puppet/util/plist.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'cfpropertylist' if Puppet.features.cfpropertylist?
 require_relative '../../puppet/util/execution'
 

data/lib/puppet/util/posix.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Utility methods for interacting with POSIX objects; mostly user and group
 module Puppet::Util::POSIX
 

data/lib/puppet/util/profiler/aggregate.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/profiler'
 require_relative '../../../puppet/util/profiler/wall_clock'
 

data/lib/puppet/util/profiler/around_profiler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # A Profiler that can be used to wrap around blocks of code. It is configured
 # with other profilers and controls them to start before the block is executed
 # and finish after the block is executed.

data/lib/puppet/util/profiler/logging.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Puppet::Util::Profiler::Logging
   def initialize(logger, identifier)
     @logger = logger

data/lib/puppet/util/profiler/object_counts.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/profiler/logging'
 
 class Puppet::Util::Profiler::ObjectCounts < Puppet::Util::Profiler::Logging

data/lib/puppet/util/profiler/wall_clock.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../puppet/util/profiler/logging'
 
 # A profiler implementation that measures the number of seconds a segment of

data/lib/puppet/util/profiler.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'benchmark'
 
 # A simple profiling callback system.

data/lib/puppet/util/provider_features.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Provides feature definitions.
 require_relative '../../puppet/util/docs'
 require_relative '../../puppet/util'
@@ -75,7 +76,7 @@ module Puppet::Util::ProviderFeatures
 
   # @return [String] Returns a string with documentation covering all features.
   def featuredocs
-    str = ""
+    str = String.new
     @features ||= {}
     return nil if @features.empty?
     names = @features.keys.sort_by(&:to_s)

data/lib/puppet/util/psych_support.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # This module should be included when a class can be serialized to yaml and
 # needs to handle the deserialization from Psych with more control. Psych normally
 # pokes values directly into an instance using `instance_variable_set` which completely

data/lib/puppet/util/rdoc/code_objects.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'rdoc/code_objects'
 
 module RDoc

data/lib/puppet/util/rdoc/generators/puppet_generator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'rdoc/generators/html_generator'
 require_relative '../../../../puppet/util/rdoc/code_objects'
 require 'digest/md5'

data/lib/puppet/util/rdoc/generators/template/puppet/puppet.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # = CSS2 RDoc HTML template
 #

data/lib/puppet/util/rdoc/parser/puppet_parser_core.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Functionality common to both our RDoc version 1 and 2 parsers.
 module RDoc::PuppetParserCore
 

data/lib/puppet/util/rdoc/parser/puppet_parser_rdoc2.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../../../puppet/util/rdoc/parser/puppet_parser_core.rb'
 
 module RDoc

data/lib/puppet/util/rdoc/parser.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Puppet "parser" for the rdoc system
 # The parser uses puppet parser and traverse the AST to instruct RDoc about
 # our current structures. It also parses ruby files that could contain

data/lib/puppet/util/rdoc.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util'
 module Puppet::Util::RDoc
   module_function

data/lib/puppet/util/reference.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/instance_loader'
 require 'fileutils'
 

data/lib/puppet/util/resource_template.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util'
 require_relative '../../puppet/util/logging'
 require 'erb'

data/lib/puppet/util/retry_action.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::RetryAction
   class RetryException < Exception; end
   class RetryException::NoBlockGiven < RetryException; end

data/lib/puppet/util/rpm_compare.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::RpmCompare
 
   ARCH_LIST = %w(

data/lib/puppet/util/rubygems.rb

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

data/lib/puppet/util/run_mode.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'etc'
 
 module Puppet

data/lib/puppet/util/selinux.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Provides utility functions to help interface Puppet to SELinux.
 #
 # This requires the very new SELinux Ruby bindings.  These bindings closely
@@ -226,7 +227,7 @@ module Puppet::Util::SELinux
 
   # Internal helper function to read and parse /proc/mounts
   def read_mounts
-    mounts = ""
+    mounts = String.new
     begin
       if File.method_defined? "read_nonblock"
         # If possible we use read_nonblock in a loop rather than read to work-

data/lib/puppet/util/skip_tags.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/tagging'
 
 class Puppet::Util::SkipTags

data/lib/puppet/util/splayer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Handle splay options (sleeping for a random interval before executing)
 module Puppet::Util::Splayer
   # Have we splayed already?

data/lib/puppet/util/storage.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'yaml'
 require 'singleton'
 require_relative '../../puppet/util/yaml'

data/lib/puppet/util/suidmanager.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require_relative '../../puppet/util/warnings'
 require 'forwardable'
 require 'etc'
@@ -17,7 +18,7 @@ module Puppet::Util::SUIDManager
 
   def osx_maj_ver
     return @osx_maj_ver unless @osx_maj_ver.nil?
-    @osx_maj_ver = Puppet.runtime[:facter].value('macosx_productversion_major') || false
+    @osx_maj_ver = Puppet.runtime[:facter].value('os.macosx.version.major') || false
   end
   module_function :osx_maj_ver
 

data/lib/puppet/util/symbolic_file_mode.rb

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

data/lib/puppet/util/tag_set.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'set'
 require_relative '../../puppet/network/format_support'
 

data/lib/puppet/util/terminal.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Puppet::Util::Terminal
   # Attempts to determine the width of the terminal.  This is currently only
   # supported on POSIX systems, and relies on the claims of `stty` (or `tput`).

data/lib/puppet/util/user_attr.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class UserAttr
   def self.get_attributes_by_name(name)
     attributes = nil

data/lib/puppet/util/warnings.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # Methods to help with handling warnings.
 module Puppet::Util::Warnings
   module_function