vagrant-goodhosts 1.0.16 → 1.1.0beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6843d62e56a2bd4b265a0b4854bcb69fba17c99b60515ad4bdb23ef1decf384b
-  data.tar.gz: dfb62145d87e3d1d3122a6777399d3fb41c620df68f762ed6284c3221066e72b
+  metadata.gz: 0f6a0225da423c0c803dacaeb97182e8768510d601db33c182a088733d282f60
+  data.tar.gz: f2756ce326a5d766baa4fd63f5d4d158c13fb0591c88a9763dc6e9846b377bbc
 SHA512:
-  metadata.gz: 9476f2832c25de5c268f2bcbda6f243f784ed641143c8a5b26fd0511582e9cbd010ff99faa96f53084699c16970a9efb71964309914f4d4e4847dacae2392346
-  data.tar.gz: 583d8ccf7c2c5e77feed7980987facb0fe156e1325efd580d19c20518cea2141a4b80e6cb18d8f15a550f945099c9c37755c13dc703d44e72c60b14c30e75104
+  metadata.gz: 11d1c0937d56326ebbd390adc477c8c047b6422f0c81c21ad454d297992876941267102811acbc7e7e32e960a2feb2767d055f086a4461b8f6008cd1b81d0581
+  data.tar.gz: 528d8f7ff5b81d8a377530a628710ef62ae4432790099f3c8a4673c01e2c3d1ef4032181680e63bc5cd49f8925d01374ed3d8ae919a09bd2849ad0def44022d2

data/.github/workflows/default.yml

@@ -0,0 +1,15 @@
+name: Linters
+
+on: [push]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: RuboCop Linter
+        uses: andrewmcodes/rubocop-linter-action@v3.3.0
+        with:
+          exit_on_failure: true
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

data/.rubocop.yml

@@ -0,0 +1,68 @@
+AllCops:
+  Exclude:
+    - 'Gemfile'
+    - 'Rakefile'
+    - 'lib/vagrant-goodhosts/config.rb'
+
+Naming/FileName:
+    Enabled: false
+Naming/AccessorMethodName:
+    Enabled: false
+Metrics/MethodLength:
+    Enabled: false
+Metrics/PerceivedComplexity:
+    Enabled: false
+Metrics/ModuleLength:
+    Enabled: false
+Metrics/CyclomaticComplexity:
+    Enabled: false
+Metrics/AbcSize:
+    Enabled: false
+Style/FrozenStringLiteralComment:
+    Enabled: false
+Style/RaiseArgs:
+    Enabled: false
+Layout/LineLength:
+    Enabled: false
+Style/StringLiterals:
+    Enabled: false
+Style/ExpandPathArguments:
+    Enabled: false
+Style/RedundantReturn:
+    Enabled: false
+Style/SymbolArray:
+    Enabled: false
+Style/ClassVars:
+    Enabled: false
+Style/YodaCondition:
+    Enabled: false
+Style/MethodCallWithoutArgsParentheses:
+    Enabled: false
+Style/NumericPredicate:
+    Enabled: false
+Style/ParallelAssignment:
+    Enabled: false
+Style/Not:
+    Enabled: false
+Style/EmptyLiteral:
+    Enabled: false
+Style/IfUnlessModifier:
+    Enabled: false
+Style/MutableConstant:
+    Enabled: false
+Style/SymbolLiteral:
+    Enabled: false
+Style/AccessorGrouping:
+    Enabled: false
+Style/Next:
+    Enabled: false
+Style/MultipleComparison:
+    Enabled: false
+Style/FormatStringToken:
+    Enabled: false
+Style/SoleNestedConditional:
+    Enabled: false
+Style/GuardClause:
+    Enabled: false
+Gemspec/RequiredRubyVersion:
+    Enabled: false

data/Gemfile

@@ -7,3 +7,5 @@ end
 group :plugins do
   gem "vagrant-goodhosts", path: "."
 end
+
+gem 'os'

data/README.md

@@ -96,15 +96,15 @@ If you understand the risks that go with supressing them, here's how to do it.
 ### Linux/OS X: Passwordless sudo
 
 To allow vagrant to automatically update the hosts file without asking for a sudo password, add one of the following snippets to a new sudoers file include, i.e. `sudo visudo -f /etc/sudoers.d/vagrant_goodhosts`.
-The command path is printed when there are errors with sudo.
+The command path is printed when there are errors with Vagrant, check the output marked in red.
 
 For Ubuntu and most Linux environments:
 
