From: Daniele Sluijters <daenney@users.noreply.github.com>
Date: Sun, 1 Mar 2015 13:42:39 +0000 (+0100)
Subject: apt::key: Be explicit about the keyserver.
X-Git-Tag: 2.0.0~26^2
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=0cb48b0c470c81b369a54336409789b28bb09f16;p=puppet-modules%2Fpuppetlabs-apt.git

apt::key: Be explicit about the keyserver.

The behaviour of passing down undef through multiple layers gets fuzzy
so for now be explicit about the keyserver.

Once Puppet 4 is out and this behaviour has been crystallised and tested
we can revisit it.
---

diff --git a/manifests/key.pp b/manifests/key.pp
index 05bda97..6761e69 100644
--- a/manifests/key.pp
+++ b/manifests/key.pp
@@ -42,9 +42,7 @@
 #   _default_: +undef+
 #
 #   The keyserver from where to fetch our GPG key. It can either be a domain
-#   name or url. It defaults to
-#   undef which results in apt_key's default keyserver being used,
-#   currently +keyserver.ubuntu.com+.
+#   name or url. It defaults to +keyserver.ubuntu.com+.
 #
 # [*options*]
 #   _default_: +undef+
@@ -55,7 +53,7 @@ define apt::key (
   $ensure  = present,
   $content = undef,
   $source  = undef,
-  $server  = undef,
+  $server  = $::apt::keyserver,
   $options = undef,
 ) {
 
diff --git a/manifests/params.pp b/manifests/params.pp
index 8799f7b..14401c6 100644
--- a/manifests/params.pp
+++ b/manifests/params.pp
@@ -11,6 +11,7 @@ class apt::params {
   $conf_d         = "${root}/apt.conf.d"
   $preferences    = "${root}/preferences"
   $preferences_d  = "${root}/preferences.d"
+  $keyserver      = 'keyserver.ubuntu.com'
 
   if $::osfamily != 'Debian' {
     fail('This module only works on Debian or derivatives like Ubuntu')
@@ -52,7 +53,7 @@ class apt::params {
   }
 
   $source_key_defaults = {
-    'server'  => $default_keyserver,
+    'server'  => $keyserver,
     'options' => undef,
     'content' => undef,
     'source'  => undef,