From: Peter Palfrader <peter@palfrader.org>
Date: Mon, 9 Sep 2019 20:35:32 +0000 (+0200)
Subject: anarcat points out that maybe Optional[String] is better to use here
X-Git-Url: https://git.adam-barratt.org.uk/?a=commitdiff_plain;h=e9cc3a59c0aa2f4f1b4288a945c4ea2a3a5703c6;p=mirror%2Fdsa-puppet.git

anarcat points out that maybe Optional[String] is better to use here

No idea if it works, but we'll find out eventually
---

diff --git a/modules/ssh/manifests/authorized_key_add.pp b/modules/ssh/manifests/authorized_key_add.pp
index 122a392d8..ec357ff70 100644
--- a/modules/ssh/manifests/authorized_key_add.pp
+++ b/modules/ssh/manifests/authorized_key_add.pp
@@ -4,9 +4,9 @@
 define ssh::authorized_key_add(
   String $target_user,
   String $command,
-  Variant[String, Undef] $key,
   Variant[Array[String], String] $collect_tag,
   String $restrict = 'restrict',
+  Optional[String] $key,
   Array[Stdlib::IP::Address] $from_hosts = $base::public_addresses,
 ) {
   $from = $from_hosts.join(',')