-    %sudo ALL=(root) NOPASSWD: [the-path]
+    %sudo ALL=(root) NOPASSWD: [the-command-path]
 
 For MacOS:
 
-    %admin ALL=(root) NOPASSWD: [the-path]
+    %admin ALL=(root) NOPASSWD: [the-command-path]
 
 Replace in both %sudo/%admin with the username it if it is not working for you.
 

data/lib/vagrant-goodhosts/Action/BaseAction.rb

@@ -1,6 +1,8 @@
+# Action to extend for the plugin needs, detects if it was already executed etc
 module VagrantPlugins
   module GoodHosts
     module Action
+      # Extend it!
       class BaseAction
         include GoodHosts
 
@@ -35,10 +37,9 @@ module VagrantPlugins
           @app.call(env)
         end
 
-        def run(env)
+        def run(_env)
           raise NotImplementedError.new("Must be implemented!")
         end
-
       end
     end
   end

data/lib/vagrant-goodhosts/Action/RemoveHosts.rb

@@ -1,8 +1,9 @@
+# Run when is removing the hosts
 module VagrantPlugins
   module GoodHosts
     module Action
+      # Remove hosts
       class RemoveHosts < BaseAction
-
         def run(env)
           machine_action = env[:machine_action]
 
@@ -12,11 +13,9 @@ module VagrantPlugins
           if ([:halt, :suspend].include? machine_action) && (false == @machine.config.goodhosts.remove_on_suspend)
             @ui.info "[vagrant-goodhosts] Removing hosts on suspend disabled"
           else
-            @ui.info "[vagrant-goodhosts] Removing hosts"
-            removeHostEntries
+            remove_host_entries
           end
         end
-
       end
     end
   end

data/lib/vagrant-goodhosts/Action/UpdateHosts.rb

@@ -1,13 +1,12 @@
+# Run when is adding hosts
 module VagrantPlugins
   module GoodHosts
     module Action
+      # Update hosts
       class UpdateHosts < BaseAction
-
-        def run(env)
-          @ui.info "[vagrant-goodhosts] Checking for host entries"
-          addHostEntries()
+        def run(_env)
+          add_host_entries()
         end
-
       end
     end
   end

data/lib/vagrant-goodhosts/GoodHosts.rb

@@ -1,26 +1,30 @@
+# The core of the plugin
 require "rbconfig"
 require "open3"
+require "resolv"
+require "os"
 
 module VagrantPlugins
   module GoodHosts
+    # Plugin module
     module GoodHosts
-      def getIps
+      def get_ips
         ips = []
 
-        if @machine.config.vm.networks.length == 0
-            @ui.error("[vagrant-goodhosts] No ip address found for this virtual machine")
-            exit
+        if @machine.config.vm.networks.empty?
+          @ui.error("[vagrant-goodhosts] No networks are available yet for this virtual machine to add IP/hosts for")
+          return ips
         end
-        
+
         @machine.config.vm.networks.each do |network|
           key, options = network[0], network[1]
-          ip = options[:ip] if (key == :private_network || key == :public_network) && options[:goodhosts] != "skip"
-          ips.push(ip) if ip
           if options[:goodhosts] == "skip"
             @ui.info '[vagrant-goodhosts] Skipped adding host entries (config.vm.network goodhosts: "skip" is set)'
           end
+          ip = options[:ip] if (key == :private_network || key == :public_network) && options[:goodhosts] != "skip"
+          ips.push(ip) if ip
 
-          @machine.config.vm.provider :hyperv do |v|
+          @machine.config.vm.provider :hyperv do
             timeout = @machine.provider_config.ip_address_timeout
             @ui.output("[vagrant-goodhosts] Waiting for the guest machine to report its IP address ( this might take some time, have patience )...")
             @ui.detail("Timeout: #{timeout} seconds")
@@ -29,44 +33,39 @@ module VagrantPlugins
               vmm_server_address: @machine.provider_config.vmm_server_address,
               proxy_server_address: @machine.provider_config.proxy_server_address,
               timeout: timeout,
-              machine: @machine,
+              machine: @machine
             }
             network = @machine.provider.driver.read_guest_ip(options)
             if network["ip"]
               ips.push(network["ip"]) unless ips.include? network["ip"]
             end
           end
-
-
         end
         return ips
       end
 
-      # https://stackoverflow.com/a/13586108/1902215
       def get_os_binary
