Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Allow removal by user for 'userkey rm' #248

Merged
merged 1 commit into from
Mar 26, 2021
Merged

Conversation

jwessel
Copy link
Contributor

@jwessel jwessel commented Feb 26, 2021

The userkey rm command implies that it can remove a key by user or the
id key, but it only works against the data base id of the key. This
patch allows the userkey rm command to work with the user name, so
that all the keys for the user can be cleared out in one command.

Signed-off-by: Jason Wessel jason.wessel@windriver.com

@auto-add-label auto-add-label bot added the rm label Feb 26, 2021
@jwessel jwessel changed the title userkey rm: Allow removal by user or id of key feat: Allow removal by user for 'userkey rm' Feb 26, 2021
@auto-add-label auto-add-label bot added enhancement and removed rm labels Feb 26, 2021
@jwessel jwessel force-pushed the master branch 2 times, most recently from eb4bbcc to 6b97ead Compare March 8, 2021 17:04
The userkey rm command implies that it can remove a key by user or the
id key, but it only works against the data base id of the key.  This
patch allows the userkey rm command to work with the user name, so
that all the keys for the user can be cleared out in one command.

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
pkg/bastion/shell.go Show resolved Hide resolved
@moul moul merged commit 40bbea5 into moul:master Mar 26, 2021
@github-actions
Copy link

🎉 This PR is included in version 1.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants