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

On delete group, SAM error message, reported in Terra, is misleading about group in group #399

Open
mbookman opened this issue Dec 18, 2019 · 1 comment

Comments

@mbookman
Copy link

Reproducible test case:

In Terra:

  1. Create a group: deleteable-test-group
  2. Create a workspace: deleteable-workspace
  3. Share deleteable-workspace with deleteable-test-group as Readers
  4. Try to delete the group deleteable-test-group

You will get an error message here:

Error deleting group
Error 409: group deleteable-test-group cannot be deleted because it is a member of
at least 1 other group
Source: sam

The error message here may be accurate from a SAM perspective, but in the Terra UI it is a little confusing. In the example above, the user group created is NOT a member of another group; it is a member of a workspace.

Not sure if SAM can provide a better error message:

Error 409: group deleteable-test-group cannot be deleted because it is a member of
at least 1 other group, workspace, or billing project

or

Error 409: group deleteable-test-group cannot be deleted because it is a member of:

workspace: deleteable-workspace

or something to that effect. Not sure if in all cases providing the specific named workspace, billing account, authorization domain would be information leakage. As Admin of the group, would someone else's use of the group prevent me from deleting the group? How would I chase down the user(s) of the group so that I could clean it up?

@melissachang
Copy link
Contributor

+1, it would be nice if error said what group/resource deleteable-test-group is in. I think the only way to find out is to run sql against SAM db.

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

2 participants