Refactor YouTube player integration to use iframe for improved performance and security
This commit is contained in:
@@ -742,17 +742,20 @@ const SlideUtils = {
|
|||||||
}
|
}
|
||||||
|
|
||||||
if (isYoutube && videoId) {
|
if (isYoutube && videoId) {
|
||||||
const playerDiv = this.createElement('div', { id: 'modal-yt-player' });
|
const ytIframe = this.createElement('iframe', {
|
||||||
contentContainer.appendChild(playerDiv);
|
id: 'modal-yt-player',
|
||||||
|
src: `https://www.youtube-nocookie.com/embed/${videoId}?enablejsapi=1&origin=${encodeURIComponent(window.location.origin)}`,
|
||||||
|
allow: 'autoplay; encrypted-media',
|
||||||
|
style: 'width: 100%; height: 100%; border: none;',
|
||||||
|
});
|
||||||
|
ytIframe.setAttribute('referrerpolicy', 'strict-origin-when-cross-origin');
|
||||||
|
|
||||||
|
contentContainer.appendChild(ytIframe);
|
||||||
overlay.append(closeButton, contentContainer);
|
overlay.append(closeButton, contentContainer);
|
||||||
document.body.appendChild(overlay);
|
document.body.appendChild(overlay);
|
||||||
|
|
||||||
this.loadYouTubeIframeAPI().then(() => {
|
this.loadYouTubeIframeAPI().then(() => {
|
||||||
new YT.Player('modal-yt-player', {
|
new YT.Player(ytIframe, {
|
||||||
height: '100%',
|
|
||||||
width: '100%',
|
|
||||||
videoId: videoId,
|
|
||||||
host: 'https://www.youtube-nocookie.com',
|
|
||||||
playerVars: {
|
playerVars: {
|
||||||
autoplay: 1,
|
autoplay: 1,
|
||||||
controls: 1,
|
controls: 1,
|
||||||
@@ -761,14 +764,6 @@ const SlideUtils = {
|
|||||||
playsinline: 1,
|
playsinline: 1,
|
||||||
origin: window.location.origin,
|
origin: window.location.origin,
|
||||||
enablejsapi: 1
|
enablejsapi: 1
|
||||||
},
|
|
||||||
events: {
|
|
||||||
'onReady': (event) => {
|
|
||||||
const iframe = event.target.getIframe();
|
|
||||||
if (iframe) {
|
|
||||||
iframe.setAttribute('referrerpolicy', 'strict-origin-when-cross-origin');
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
@@ -1750,12 +1745,16 @@ const SlideCreator = {
|
|||||||
style: "opacity: 0; transition: opacity 1.2s ease-in-out;" // Start interrupted/transparent
|
style: "opacity: 0; transition: opacity 1.2s ease-in-out;" // Start interrupted/transparent
|
||||||
});
|
});
|
||||||
|
|
||||||
const ytPlayerDiv = SlideUtils.createElement("div", {
|
// Create an iframe upfront
|
||||||
|
const ytPlayerIframe = SlideUtils.createElement("iframe", {
|
||||||
id: `youtube-player-${itemId}`,
|
id: `youtube-player-${itemId}`,
|
||||||
style: "width: 100%; height: 100%;"
|
src: `https://www.youtube-nocookie.com/embed/${videoId}?enablejsapi=1&origin=${encodeURIComponent(window.location.origin)}`,
|
||||||
|
style: "width: 100%; height: 100%; border: none;",
|
||||||
|
allow: "autoplay; encrypted-media",
|
||||||
|
referrerpolicy: "strict-origin-when-cross-origin"
|
||||||
});
|
});
|
||||||
|
|
||||||
videoBackdrop.appendChild(ytPlayerDiv);
|
videoBackdrop.appendChild(ytPlayerIframe);
|
||||||
|
|
||||||
// Initialize YouTube Player
|
// Initialize YouTube Player
|
||||||
SlideUtils.loadYouTubeIframeAPI().then(() => {
|
SlideUtils.loadYouTubeIframeAPI().then(() => {
|
||||||
@@ -1800,19 +1799,10 @@ const SlideCreator = {
|
|||||||
console.info(`SponsorBlock outro detected for video ${videoId}: ending at ${playerVars.end}s`);
|
console.info(`SponsorBlock outro detected for video ${videoId}: ending at ${playerVars.end}s`);
|
||||||
}
|
}
|
||||||
|
|
||||||
STATE.slideshow.videoPlayers[itemId] = new YT.Player(`youtube-player-${itemId}`, {
|
STATE.slideshow.videoPlayers[itemId] = new YT.Player(ytPlayerIframe, {
|
||||||
height: '100%',
|
|
||||||
width: '100%',
|
|
||||||
videoId: videoId,
|
|
||||||
host: 'https://www.youtube-nocookie.com',
|
|
||||||
playerVars: playerVars,
|
playerVars: playerVars,
|
||||||
events: {
|
events: {
|
||||||
'onReady': (event) => {
|
'onReady': (event) => {
|
||||||
const iframe = event.target.getIframe();
|
|
||||||
if (iframe) {
|
|
||||||
iframe.setAttribute('referrerpolicy', 'strict-origin-when-cross-origin');
|
|
||||||
}
|
|
||||||
|
|
||||||
// Store start/end time and videoId for later use
|
// Store start/end time and videoId for later use
|
||||||
event.target._startTime = playerVars.start || 0;
|
event.target._startTime = playerVars.start || 0;
|
||||||
event.target._endTime = playerVars.end || undefined;
|
event.target._endTime = playerVars.end || undefined;
|
||||||
|
|||||||
Reference in New Issue
Block a user