-        return os ||= (host_os = RbConfig::CONFIG["host_os"]
-                 case host_os
-               when /mswin|msys|mingw|cygwin|bccwin|wince|emc/
-                 :'cli.exe'
-               when /darwin|mac os/
-                 :'cli_osx'
-               when /linux/
-                 :'cli'
-               else
-                 raise Error::WebDriverError, "unknown os: #{host_os.inspect}"
-               end)
+        if OS.windows?
+          return 'cli.exe'
+        elsif OS.mac?
+          return 'cli_osx'
+        elsif OS.linux?
+          return 'cli'
+        else
+          raise Error::WebDriverError, "unknown os: #{host_os.inspect}"
+        end
       end
 
       def get_cli
         binary = get_os_binary
-        path = File.expand_path(File.dirname(File.dirname(__FILE__))) + "/vagrant-goodhosts/bundle/"
+        path = format('%s%s', File.expand_path(File.dirname(File.dirname(__FILE__))), "/vagrant-goodhosts/bundle/")
         path = "#{path}#{binary}"
 
         return path
       end
 
       # Get a hash of hostnames indexed by ip, e.g. { 'ip1': ['host1'], 'ip2': ['host2', 'host3'] }
-      def getHostnames(ips)
+      def get_hostnames(ips)
         hostnames = Hash.new { |h, k| h[k] = [] }
 
         case @machine.config.goodhosts.aliases
@@ -84,109 +83,144 @@ module VagrantPlugins
 
         return hostnames
       end
-      
-      def disableClean(ip_address)
+
+      def disable_clean(ip_address)
         unless ip_address.nil?
           return @machine.config.goodhosts.disable_clean
         end
+
         return true
       end
 
-      def addHostEntries
-        error = false
-        errorText = ""
+      def check_hostnames_to_add(ip_address, hostnames)
+        hostnames_to_add = Array.new
+        hostnames = hostnames.split
+        # check which hostnames actually need adding
+        hostnames.each do |hostname|
+          address = Resolv.getaddress(hostname)
+          if address != ip_address
+            hostnames_to_add.append(hostname)
+          end
+        rescue StandardError => _e
+          hostnames_to_add.append(hostname)
+        end
+        return hostnames_to_add
+      end
+
+      def add_goodhost_entries(ip_address, hostnames)
         cli = get_cli
-        hostnames_by_ips = generateHostnamesByIps
-        
-        return if not hostnames_by_ips.any?
+        if cli.include? ".exe"
+          clean = "\"--clean\","
+          if disable_clean(ip_address)
+            clean = ''
+          end
+          stdin, stdout, stderr, wait_thr = Open3.popen3("powershell", "-Command", "Start-Process '#{cli}' -ArgumentList \"add\",#{clean}\"#{ip_address}\",\"#{hostnames}\" -Verb RunAs")
+        else
+          clean = "--clean"
+          if disable_clean(ip_address)
+            clean = ''
+          end
+          stdin, stdout, stderr, wait_thr = Open3.popen3("sudo '#{cli}' add #{clean} #{ip_address} #{hostnames}")
+        end
+        return stdin, stdout, stderr, wait_thr
+      end
+
+      def add_host_entries
+        error = false
+        error_text = ""
+        hostnames_by_ips = generate_hostnames_by_ips
+
+        return if hostnames_by_ips.none?
+
+        @ui.info "[vagrant-goodhosts] Checking for host entries"
 
         hostnames_by_ips.each do |ip_address, hostnames|
           if ip_address.nil?
             @ui.error "[vagrant-goodhosts] Error adding some hosts, no IP was provided for the following hostnames: #{hostnames}"
             next
           end
-          if cli.include? ".exe"
-            clean = "\"--clean\","
-            if disableClean(ip_address)
-                clean = ''
-            end
-            stdin, stdout, stderr, wait_thr = Open3.popen3("powershell", "-Command", "Start-Process '#{cli}' -ArgumentList \"add\",#{clean}\"#{ip_address}\",\"#{hostnames}\" -Verb RunAs")
-          else
-            clean = "--clean"
-            if disableClean(ip_address)
-                clean = ''
-            end
-            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo '#{cli}' add #{clean} #{ip_address} #{hostnames}")
-          end
-          if !wait_thr.value.success?
+
+          # filter out the hosts we've already added
+          hosts_to_add = check_hostnames_to_add(ip_address, hostnames)
+          next if hosts_to_add.none?
+
+          _stdin, _stdout, stderr, wait_thr = add_goodhost_entries(ip_address, hosts_to_add)
+          unless wait_thr.value.success?
             error = true
