Skip to content

Commit 12976d6

Browse files
authored
EZP-21059: Fixed infinite redirect loop in Ibexa\Rest\Server\Controller\Content::redirectCurrentVersionRelations
Ibexa\Rest\Server\Controller\Content::redirectCurrentVersionRelations was redirecting to itself's route and looping indefinitely.
1 parent 15bc46a commit 12976d6

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/lib/Server/Controller/Content.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -557,7 +557,7 @@ public function redirectCurrentVersionRelations($contentId)
557557

558558
return new Values\TemporaryRedirect(
559559
$this->router->generate(
560-
'ibexa.rest.redirect_current_version_relations',
560+
'ibexa.rest.load_version_relations',
561561
[
562562
'contentId' => $contentId,
563563
'versionNumber' => $contentInfo->currentVersionNo,

tests/bundle/Functional/ContentTest.php

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -328,9 +328,7 @@ public function testRedirectCurrentVersionRelations($restContentHref)
328328

329329
self::assertHttpResponseCodeEquals($response, 307);
330330

331-
// @todo Fix, see EZP-21059. Meanwhile, the test is skipped if it fails as expected
332-
// self::assertHttpResponseHasHeader( $response, 'Location', "$restContentHref/versions/1/relations" );
333-
self::assertHttpResponseHasHeader($response, 'Location', "$restContentHref/relations?versionNumber=1");
331+
self::assertHttpResponseHasHeader($response, 'Location', "$restContentHref/versions/1/relations");
334332
self::markTestIncomplete('@todo Fix issue EZP-21059');
335333
}
336334

0 commit comments

Comments
 (0)