From c02b1151e584ea6c0bca35ba6aa87504566f6949 Mon Sep 17 00:00:00 2001 From: zsdc Date: Thu, 11 Jul 2024 15:04:16 +0300 Subject: SSH KEY: T6568: Fixed adding SSH keys with same comments If a key with the same comment already exists in a configuration, generate a new ID for a new one. Example of such a case: ``` ssh-rsa my_user_name ssh-ed25519 my_user_name ``` --- cloudinit/config/cc_vyos.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/cloudinit/config/cc_vyos.py b/cloudinit/config/cc_vyos.py index 9c9ac697..96324e4b 100644 --- a/cloudinit/config/cc_vyos.py +++ b/cloudinit/config/cc_vyos.py @@ -91,6 +91,11 @@ def set_ssh_login(config, user, key_string): logger.info("Generating UUID for an SSH key because a comment is empty or unacceptable by CLI") key_parsed.comment = "cloud-init-{}".format(uuid4()) + # check if a key with the same comment already exists + if config.exists(['system', 'login', 'user', user, 'authentication', 'public-keys', key_parsed.comment]): + logger.debug("Generating UUID for an SSH key because a public key with comment {} already exists for user {}".format(key_parsed.comment, user)) + key_parsed.comment = "cloud-init-{}".format(uuid4()) + config.set(['system', 'login', 'user', user, 'authentication', 'public-keys', key_parsed.comment, 'key'], value=key_parsed.base64, replace=True) config.set(['system', 'login', 'user', user, 'authentication', 'public-keys', key_parsed.comment, 'type'], value=key_parsed.keytype, replace=True) if key_parsed.options: -- cgit v1.2.3