Skip to content

Conversation

@andrewkmin
Copy link
Contributor

@andrewkmin andrewkmin commented Mar 9, 2023

Exit to prevent overwriting existing keys

@andrewkmin andrewkmin force-pushed the andrew/prevent-overwrite branch from 273ec8e to 2794c6c Compare March 9, 2023 23:51
@andrewkmin andrewkmin force-pushed the andrew/prevent-overwrite branch from 2794c6c to a68649a Compare March 9, 2023 23:53
return cli.Exit("Error checking for pre-existing keys", 1)
}
if pubExists || privExists {
return cli.Exit("A keypair with that name already exists! Exiting...", 1)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alternatively, we could offer the user the option to elect to overwrite, but I would imagine this is sufficient.

Copy link
Contributor

@keyz-tk keyz-tk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@andrewkmin andrewkmin merged commit 09aad8b into main Mar 10, 2023
@andrewkmin andrewkmin deleted the andrew/prevent-overwrite branch March 10, 2023 19:28
lrvick pushed a commit that referenced this pull request Apr 29, 2023
Prevent overwriting keys

Former-commit-id: 09aad8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants