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

Inconsistent User access behaviour #410

Closed
2 tasks done
strawhat5 opened this issue Apr 6, 2021 · 2 comments · Fixed by #424
Closed
2 tasks done

Inconsistent User access behaviour #410

strawhat5 opened this issue Apr 6, 2021 · 2 comments · Fixed by #424
Assignees
Labels
bug Something isn't working

Comments

@strawhat5
Copy link
Contributor

Checklist:

  • I've searched in the docs and FAQ for my answer
  • I've included steps to reproduce the bug.

Describe the bug

We have observed many inconsistencies in the User access section, details of which are added below.

Current behavior

  1. Assigning direct permission to user fails.

Screenshot 2021-04-06 at 4 45 18 PM

  1. I can assign a group to a user, even though I am not the manager of that group/project. Also, on updation it always gives Created user successfully message, even though the user already exists.

Screenshot 2021-04-06 at 4 48 10 PM

  1. I can unassign a group from a user, even though I am not the manager of that group/project. Gets the same Created user successfully message here.

Screenshot 2021-04-06 at 4 47 24 PM

  1. Even when I am a manager, I am not able to create groups (with lower permissions) for my own project.

Screenshot 2021-04-06 at 4 56 53 PM

  1. Should be able to see the permissions of all groups (View only for non-managers of that group).

Screenshot 2021-04-06 at 4 57 30 PM

  1. While adding a bunch of users, it gives User already exists error (if even one of the users already exists), but successfully creates other users. Which means the Add User operation is non-atomic.

Expected behavior

All the above inconsistencies should be resolved for a smoother user experience.

@vikramdevtron
Copy link
Contributor

@strawhat5 issue 1, 4, 5 unable to reproduce, found works as expected. issue 2 and 3 user message has been fixed, other will be fixed soon, created new ticket for that.
issue 6, User already exists on bulk creation has fixed in this PR.

@strawhat5
Copy link
Contributor Author

@nishant-d Let's connect on this to see if 1, 4, 5 can be reproduced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants