diff --git a/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php b/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php index a4dfb1eb00bdb3568a09e8dc561a49482091ba66..acdee82a9e35f127764572f4aa3d896861c445bb 100644 --- a/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php +++ b/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php @@ -108,7 +108,7 @@ class YouTubeRenderer implements FileRendererInterface $urlParams[] = 'loop=1'; } if (!isset($options['enablejsapi']) || !empty($options['enablejsapi'])) { - $urlParams[] = 'enablejsapi=1&origin=' . GeneralUtility::getIndpEnv('HTTP_HOST'); + $urlParams[] = 'enablejsapi=1&origin=' . GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL'); } $urlParams[] = 'showinfo=' . (int)!empty($options['showinfo']); diff --git a/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php b/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php index 82060c76b4a496689b960f32106cbbe9aa2e3157..477b6e9c9bc6192a76808d5d7225e5dd34f88dd5 100644 --- a/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php +++ b/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php @@ -39,6 +39,7 @@ class YouTubeRendererTest extends UnitTestCase parent::setUp(); GeneralUtility::flushInternalRuntimeCaches(); $_SERVER['HTTP_HOST'] = 'test.server.org'; + $_SERVER['REQUEST_URI'] = '/test-page/'; /** @var YouTubeHelper|\PHPUnit_Framework_MockObject_MockObject $youTubeHelper */ $youTubeHelper = $this->getAccessibleMock(YouTubeHelper::class, array('getOnlineMediaId'), array('youtube')); @@ -94,7 +95,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&enablejsapi=1&origin=test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&enablejsapi=1&origin=http://test.server.org/test-page/&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200') ); } @@ -108,7 +109,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&loop=1&enablejsapi=1&origin=test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&loop=1&enablejsapi=1&origin=http://test.server.org/test-page/&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', array('loop' => 1)) ); } @@ -122,7 +123,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http://test.server.org/test-page/&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', array('autoplay' => 1)) ); } @@ -141,7 +142,7 @@ class YouTubeRendererTest extends UnitTestCase $fileReferenceMock->expects($this->any())->method('getOriginalFile')->willReturn($fileResourceMock); $this->assertSame( - '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="//www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http://test.server.org/test-page/&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileReferenceMock, '300m', '200') ); } @@ -155,7 +156,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="//www.youtube.com/embed/7331?autohide=1&autoplay=1&enablejsapi=1&origin=test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="//www.youtube.com/embed/7331?autohide=1&autoplay=1&enablejsapi=1&origin=http://test.server.org/test-page/&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', array('controls' => 0, 'autoplay' => 1)) ); }