diff options
| author | Sam Potts <sam@potts.es> | 2019-04-11 21:18:15 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-04-11 21:18:15 +1000 |
| commit | 0f3098040d8650a762067d1f1aa5ab520cb9b90c (patch) | |
| tree | 4af0610d04897090820ba8a8b0beec7eae059a92 /src | |
| parent | 6e68ad6d1585e811c7f0624b71633d0c0bbce48d (diff) | |
| parent | 21539be3f2b01a549c920ee683c2121f53929d96 (diff) | |
| download | plyr-0f3098040d8650a762067d1f1aa5ab520cb9b90c.tar.lz plyr-0f3098040d8650a762067d1f1aa5ab520cb9b90c.tar.xz plyr-0f3098040d8650a762067d1f1aa5ab520cb9b90c.zip | |
Merge pull request #1407 from freezer278/http-youtube-fix
fixed setting youtube host for non-https case
Diffstat (limited to 'src')
| -rw-r--r-- | src/js/plugins/youtube.js | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/js/plugins/youtube.js b/src/js/plugins/youtube.js index 0bd232e0..9c447f7d 100644 --- a/src/js/plugins/youtube.js +++ b/src/js/plugins/youtube.js @@ -34,6 +34,14 @@ function assurePlaybackState(play) { } } +function getHost(config) { + if (config.noCookie) { + return 'https://www.youtube-nocookie.com'; + } + + return `${window.location.protocol}//www.youtube.com`; +} + const youtube = { setup() { // Add embed class for responsive @@ -130,7 +138,7 @@ const youtube = { player.media = replaceElement(container, player.media); // Id to poster wrapper - const posterSrc = format => `https://img.youtube.com/vi/${videoId}/${format}default.jpg`; + const posterSrc = format => `https://i.ytimg.com/vi/${videoId}/${format}default.jpg`; // Check thumbnail images in order of quality, but reject fallback thumbnails (120px wide) loadImage(posterSrc('maxres'), 121) // Higest quality and unpadded @@ -151,7 +159,7 @@ const youtube = { // https://developers.google.com/youtube/iframe_api_reference player.embed = new window.YT.Player(id, { videoId, - host: config.noCookie ? 'https://www.youtube-nocookie.com' : undefined, + host: getHost(config), playerVars: extend( {}, { |