-            errorText = stderr.read.strip
+            error_text = stderr.read.strip
           end
         end
-        printReadme(error, errorText)
+        print_readme(error, error_text)
       end
 
-      def removeHostEntries
-        error = false
-        errorText = ""
+      def remove_goodhost_entries(ip_address, hostnames)
         cli = get_cli
-        hostnames_by_ips = generateHostnamesByIps
+        clean = "\"--clean\","
+        if disable_clean(ip_address)
+          clean = ''
+        end
+        if cli.include? ".exe"
+          stdin, stdout, stderr, wait_thr = Open3.popen3("powershell", "-Command", "Start-Process '#{cli}' -ArgumentList \"remove\",#{clean}\"#{ip_address}\",\"#{hostnames}\" -Verb RunAs")
+        else
+          stdin, stdout, stderr, wait_thr = Open3.popen3("sudo '#{cli}' remove #{clean} #{ip_address} #{hostnames}")
+        end
+        return stdin, stdout, stderr, wait_thr
+      end
 
-        return if not hostnames_by_ips.any?
+      def remove_host_entries
+        error = false
+        error_text = ""
+        hostnames_by_ips = generate_hostnames_by_ips
+
+        return if hostnames_by_ips.none?
+
+        @ui.info "[vagrant-goodhosts] Removing hosts"
 
         hostnames_by_ips.each do |ip_address, hostnames|
           if ip_address.nil?
             @ui.error "[vagrant-goodhosts] Error adding some hosts, no IP was provided for the following hostnames: #{hostnames}"
             next
           end
-          if cli.include? ".exe"
-            clean = "\"--clean\","
-            if disableClean(ip_address)
-                clean = ''
-            end
-            stdin, stdout, stderr, wait_thr = Open3.popen3("powershell", "-Command", "Start-Process '#{cli}' -ArgumentList \"remove\",#{clean}\"#{ip_address}\",\"#{hostnames}\" -Verb RunAs")
-          else
-            clean = "\"--clean\","
-            if disableClean(ip_address)
-                clean = ''
-            end
-            stdin, stdout, stderr, wait_thr = Open3.popen3("sudo '#{cli}' remove #{clean} #{ip_address} #{hostnames}")
-          end
-          if !wait_thr.value.success?
+
+          _stdin, _stdout, stderr, wait_thr = remove_goodhost_entries(ip_address, hostnames)
+          unless wait_thr.value.success?
             error = true
-            errorText = stderr.read.strip
+            error_text = stderr.read.strip
           end
         end
-        printReadme(error, errorText)
+        print_readme(error, error_text)
       end
 
-      def printReadme(error, errorText)
-        if error
-          cli = get_cli
-          @ui.error "[vagrant-goodhosts] Issue executing goodhosts CLI: #{errorText}"
-          @ui.error "[vagrant-goodhosts] Cli path: #{cli}"
-          if cli.include? ".exe"
-            @ui.error "[vagrant-goodhosts] Check the readme at https://github.com/goodhosts/vagrant#windows-uac-prompt"
-            exit
-          else
-            @ui.error "[vagrant-goodhosts] Check the readme at https://github.com/goodhosts/vagrant#passwordless-sudo"
-          end
+      def print_readme(error, error_text)
+        unless error
+          return false
+        end
+
+        cli = get_cli
+        @ui.error "[vagrant-goodhosts] Issue executing goodhosts CLI: #{error_text}"
+        @ui.error "[vagrant-goodhosts] Cli path: #{cli}"
+        if cli.include? ".exe"
+          @ui.error "[vagrant-goodhosts] Check the readme at https://github.com/goodhosts/vagrant#windows-uac-prompt"
+          exit
+        else
+          @ui.error "[vagrant-goodhosts] Check the readme at https://github.com/goodhosts/vagrant#passwordless-sudo"
         end
       end
 
-      def generateHostnamesByIps()
+      def generate_hostnames_by_ips
         hostnames_by_ips = []
-        ips = getIps
+        ips = get_ips
         if ips.count() < 1
+          @ui.error("[vagrant-goodhosts] No ip address found for this virtual machine")
           return hostnames_by_ips
         end
-        hostnames = getHostnames(ips)
+        hostnames = get_hostnames(ips)
         if ips.count() > 1
           ips.each do |ip|
             ip_address = ip
             if hostnames[ip].count() > 0
               hostnames[ip].each do |hostname|
