Skip to content

Commit

Permalink
fix(FEC-8235): remove default service url from OTT provider and analy…
Browse files Browse the repository at this point in the history
…tics (#125)

pass the serviceUrl to ottAnalytics config
  • Loading branch information
yairans authored May 23, 2018
1 parent 71401cc commit 104bc9a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
5 changes: 4 additions & 1 deletion src/common/plugins/plugins-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,13 @@ import {Utils} from 'playkit-js'
*/
function evaluatePluginsConfig(options: KalturaPlayerOptionsObject): void {
if (options.plugins) {
const dataModel = {
const dataModel: Object = {
pVersion: __VERSION__,
pName: __NAME__
};
if (options.provider && options.provider.env) {
dataModel['serviceUrl'] = options.provider.env.serviceUrl;
}
if (options.session && options.sources) {
const entryDataModel = {
entryId: options.sources.id,
Expand Down
3 changes: 2 additions & 1 deletion src/common/plugins/plugins-config.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@
"ottAnalytics": {
"entryId": "{{entryId}}",
"ks": "{{ks}}",
"partnerId": "{{partnerId}}"
"partnerId": "{{partnerId}}",
"serviceUrl": "{{serviceUrl}}"
},
"ima": {
"playerVersion": "{{pVersion}}",
Expand Down

0 comments on commit 104bc9a

Please sign in to comment.