Skip to content

Commit

Permalink
Merge pull request #3055 from DSpace/backport-3046-to-dspace-7_x
Browse files Browse the repository at this point in the history
[Port dspace-7_x] Fix self link don't match warnings for authorization endpoint
  • Loading branch information
tdonohue authored May 16, 2024
2 parents 8ea7a25 + e0fb590 commit 088e56f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ describe('AuthorizationDataService', () => {
const ePersonUuid = 'fake-eperson-uuid';

function createExpected(providedObjectUrl: string, providedEPersonUuid?: string, providedFeatureId?: FeatureID): FindListOptions {
const searchParams = [new RequestParam('uri', providedObjectUrl)];
const searchParams = [new RequestParam('uri', providedObjectUrl, false)];
if (hasValue(providedFeatureId)) {
searchParams.push(new RequestParam('feature', providedFeatureId));
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,8 @@ export class AuthorizationDataService extends BaseDataService<Authorization> imp
if (isNotEmpty(options.searchParams)) {
params = [...options.searchParams];
}
params.push(new RequestParam('uri', objectUrl));
// TODO fix encode the uri parameter in the self link in the backend and set encodeValue to true afterwards
params.push(new RequestParam('uri', objectUrl, false));
if (hasValue(featureId)) {
params.push(new RequestParam('feature', featureId));
}
Expand Down

0 comments on commit 088e56f

Please sign in to comment.