-                if !ip_address.nil?
+                unless ip_address.nil?
                   @ui.info "[vagrant-goodhosts] - found entry for: #{ip_address} #{hostname}"
                 end
               end
@@ -197,7 +231,7 @@ module VagrantPlugins
           ip_address = ips[0]
           if hostnames[ip_address].count() > 0
             hostnames[ip_address].each do |hostname|
-              if !ip_address.nil?
+              unless ip_address.nil?
                 @ui.info "[vagrant-goodhosts] - found entry for: #{ip_address} #{hostname}"
               end
             end

data/lib/vagrant-goodhosts/config.rb

@@ -2,16 +2,17 @@ require "vagrant"
 
 module VagrantPlugins
   module GoodHosts
+    # Vagrant plugin definition
     class Config < Vagrant.plugin("2", :config)
-        attr_accessor :aliases
-        attr_accessor :id
-        attr_accessor :remove_on_suspend
-        attr_accessor :disable_clean
-        
-        def initialize
-            @remove_on_suspend = true
-            @disable_clean = true
-        end
+      attr_accessor :aliases
+      attr_accessor :id
+      attr_accessor :remove_on_suspend
+      attr_accessor :disable_clean
+
+      def initialize
+        @remove_on_suspend = true
+        @disable_clean = true
+      end
     end
   end
 end

data/lib/vagrant-goodhosts/plugin.rb

@@ -5,6 +5,7 @@ require_relative "Action/RemoveHosts"
 
 module VagrantPlugins
   module GoodHosts
+    # Various Vagrant hooks
     class Plugin < Vagrant.plugin('2')
       name 'GoodHosts'
       description <<-DESC
@@ -21,6 +22,10 @@ module VagrantPlugins
         hook.append(Action::UpdateHosts)
       end
 
+      action_hook(:goodhosts, :machine_action_boot) do |hook|
+        hook.append(Action::UpdateHosts)
+      end
+
       action_hook(:goodhosts, :machine_action_provision) do |hook|
         hook.before(Vagrant::Action::Builtin::Provision, Action::UpdateHosts)
       end

data/lib/vagrant-goodhosts/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module GoodHosts
-    VERSION = '1.0.16'
+    VERSION = '1.1.0beta'
   end
 end

data/lib/vagrant-goodhosts.rb

@@ -1,11 +1,13 @@
+# Root file of the plugin
 require "vagrant-goodhosts/version"
 require "vagrant-goodhosts/plugin"
 
+# Extend Vagrant Plugins
 module VagrantPlugins
+  # Load our plugin
   module GoodHosts
     def self.source_root
       @source_root ||= Pathname.new(File.expand_path('../../', __FILE__))
     end
   end
 end
-

data/package.sh

@@ -13,3 +13,4 @@ rm -f ./*.txt
 # Generate
 cd ../../../
 gem build vagrant-goodhosts.gemspec
+# gem push

data/vagrant-goodhosts.gemspec

@@ -1,4 +1,3 @@
-# coding: utf-8
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'vagrant-goodhosts/version'
@@ -14,7 +13,7 @@ Gem::Specification.new do |s|
   s.homepage      = 'https://github.com/goodhosts/vagrant'
   s.license       = 'MIT'
 
-  s.files         = `git ls-files`.split($/)
+  s.files         = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
   s.files        += Dir.glob("lib/vagrant-goodhosts/bundle/*")
   s.executables   = s.files.grep(%r{^bin/}) { |f| File.basename(f) }
   s.test_files    = s.files.grep(%r{^(test|spec|features)/})
@@ -22,4 +21,6 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency 'bundler', '~> 1.3'
   s.add_development_dependency 'rake'
+
+  s.add_runtime_dependency "os"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-goodhosts
 version: !ruby/object:Gem::Version
-  version: 1.0.16
+  version: 1.1.0beta
 platform: ruby
 authors:
 - Daniele Scasciafratte
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-22 00:00:00.000000000 Z
+date: 2021-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: os
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Enables Vagrant to update hosts file on the host machine with goodhosts
 email:
 - mte90net@gmail.com
@@ -45,7 +59,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/default.yml"
 - ".gitignore"
+- ".rubocop.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -67,7 +83,7 @@ homepage: https://github.com/goodhosts/vagrant
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -78,12 +94,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubygems_version: 3.2.5
-signing_key: 
+rubygems_version: 3.2.27
+signing_key:
 specification_version: 4
 summary: Enables Vagrant to update hosts file on the host machine with goodhosts
 test_files: []