Merge pull request 'feat: puppet::client multiple altnames' (#221) from neoloc/puppetdbapi_certs into develop

Reviewed-on: unkinben/puppet-prod#221

site/profiles/manifests/puppet/client.pp

@@ -4,7 +4,7 @@
 #
 # site/profile/manifests/puppet/client.pp
 class profiles::puppet::client (
-  String $dns_alt_names       = $trusted['certname'],
+  Array  $dns_alt_names       = [$trusted['certname']],
   String $server              = 'puppetmaster',
   String $ca_server           = 'puppetca',
   String $environment         = 'develop',
@@ -17,6 +17,9 @@ class profiles::puppet::client (
   # dont manage puppet.conf if this is a puppetmaster
   if $facts['enc_role'] != 'roles::infra::puppet::master' {
 
+
+    $dns_alt_names_string = join(sort($dns_alt_names), ',')
+
     # Assuming you want to manage puppet.conf with this profile
     file { '/etc/puppetlabs/puppet/puppet.conf':
       ensure  => 'present',

site/profiles/templates/puppet/client/puppet.conf.erb

@@ -1,5 +1,5 @@
 [main]
-dns_alt_names = <%= @dns_alt_names %>
+dns_alt_names = <%= @dns_alt_names_string %>
 
 [agent]
 server = <%= @server %>