feat(FEC-10656): set forceRedirectExternalStreams as true by default on IE11 and Chromecast

src/common/utils/setup-helpers.js

@@ -15,7 +15,6 @@ import {
 } from '@playkit-js/playkit-js';
 import {ValidationErrorType} from './validation-error';
 import StorageManager from '../storage/storage-manager';
-import {configureExternalStreamRedirect} from './external-stream-redirect-helper';
 import {RemotePlayerManager} from '../cast/remote-player-manager';
 import {RemoteControl} from '../cast/remote-control';
 import {KalturaPlayer} from '../../kaltura-player';
@@ -299,7 +298,6 @@ function getDefaultOptions(options: PartialKPOptionsObject): KPOptionsObject {
   configureIMADefaultOptions(defaultOptions);
   configureDAIDefaultOptions(defaultOptions);
   configureBumperDefaultOptions(defaultOptions);
-  configureExternalStreamRedirect(defaultOptions);
   maybeSetFullScreenConfig(defaultOptions);
   maybeSetCapabilitiesForIos(defaultOptions);
   return defaultOptions;

src/kaltura-player.js

@@ -16,6 +16,7 @@ import {RemoteSession} from './common/cast/remote-session';
 import {ControllerProvider, AdsController} from './common/controllers';
 import {BasePlugin} from './common/plugins';
 import {PluginManager} from './common/plugins';
+import {getDefaultRedirectOptions} from 'player-defaults';
 import {
   Error,
   EventManager,
@@ -106,6 +107,7 @@ class KalturaPlayer extends FakeEventTarget {
             }
             mediaConfig.sources = Utils.Object.mergeDeep(mediaConfig.sources, mediaOptions);
           }
+          this.configure(getDefaultRedirectOptions(this.config, mediaConfig));
           this.setMedia(mediaConfig);
         },
         e =>

src/ott/player-defaults.js

@@ -27,3 +27,12 @@ export function setDefaultAnalyticsPlugin(options: KPOptionsObject): void {
     });
   }
 }
+
+/**
+ * get the default config for forcing external stream redirect.
+ * @public
+ * @returns {Object} - config object
+ */
+export function getDefaultRedirectOptions(): Object {
+  return {};
+}

src/ovp/player-defaults.js

@@ -1,5 +1,5 @@
 // @flow
-import {Utils} from '@playkit-js/playkit-js';
+import {Env, Utils, MediaType} from '@playkit-js/playkit-js';
 
 /**
  * Sets the default analytics plugin for the ovp player.
@@ -17,3 +17,75 @@ export function setDefaultAnalyticsPlugin(options: KPOptionsObject): void {
     });
   }
 }
+
+/**
+ * JSONP handler function, returns the direct manifest uri.
+ * @private
+ * @param {Object} data - The json object that returns from the server.
+ * @param {string} uri - Original request uri.
+ * @returns {string} - The direct uri.
+ */
+function getDirectManifestUri(data: Object, uri: string): string {
+  const getHostName = uri => {
+    const parser = document.createElement('a');
+    parser.href = uri;
+    return parser.hostname;
+  };
+  // if the json contains one url, it means it is a redirect url. if it contains few urls, it means its the flavours
+  // so we should use the original url.
+  const uriHost = getHostName(uri);
+  let hasOneFlavor = false;
+  let redirectedUriHost = '';
+  let redirectedUri = '';
+  if (data) {
+    if (data.flavors && Array.isArray(data.flavors)) {
+      hasOneFlavor = data.flavors.length === 1;
+      redirectedUriHost = hasOneFlavor && getHostName(data.flavors[0].url);
+      redirectedUri = data.flavors[0].url;
+    } else if (data.result) {
+      hasOneFlavor = true;
+      redirectedUriHost = getHostName(data.result.url);
+      redirectedUri = data.result.url;
+    }
+  }
+  if (hasOneFlavor && uriHost !== redirectedUriHost) {
+    return redirectedUri;
+  }
+  return uri;
+}
+
+/**
+ * get the default config for forcing external stream redirect.
+ * @public
+ * @param {KPOptionsObject} playerOptions - The player config.
+ * @param {KPOptionsObject} mediaOptions - The media config.
+ * @returns {Object} - config object
+ */
+export function getDefaultRedirectOptions(playerOptions: KPOptionsObject, mediaOptions: KPOptionsObject = {}): Object {
+  const configObj = {};
+  if (mediaOptions.sources && mediaOptions.sources.type === MediaType.LIVE && (Env.browser.name === 'IE' || Env.device.model === 'Chromecast')) {
+    const playerForceRedirectExternalStreams = Utils.Object.getPropertyPath(playerOptions, 'sources.options.forceRedirectExternalStreams');
+    const mediaForceRedirectExternalStreams = Utils.Object.getPropertyPath(mediaOptions, 'sources.options.forceRedirectExternalStreams');
+    if (typeof playerForceRedirectExternalStreams !== 'boolean' && typeof mediaForceRedirectExternalStreams !== 'boolean') {
+      Utils.Object.mergeDeep(configObj, {
+        sources: {
+          options: {
+            forceRedirectExternalStreams: true
+          }
+        }
+      });
+    }
+  }
+  const playerRedirectExternalStreamsHandler = Utils.Object.getPropertyPath(playerOptions, 'sources.options.redirectExternalStreamsHandler');
+  const mediaRedirectExternalStreamsHandler = Utils.Object.getPropertyPath(mediaOptions, 'sources.options.redirectExternalStreamsHandler');
+  if (typeof playerRedirectExternalStreamsHandler !== 'function' && typeof mediaRedirectExternalStreamsHandler !== 'function') {
+    Utils.Object.mergeDeep(configObj, {
+      sources: {
+        options: {
+          redirectExternalStreamsHandler: getDirectManifestUri
+        }
+      }
+    });
+  }
+  return configObj;
+}

