]> review.fuel-infra Code Review - puppet-modules/puppetlabs-apt.git/commit
Fix inconsistent $proxy_host handling in apt and apt::ppa.
authorDaniel Friesen <daniel@nadir-seen-fire.com>
Thu, 10 Jul 2014 23:38:45 +0000 (16:38 -0700)
committerMorgan Haskel <morgan@puppetlabs.com>
Fri, 8 Aug 2014 13:38:48 +0000 (09:38 -0400)
commit9510d1d37a67db4038b3ff449cf144388f147ac0
tree41f8bcac497fe698549c8cb66a7aab7810ffc79d
parent7617b54071f174b7f747b0c830ae1a4ae764824c
Fix inconsistent $proxy_host handling in apt and apt::ppa.

- The default for $proxy_host is undef
- apt considers $proxy_set to be absent if $proxy_host is undef
- apt::ppa considers proxy_env to be empty if $proxy_host is false or ''

This results in apt::ppa to consider $proxy_host to be set when the default undef is used
breaking ppa resources because $proxy_env becomes:
  [http_proxy=http://:8080, https_proxy=http://:8080]

Fix this by making both apt and apt::ppa consider $proxy_host to be unset when it is
any of false, '', or undef.
manifests/init.pp
manifests/ppa.pp