kodi-svtplay / xbmc-svtplay

Unofficial addon for watching SVT Play on Kodi
GNU General Public License v3.0
66 stars 34 forks source link

Use SVT multi cdn selection logic to ensure optimal playback performance #202

Closed even-stephen closed 7 years ago

linqcan commented 7 years ago

Thanks for contributing! Please provide some background and purpose of this change.

even-stephen commented 7 years ago

This change is the equivalent of what the official webplayer (on svtplay.se) is doing when resolving video links, i.e if the alt query is present it will use that as a stream url, otherwise it will use the url "as is"

Ex: useAltHlsUrl variable in: https://www.svtstatic.se/video/svtvideoplayer/js/svtvideoplayer.js

linqcan commented 7 years ago

Sorry for being slow to understand, but why is this needed? What problem or issue does it resolve?

On Fri, Apr 7, 2017, 00:44 even-stephen notifications@github.com wrote:

This change is the equivalent of what the webplayer is doing when resolving video links, i.e if the alt query is present it will use that as a stream url, otherwise it will use the url "as is"

Ex: useAltHlsUrl variable in: https://www.svtstatic.se/video/svtvideoplayer/js/svtvideoplayer.js

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/nilzen/xbmc-svtplay/pull/202#issuecomment-292347689, or mute the thread https://github.com/notifications/unsubscribe-auth/AAy12GMoP4nEBVI6tM2xVAo_Dhjl134Tks5rtWrQgaJpZM4M1oOw .

even-stephen commented 7 years ago

SVT is using a multi-CDN approach to serve the content, using the "alturl" (when present) ensures that the best performing CDN get picked for each stream start, allowing for a better user experience.

linqcan commented 7 years ago

OK, can you update the commit message to describe this? (the reason behind the change and the benefit of it)

even-stephen commented 7 years ago

Ok, updated commit message.

linqcan commented 7 years ago

Not really what I meant, but I could have been clearer I suppose. However, it will work.

'Till next time: https://gist.github.com/matthewhudson/1475276