Filippo Giunchedi has submitted this change and it was merged.

Change subject: Fix key name in scap::target
......................................................................


Fix key name in scap::target

Change-Id: I23b0939ba51e0442be08931fd5f3d5c661ed01cb
---
M modules/scap/manifests/target.pp
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Filippo Giunchedi: Verified; Looks good to me, approved



diff --git a/modules/scap/manifests/target.pp b/modules/scap/manifests/target.pp
index 8479ca4..062981c 100644
--- a/modules/scap/manifests/target.pp
+++ b/modules/scap/manifests/target.pp
@@ -78,9 +78,11 @@
             }
         }
         if !defined(Ssh::Userkey[$deploy_user]) {
+            $key_name_safe = regsubst($key_name, '\W', '_', 'G')
+
             ssh::userkey { $deploy_user:
                 ensure  => 'present',
-                content => secret("keyholder/${key_name}.pub"),
+                content => secret("keyholder/${key_name_safe}.pub"),
             }
         }
     }

-- 
To view, visit https://gerrit.wikimedia.org/r/290973
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I23b0939ba51e0442be08931fd5f3d5c661ed01cb
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Thcipriani <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to