Use `has_key?` so as to not add the blank keys to the config unless needed.

Signed-off-by: Noah Kantrowitz <noah@coderanger.net>

1 files changed, 4 insertions, 4 deletions

diff --git a/chef-config/lib/chef-config/config.rb b/chef-config/lib/chef-config/config.rb
index 1f98936188..c41e2b420f 100644
--- a/chef-config/lib/chef-config/config.rb
+++ b/chef-config/lib/chef-config/config.rb
@@ -970,10 +970,10 @@ module ChefConfig
     # TODO add some post-file-parsing logic that automatically calls this so
     # users don't have to
     def self.export_proxies
-      export_proxy("http", http_proxy, http_proxy_user, http_proxy_pass) if http_proxy
-      export_proxy("https", https_proxy, https_proxy_user, https_proxy_pass) if https_proxy
-      export_proxy("ftp", ftp_proxy, ftp_proxy_user, ftp_proxy_pass) if ftp_proxy
-      export_no_proxy(no_proxy) if no_proxy
+      export_proxy("http", http_proxy, http_proxy_user, http_proxy_pass) if has_key?(:http_proxy)
+      export_proxy("https", https_proxy, https_proxy_user, https_proxy_pass) if has_key?(:https_proxy)
+      export_proxy("ftp", ftp_proxy, ftp_proxy_user, ftp_proxy_pass) if has_key?(:ftp_proxy)
+      export_no_proxy(no_proxy) if has_key?(:no_proxy)
     end
 
     # Character classes for Addressable