openfact 6.1.0 → 6.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4daa243a80d8ba47b89553843beda2cb577af1592688fc50824b28b4453c4e72
-  data.tar.gz: 1e1cbf1a2817cf2eb906ed791fd58bc60d0b1acaa229ace93e8a55d2f353c758
+  metadata.gz: 3bad8912857bdd63fa3c31cacf9e6e5d6113e85216f1a57662c0b0101c20f2f9
+  data.tar.gz: 16f3de88eecfd77e0f67d7d85c48d07b228fbad61d020a7998841d184ee327a5
 SHA512:
-  metadata.gz: 1791dc19b98a3676c9adcbc03df513ba5907d123159de09e6a67317ecdee1003eb5c1ea057b9099d81972c10f39e55a9f73f02f5a498918cc7fd227a710990b6
-  data.tar.gz: ba01bd6bf64dfe485897a0be96bcb8f7686b8500298f3205b28c7d8f20905b3d4db9530e11221be1e5bc55e92370eea7d82f0dcc80d82eaf58d19b4f6b74a861
+  metadata.gz: 0d7c6eda30361358651330ccd7a1a2dc6ffab30530e6a67806f5e65249128870bd2faf75553176c829e26fb9850e805b6f581733efe5714b06873ae5c5c60e56
+  data.tar.gz: 1fc5de3dc1cfafc7039fae02c708fc6baa8d36d2cb56bdabdf85fe2be43d0eac82bc24b7643e8f9921e28addea138778453ea2176bc38314ddafdc26c8e93382

data/lib/facter/resolvers/hostname.rb

@@ -9,6 +9,11 @@ module Facter
 
       init_resolver
 
+      # Upper bound, in seconds, for the fqdn lookup. A hung name service
+      # otherwise stalls fact collection until the agent's runtimeout fires
+      # (OpenVoxProject/openvox#485). Only applied on Rubies that honour it.
+      FQDN_LOOKUP_TIMEOUT = 10
+
       class << self
         private
 
@@ -48,14 +53,27 @@ module Facter
 
         def retrieve_with_addrinfo(host)
           begin
-            name = Socket.getaddrinfo(host, 0, Socket::AF_UNSPEC, Socket::SOCK_STREAM, nil, Socket::AI_CANONNAME)[0]
+            addr = Addrinfo.getaddrinfo(host, 0, Socket::AF_UNSPEC, Socket::SOCK_STREAM, nil, Socket::AI_CANONNAME,
+                                        **addrinfo_options)[0]
           rescue StandardError => e
-            @log.debug("Socket.getaddrinfo failed to retrieve fqdn for hostname #{host} with: #{e}")
+            @log.debug("Addrinfo.getaddrinfo failed to retrieve fqdn for hostname #{host} with: #{e}")
             return
           end
-          return if name.nil? || name.empty? || host == name[2] || name[2] == name[3]
 
-          name[2]
+          name = addr&.canonname
+          return if name.nil? || name.empty? || host == name || name == addr.ip_address
+
+          name
+        end
+
+        # Addrinfo.getaddrinfo only honours the timeout keyword from Ruby 4.0
+        # on. Older MRI accepts and ignores it; JRuby does not accept it.
+        def addrinfo_options
+          addrinfo_timeout_supported? ? { timeout: FQDN_LOOKUP_TIMEOUT } : {}
+        end
+
+        def addrinfo_timeout_supported?
+          RUBY_ENGINE == 'ruby' && Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('4.0')
         end
 
         def exists_and_valid_fqdn?(fqdn, hostname)

data/lib/facter/resolvers/linux/hostname.rb

@@ -10,6 +10,11 @@ module Facter
 
         init_resolver
 
+        # Upper bound, in seconds, for the fqdn lookup. A hung name service
+        # otherwise stalls fact collection until the agent's runtimeout fires
+        # (OpenVoxProject/openvox#485). Only applied on Rubies that honour it.
+        FQDN_LOOKUP_TIMEOUT = 10
+
         class << self
           private
 
@@ -65,16 +70,35 @@ module Facter
 
           def retrieve_fqdn_for_host(host)
             begin
-              name = Socket.getaddrinfo(host, 0, Socket::AF_UNSPEC, Socket::SOCK_STREAM, nil, Socket::AI_CANONNAME)[0]
+              addr = Addrinfo.getaddrinfo(host, 0, Socket::AF_UNSPEC, Socket::SOCK_STREAM, nil, Socket::AI_CANONNAME,
+                                          **addrinfo_options)[0]
             rescue StandardError => e
-              log.debug("Socket.getaddrinfo failed to retrieve fqdn for hostname #{host} with: #{e}")
+              log.debug("Addrinfo.getaddrinfo failed to retrieve fqdn for hostname #{host} with: #{e}")
+              # The name service is not answering. The FFI lookup below has no
+              # timeout and holds the GVL while it waits, so do not try it.
+              return if lookup_timed_out?(e)
             end
 
-            return name[2] if !name.nil? && !name.empty? && host != name[2] && name[2] != name[3]
+            name = addr&.canonname
+            return name if exists_and_not_empty?(name) && host != name && name != addr.ip_address
 
             retrieve_fqdn_for_host_with_ffi(host)
           end
 
+          def lookup_timed_out?(error)
+            error.is_a?(Errno::ETIMEDOUT) || (defined?(IO::TimeoutError) && error.is_a?(IO::TimeoutError))
+          end
+
+          # Addrinfo.getaddrinfo only honours the timeout keyword from Ruby 4.0
+          # on. Older MRI accepts and ignores it; JRuby does not accept it.
+          def addrinfo_options
+            addrinfo_timeout_supported? ? { timeout: FQDN_LOOKUP_TIMEOUT } : {}
+          end
+
+          def addrinfo_timeout_supported?
+            RUBY_ENGINE == 'ruby' && Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('4.0')
+          end
+
           def retrieve_fqdn_for_host_with_ffi(host)
             require_relative '../../../facter/util/resolvers/ffi/hostname'
 

data/lib/facter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Facter
-  VERSION = '6.1.0' unless defined?(VERSION)
+  VERSION = '6.2.0' unless defined?(VERSION)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openfact
 version: !ruby/object:Gem::Version
-  version: 6.1.0
+  version: 6.2.0
 platform: ruby
 authors:
 - OpenVox Project
@@ -87,14 +87,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.90.0
+        version: 1.91.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.90.0
+        version: 1.91.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -1287,7 +1287,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.16
+rubygems_version: 4.0.20
 specification_version: 4
 summary: OpenFact, a system inventory tool
 test_files: []