hammer_cli_foreman_virt_who_configure 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ee53d10c25c7bb73cf7ec8bb3d7f8fc3dcbea957cc4ccde0652ce37c5aa9660
-  data.tar.gz: 7bde63d3680ca55eedf00df46f800b08d0858c3934b995585ee515cb5e408d66
+  metadata.gz: 160c6a49a4ece0bace60fb2a2476a912190d2311e9e51adeb12378d0301c3269
+  data.tar.gz: 58c205f21be5be1637b0fd0aa6ee7491598eaaa0aef733d46f8da5c56eeacb42
 SHA512:
-  metadata.gz: 7794300b03933db3ffa7438863ea2e98c10b9a13608109f567deef76911afe9741e6e755df11d8b6068c7773c3c72a714764e82e8be1d4f41217071dcee5c67c
-  data.tar.gz: a394bf2a667af20930679c72875164f684cdc2b8116d972de7e73153d58cf0bfb36ff0d000a4ac873d809e6502939fbec02e79f5828d94d17e568bc9c7c7ea10
+  metadata.gz: b57c0415e725c1e583379195930b6421e9182619b0ffa2db7a85b4e4ebaa9a47167ad5e8f4c6d00eb48105cad80c01723e5fe983650f86d615a9ff4317703bd3
+  data.tar.gz: c1875d96d19cf876d87a9168abdd08ec09f6b108e2f8c97dea0d2306e999ed513414538d20dc9430175d22e075de7eb7818b0c3791cd9fcd52320c6c1858b93b

data/lib/hammer_cli_foreman_virt_who_configure/system_caller.rb

@@ -1,15 +1,27 @@
 module HammerCLIForemanVirtWhoConfigure
   class SystemCaller
+    WHITELISTED_VARS = %w[HOME USER LANG].freeze
+
     def initialize(tempfile = nil)
       @tempfile = tempfile || Tempfile.new('virt_who')
     end
 
-    def system(commad)
+    def clean_env_vars
+      # Cleaning ENV vars and keeping required vars only because,
+      # When using SCL it adds GEM_HOME and GEM_PATH ENV vars.
+      # These vars break foreman-maintain tool.
+      # This way we use system ruby to execute the bash script.
+      cleaned_env = ENV.select { |var| WHITELISTED_VARS.include?(var) || var.start_with?('LC_') }
+      cleaned_env['PATH'] = '/sbin:/bin:/usr/sbin:/usr/bin'
+      cleaned_env
+    end
+
+    def system(command)
       result = nil
       begin
-        @tempfile.write(commad)
+        @tempfile.write(command)
         @tempfile.flush # to make sure the command is complete
-        result = Kernel.system("/usr/bin/bash #{@tempfile.path}")
+        result = Kernel.system(clean_env_vars, "/usr/bin/bash #{@tempfile.path}", unsetenv_others: true)
       ensure
         @tempfile.close
         @tempfile.unlink

data/lib/hammer_cli_foreman_virt_who_configure/version.rb

@@ -1,5 +1,5 @@
 module HammerCLIForemanVirtWhoConfigure
   def self.version
-    @version ||= Gem::Version.new '0.0.5'
+    @version ||= Gem::Version.new '0.0.6'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman_virt_who_configure
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Tomáš Strachota
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-27 00:00:00.000000000 Z
+date: 2020-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli