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

Bug: user fetch list does not supports managers access provided by groups #2357

Closed
2 tasks done
kartik-579 opened this issue Sep 14, 2022 · 0 comments · Fixed by #2358
Closed
2 tasks done

Bug: user fetch list does not supports managers access provided by groups #2357

kartik-579 opened this issue Sep 14, 2022 · 0 comments · Fixed by #2358
Assignees
Labels
bug Something isn't working

Comments

@kartik-579
Copy link
Member

kartik-579 commented Sep 14, 2022

📜 Description

User must be authorised to fetch user list when they have super admin or manager access. Although, this works for direct permisssions but breaks for manager access that are provided through user groups.

👟 Reproduction steps

Give a user manager permission through permission group and ensure the user does not have direct manager permissions. The api for fetching user list will give forbidden error.

👍 Expected behavior

The user must be able to access the user list fetch api even with group-manager permissions.

👎 Actual Behavior

Not able to pass the api authorization.

💻 Device

Desktop/Laptop

💻 Operating system

MacOS

🌍 Browser

Chrome

🧱 Your Environment

No response

✅ Proposed Solution

No response

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

AB#508

@kartik-579 kartik-579 added the bug Something isn't working label Sep 14, 2022
@kartik-579 kartik-579 self-assigned this Sep 14, 2022
@kartik-579 kartik-579 linked a pull request Sep 14, 2022 that will close this issue
13 tasks
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.

1 participant