diff --git a/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php b/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php index f0e1b88d4dcb2cece6dc0e83c12018bd6f57a248..68ba3de031c1ef233c9fa4a4a184547633bb1cfc 100644 --- a/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php +++ b/typo3/sysext/core/Classes/Resource/Rendering/YouTubeRenderer.php @@ -119,7 +119,7 @@ class YouTubeRenderer implements FileRendererInterface $urlParams[] = 'rel=' . (int)(bool)$options['relatedVideos']; } if (!isset($options['enablejsapi']) || !empty($options['enablejsapi'])) { - $urlParams[] = 'enablejsapi=1&origin=' . GeneralUtility::getIndpEnv('TYPO3_REQUEST_HOST'); + $urlParams[] = 'enablejsapi=1&origin=' . rawurlencode(GeneralUtility::getIndpEnv('TYPO3_REQUEST_HOST')); } $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 d837da34e24044d827ca3583bf74ac3313614a14..3b5487a592c25ffe561c90e605e0040bfc649c46 100644 --- a/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php +++ b/typo3/sysext/core/Tests/Unit/Resource/Rendering/YouTubeRendererTest.php @@ -94,7 +94,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200') ); } @@ -108,7 +108,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&loop=1&playlist=7331&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&loop=1&playlist=7331&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', ['loop' => 1]) ); } @@ -122,7 +122,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', ['autoplay' => 1]) ); } @@ -141,7 +141,7 @@ class YouTubeRendererTest extends UnitTestCase $fileReferenceMock->expects($this->any())->method('getOriginalFile')->willReturn($fileResourceMock); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&autoplay=1&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileReferenceMock, '300m', '200') ); } @@ -155,7 +155,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&autoplay=1&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&autoplay=1&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', ['controls' => 0, 'autoplay' => 1]) ); } @@ -169,7 +169,7 @@ class YouTubeRendererTest extends UnitTestCase $fileResourceMock = $this->createMock(File::class); $this->assertSame( - '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&rel=0&enablejsapi=1&origin=http://test.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', + '<iframe src="https://www.youtube.com/embed/7331?autohide=1&controls=2&rel=0&enablejsapi=1&origin=http%3A%2F%2Ftest.server.org&showinfo=0" allowfullscreen width="300" height="200"></iframe>', $this->subject->render($fileResourceMock, '300m', '200', ['relatedVideos' => 0]) ); }