Hey Team,
I was working at this bug: <BUG>
I am ha...
# contributingm
many-queen-54958
05/20/2024, 3:00 AMHey Team,
I was working at this bug: BUG
I am having issues in replicating the bug.
I want to know how to assign a user global no-access role.
The list of roles which I can see is:
many-queen-54958
05/20/2024, 4:58 AM+ Adding
Was able to view the no-access role by manually adding true to one of the permissions.
Now to solve this bug we can do these things:
Check the role of user, and if the role of user is no-access in the get API of Personal token and then manipulate.
OR
Change the role so that we give this role some permissions.
Would like some guidance in approaching this problem, as this will be my first PR to the project.
f
fresh-football-47124
05/20/2024, 6:05 AM
@billions-xylophone-11752?
fresh-football-47124
05/20/2024, 6:05 AM
or @future-teacher-7046?
m
many-queen-54958
05/22/2024, 3:27 AMHey @fresh-football-47124,
Thanks
@billions-xylophone-11752 @future-teacher-7046 I want to confirm this thing.
Does all the user irrespective of the role should be able to see their Personal Access Token?
many-queen-54958
05/22/2024, 3:29 AMIf that is the case:
Then we can show the key the user owns irrespective of the role.
b
billions-xylophone-11752
05/22/2024, 7:14 PMHey, Akash! Sorry - I missed the original notification. Yes, your understanding is correct - all users, regardless of the role, should be able to see the personal access tokens they have created.
It looks like we're filtering all keys based on the user's read access level within
getAllApiKeysByOrganizationnoaccessm
many-queen-54958
05/23/2024, 1:04 PMGot it will raise a PR for the fix.
many-queen-54958
05/29/2024, 6:01 PM@billions-xylophone-11752 I have fixed the PR i.e. added strict equality. Please review it once more.
many-queen-54958
06/01/2024, 3:16 AMCan we get this PR merged @billions-xylophone-11752?
9 Views