Mirrors/unleash.unleash
1
0
Fork 0
mirror of https://github.com/Unleash/unleash.git synced 2025-10-27 11:02:16 +01:00
Code Issues Projects Releases Wiki Activity

fix: consider ADMIN in API tokens fetch permissions (#4032)

Browse Source 
https://github.com/Unleash/unleash/pull/4019 introduced a bug where API
token filtering was not taking into account ADMIN permissions, which
means the API tokens were not being displayed on the UI.
This commit is contained in:
Nuno Góis 2023-06-21 13:12:21 +01:00 committed by GitHub
parent 35d323aea7
commit 197df96ff4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/lib/routes/admin-api/api-token.ts
View File

@ -353,13 +353,16 @@ export class ApiTokenController extends Controller {
const userPermissions = await this.accessService.getPermissionsForUser( const userPermissions = await this.accessService.getPermissionsForUser(
user, user,
); );
let allowedTokenTypes = [
const allowedTokenTypes = [
READ_ADMIN_API_TOKEN, READ_ADMIN_API_TOKEN,
READ_CLIENT_API_TOKEN, READ_CLIENT_API_TOKEN,
READ_FRONTEND_API_TOKEN, READ_FRONTEND_API_TOKEN,
] ]
.filter((readPerm) => .filter((readPerm) =>
userPermissions.some((p) => p.permission === readPerm), userPermissions.some(
(p) => p.permission === readPerm || p.permission === ADMIN,
),
) )
.map(permissionToTokenType) .map(permissionToTokenType)
.filter((t) => t); .filter((t) => t);