test/src/ovp/player-defaults.spec.js

@@ -0,0 +1,84 @@
+import {getDefaultRedirectOptions} from './../../../src/ovp/player-defaults';
+import {Env, MediaType} from '@playkit-js/playkit-js';
+
+describe('getDefaultRedirectOptions', function () {
+  let sandbox;
+  beforeEach(function () {
+    sandbox = sinon.createSandbox();
+  });
+
+  afterEach(function () {
+    sandbox.restore();
+  });
+  describe('forceRedirectExternalStreams', function () {
+    it('should return void for vod on ie11', function () {
+      sandbox.stub(Env, 'browser').get(() => ({name: 'IE'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.VOD}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return void for vod on chromecast', function () {
+      sandbox.stub(Env, 'device').get(() => ({model: 'Chromecast'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.VOD}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return void for live chrome', function () {
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.LIVE}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return true for live on ie11', function () {
+      sandbox.stub(Env, 'browser').get(() => ({name: 'IE'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.LIVE}});
+      defaultConfig.sources.options.forceRedirectExternalStreams.should.be.true;
+    });
+
+    it('should return true for live on chromecast', function () {
+      sandbox.stub(Env, 'device').get(() => ({model: 'Chromecast'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.LIVE}});
+      defaultConfig.sources.options.forceRedirectExternalStreams.should.be.true;
+    });
+
+    it('should return void for live on ie11 if already configured on player config', function () {
+      sandbox.stub(Env, 'browser').get(() => ({name: 'IE'}));
+      const defaultConfig = getDefaultRedirectOptions({sources: {options: {forceRedirectExternalStreams: false}}}, {sources: {type: MediaType.LIVE}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return void for live on ie11 if already configured on media config', function () {
+      sandbox.stub(Env, 'browser').get(() => ({name: 'IE'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.LIVE, options: {forceRedirectExternalStreams: false}}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return void for live on chromecast if already configured on player config', function () {
+      sandbox.stub(Env, 'device').get(() => ({model: 'Chromecast'}));
+      const defaultConfig = getDefaultRedirectOptions({sources: {options: {forceRedirectExternalStreams: false}}}, {sources: {type: MediaType.LIVE}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+
+    it('should return void for live on chromecast if already configured on media config', function () {
+      sandbox.stub(Env, 'device').get(() => ({model: 'Chromecast'}));
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {type: MediaType.LIVE, options: {forceRedirectExternalStreams: false}}});
+      (defaultConfig.sources.options.forceRedirectExternalStreams === undefined).should.be.true;
+    });
+  });
+
+  describe('redirectExternalStreamsHandler', function () {
+    it('should return the default', function () {
+      const defaultConfig = getDefaultRedirectOptions({});
+      (typeof defaultConfig.sources.options.redirectExternalStreamsHandler === 'function').should.be.true;
+    });
+
+    it('should return void if already configured on player config', function () {
+      const defaultConfig = getDefaultRedirectOptions({sources: {options: {redirectExternalStreamsHandler: () => {}}}});
+      (defaultConfig.sources === undefined).should.be.true;
+    });
+
+    it('should return void if already configured on media config', function () {
+      const defaultConfig = getDefaultRedirectOptions({}, {sources: {options: {redirectExternalStreamsHandler: () => {}}}});
+      (defaultConfig.sources === undefined).should.be.true;
+    });
+  });
+});