ports_mirror: replace the vhost_listen string with an Array of IP addresses

[mirror/dsa-puppet.git] / modules / roles / manifests / ports_mirror.pp

diff --git a/modules/roles/manifests/ports_mirror.pp b/modules/roles/manifests/ports_mirror.pp
index d8ae988..1f31af2 100644 (file)
--- a/modules/roles/manifests/ports_mirror.pp
+++ b/modules/roles/manifests/ports_mirror.pp
@@ -1,12 +1,22 @@
+# a mirror of ports.debian.org
+#
+# @param listen_addr IP addresses to have apache listen on
+# @param onion_service provide the onion service from this host
 class roles::ports_mirror(
-  String  $vhost_listen = '*:80',
+  Array[Stdlib::IP::Address] $listen_addr = [],
   Boolean $onion_service = false,
-  Optional[Stdlib::IP::Address] $onion_addr = undef,
 ) {
   include roles::archvsync_base
 
   include apache2
 
+  $_enclosed_addresses = empty($listen_addr) ? {
+    true    => ['*'],
+    default => enclose_ipv6($listen_addr),
+  }
+  $_addr_port_elements = $_enclosed_addresses.map |$a| { "${a}:80" }
+  $vhost_listen = join($_addr_port_elements, ' ')
+
   $mirror_basedir_prefix = hiera('role_config__mirrors.mirror_basedir_prefix')
   $archive_root = "${mirror_basedir_prefix}debian-ports"
 
@@ -16,8 +26,9 @@ class roles::ports_mirror(
   }
 
   if $onion_service {
+    $onion_addr = filter_ipv4($listen_addr)[0]
     if ! $onion_addr {
-      fail("Do not have an onion_addr set for ${::hostname}.")
+      fail("Do not have a useable address for the onionservice on ${::hostname}.  Is \$listen_addr empty or does it not have an IPv4 address?.")
     }
 
     onion::service { 'ftp.ports.debian.org':