From 9e2ab6d6470e073e868f58878f9e81bdf418ff95 Mon Sep 17 00:00:00 2001 From: Diana Olarte Date: Thu, 9 Oct 2025 23:45:58 +1100 Subject: [PATCH] fix: update the library scope according to the backend requirements --- src/authz-module/data/api.ts | 6 +++--- src/authz-module/data/hooks.test.tsx | 2 +- src/authz-module/libraries-manager/context.tsx | 3 ++- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/authz-module/data/api.ts b/src/authz-module/data/api.ts index fd570d6..ea1e223 100644 --- a/src/authz-module/data/api.ts +++ b/src/authz-module/data/api.ts @@ -16,7 +16,7 @@ export type PermissionsByRole = { // TODO: replece api path once is created export const getTeamMembers = async (object: string): Promise => { - const { data } = await getAuthenticatedHttpClient().get(getApiUrl(`/api/authz/v1/roles/users?scope=${object}`)); + const { data } = await getAuthenticatedHttpClient().get(getApiUrl(`/api/authz/v1/roles/users/?scope=${object}`)); return camelCaseObject(data.results); }; @@ -32,8 +32,8 @@ export const getLibrary = async (libraryId: string): Promise => }; export const getPermissionsByRole = async (scope: string): Promise => { - const url = new URL(getApiUrl('/api/authz/v1/roles')); + const url = new URL(getApiUrl('/api/authz/v1/roles/')); url.searchParams.append('scope', scope); const { data } = await getAuthenticatedHttpClient().get(url); - return camelCaseObject(data); + return camelCaseObject(data.results); }; diff --git a/src/authz-module/data/hooks.test.tsx b/src/authz-module/data/hooks.test.tsx index c8fb969..79fafc4 100644 --- a/src/authz-module/data/hooks.test.tsx +++ b/src/authz-module/data/hooks.test.tsx @@ -132,7 +132,7 @@ describe('usePermissionsByRole', () => { ]; getAuthenticatedHttpClient.mockReturnValue({ - get: jest.fn().mockResolvedValue({ data: mockRoles }), + get: jest.fn().mockResolvedValue({ data: { results: mockRoles } }), }); const wrapper = createWrapper(); diff --git a/src/authz-module/libraries-manager/context.tsx b/src/authz-module/libraries-manager/context.tsx index e9c8905..0def9e0 100644 --- a/src/authz-module/libraries-manager/context.tsx +++ b/src/authz-module/libraries-manager/context.tsx @@ -9,7 +9,8 @@ import { PermissionMetadata, ResourceMetadata, Role } from 'types'; import { libraryPermissions, libraryResourceTypes, libraryRolesMetadata } from './constants'; const LIBRARY_TEAM_PERMISSIONS = ['view_library_team', 'manage_library_team']; -const LIBRARY_AUTHZ_SCOPE = 'lib:*'; +// Note: This value can change in the future +const LIBRARY_AUTHZ_SCOPE = '*'; export type AppContextType = { authenticatedUser: {