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: {