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

concern - UserDelete method called on only existing BMC user #182

Open
jacobweinstock opened this issue Jan 7, 2021 · 0 comments
Open

Comments

@jacobweinstock
Copy link
Member

With this interface method below, we could potentially lock ourselves out of a BMC by removing all users. If the user given is the only user in the BMC, we might not want to delete it.

UserDelete(ctx context.Context, user string) (ok bool, err error)

We should probably add some comments to the method so that implementors know they should check if the user is the only user left before deleting. We could also maybe add a parameter like force bool to delete the user regardless.

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

No branches or pull requests

1 participant