Merge pull request #4402 from Dash-Industry-Forum/feature/mergeToDeve…

…lopment Add version 5 changes
Dash-Industry-Forum · Mar 4, 2024 · 3dbce17 · 3dbce17
2 parents 1249205 + 71ddfc9
commit 3dbce17
Show file tree

Hide file tree

Showing 485 changed files with 27,041 additions and 44,058 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -1,6 +1,6 @@
 version: 2.1
 orbs:
-    browser-tools: circleci/[email protected].6
+    browser-tools: circleci/[email protected].8
 
 executors:
     dashjs-executor:
@@ -49,13 +49,13 @@ commands:
 
     build_unit_test_steps:
         steps:
-            -   browser-tools/install-browser-tools:
-                    chrome-version: "120.0.6099.224"
+            - browser-tools/install-browser-tools
             -   run:
                     name: Build and run unit tests
                     command: |
                         npm run build
                         npm run test
+
     process_test_results: # CircleCI can not handle NaN values so we replace them with 0
         steps:
             -   run:
@@ -97,25 +97,44 @@ commands:
             -   run:
                     name: Run functional tests (<<parameters.browser>> / <<parameters.protocol>>) <<parameters.groupname>>
                     command:
-                        node test/functional/runTests.js --selenium=remote --reporters=junit --app=remote --browsers=<<parameters.browser>> --protocol=<<parameters.protocol>> --groupname="<<parameters.groupname>>"
+                        node test/functional/runTests.cjs --selenium=remote --reporters=junit --app=remote --browsers=<<parameters.browser>> --protocol=<<parameters.protocol>> --groupname="<<parameters.groupname>>"
+    build_samples:
+        # parameters:
+        #     samples:
+        #         default: "network-interceptor"
+        #         type: string
+        steps:
+            -   run:
+                    name: Install dependencies and build samples
+                    command: |
+                        for sample in "${SAMPLES}"; do
+                            echo "Build sample $sample"
+                            cd samples/$sample
+                            npm install
+                            npm run build
+                        done
+
 jobs:
     build-and-unit-test:
+        environment:
+            SAMPLES: "network-interceptor"
         executor: dashjs-executor
         steps:
             - checkout
             - dependencies_setup
             - build_unit_test_steps
+            - build_samples
 
     merge-build-and-unit-test:
         executor: dashjs-executor
         steps:
             - checkout
             -   run:
-                    name: Merge into development virtually
+                    name: Merge into v5.0.0 virtually
                     command: |
                         git config --global user.email "[email protected]"
                         git config --global user.name "CircleCI"
-                        git checkout development
+                        git checkout v5.0.0
                         git merge --no-edit --no-ff $CIRCLE_BRANCH
             - dependencies_setup
             - build_unit_test_steps
@@ -126,14 +145,14 @@ jobs:
             - functional_precondition
             - checkout
             -   run:
-                    name: Virtual merge into development branch
+                    name: Virtual merge into v5.0.0 branch
                     command: |
-                        if [ "${CIRCLE_BRANCH}" = "development" ]; then
-                            echo "On development branch already, no merge needed"
+                        if [ "${CIRCLE_BRANCH}" = "v5.0.0" ]; then
+                            echo "On v5.0.0 branch already, no merge needed"
                         else
                             git config --global user.email "[email protected]"
                             git config --global user.name "CircleCI"
-                            git checkout development
+                            git checkout v5.0.0
                             git merge --no-edit --no-ff $CIRCLE_BRANCH
                         fi
             - dependencies_setup
@@ -142,13 +161,13 @@ jobs:
             -   run:
                     name: Run functional tests for one vector (chrome / https)
                     command:
-                        node test/functional/runTests.js --selenium=remote --reporters=junit --debug=true --app=remote --browsers=chrome --protocol=https --source=./test/functional/config/singleVector.json
+                        node test/functional/runTests.cjs --selenium=remote --reporters=junit --debug=true --app=remote --browsers=chrome --protocol=https --source=./test/functional/config/singleVector.json
             -   run:
                     name: Run functional tests for smoke vectors (chrome / https)
                     command:
-                        node test/functional/runTests.js --selenium=remote --reporters=junit --debug=true --app=remote --browsers=chrome --protocol=https --source=./test/functional/config/smokeVectors.json
+                        node test/functional/runTests.cjs --selenium=remote --reporters=junit --debug=true --app=remote --browsers=chrome --protocol=https --source=./test/functional/config/smokeVectors.json
             - process_test_results
-    
+
     functional-tests-VOD_LIVE:
         executor: dashjs-executor
         steps:

diff --git a/.eslintrc b/.eslintrc
@@ -22,6 +22,13 @@
         "strict": 0,
         "no-loop-func": 0,
         "no-multi-spaces": "error",
+        "keyword-spacing": [
+            "error",
+            {
+                "before": true,
+                "after": true
+            }
+        ],
         "quotes": [
             "error",
             "single",

diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
@@ -22,11 +22,11 @@ jobs:
         runs-on: ubuntu-latest
 
         steps:
-            -   uses: actions/checkout@v2
+            -   uses: actions/checkout@v4
             -   name: Use Node.js
-                uses: actions/setup-node@v2
+                uses: actions/setup-node@v4
                 with:
-                    node-version: "14.x"
+                    node-version: "18.x"
             -   name: Install dependencies
                 run: npm install
             -   name: Run build

diff --git a/.github/workflows/deploy_latest.yml b/.github/workflows/deploy_latest.yml
@@ -1,4 +1,4 @@
-name: deploy
+name: deploy_latest
 
 on:
     push:

diff --git a/.github/workflows/deploy_nightly.yml b/.github/workflows/deploy_nightly.yml
@@ -1,4 +1,4 @@
-name: deploy
+name: deploy_nightly
 
 on:
     push:

diff --git a/.github/workflows/deploy_v4_latest.yml b/.github/workflows/deploy_v4_latest.yml
@@ -0,0 +1,18 @@
+name: deploy_v4_latest
+
+on:
+    push:
+        branches:
+            - 'v4_main'
+
+jobs:
+    deploy_staging:
+        if: github.repository == 'Dash-Industry-Forum/dash.js'
+        uses: ./.github/workflows/deploy.yml
+        with:
+            envname: v4_latest
+            deploy_path: '/377335/dash.js'
+        secrets:
+            host: ${{secrets.HOST}}
+            user: ${{secrets.USER}}
+            private_key: ${{secrets.PRIVATE_KEY}}
diff --git a/.github/workflows/deploy_v4_nightly.yml b/.github/workflows/deploy_v4_nightly.yml
@@ -0,0 +1,18 @@
+name: deploy_v4_nightly
+
+on:
+    push:
+        branches:
+            - 'v4_development'
+
+jobs:
+    deploy_staging:
+        if: github.repository == 'Dash-Industry-Forum/dash.js'
+        uses: ./.github/workflows/deploy.yml
+        with:
+            envname: v4_nightly
+            deploy_path: '/377335/dash.js'
+        secrets:
+            host: ${{secrets.HOST}}
+            user: ${{secrets.USER}}
+            private_key: ${{secrets.PRIVATE_KEY}}
diff --git a/build/helpers/LLsegment.js → build/helpers/LLsegment.cjs b/build/helpers/LLsegment.js → build/helpers/LLsegment.cjs
diff --git a/build/karma.conf.js → build/karma.conf.cjs b/build/karma.conf.js → build/karma.conf.cjs
@@ -1,5 +1,5 @@
 
-const llSegmentMiddleware = require('./helpers/LLsegment.js');
+const llSegmentMiddleware = require('./helpers/LLsegment.cjs');
 
 function CustomMiddlewareFactory(/*config*/) {
   return function (request, response, next ) {

diff --git a/build/webpack.base.js → build/webpack.base.cjs b/build/webpack.base.js → build/webpack.base.cjs
diff --git a/build/webpack.dev.js → build/webpack.dev.cjs b/build/webpack.dev.js → build/webpack.dev.cjs
@@ -1,5 +1,5 @@
 const { merge } = require('webpack-merge');
-const common = require('./webpack.base.js').config;
+const common = require('./webpack.base.cjs').config;
 const path = require('path');
 
 const config = merge(common, {

diff --git a/build/webpack.prod.js → build/webpack.prod.cjs b/build/webpack.prod.js → build/webpack.prod.cjs
@@ -1,6 +1,6 @@
 const { merge } = require('webpack-merge');
 const ESLintPlugin = require('eslint-webpack-plugin');
-const common = require('./webpack.base.js').config;
+const common = require('./webpack.base.cjs').config;
 
 const entries = {
     'dash.all': './index.js',

diff --git a/contrib/akamai/controlbar/ControlBar.js b/contrib/akamai/controlbar/ControlBar.js
@@ -529,13 +529,13 @@ var ControlBar = function (dashjsMediaPlayer, displayUTCTimeCodes) {
             destroyMenu(bitrateListMenu, bitrateListBtn, menuHandlersList.bitrate);
             bitrateListMenu = null;
             var availableBitrates = { menuType: 'bitrate' };
-            availableBitrates.audio = self.player.getBitrateInfoListFor && self.player.getBitrateInfoListFor('audio') || [];
-            availableBitrates.video = self.player.getBitrateInfoListFor && self.player.getBitrateInfoListFor('video') || [];
-            availableBitrates.images = self.player.getBitrateInfoListFor && self.player.getBitrateInfoListFor('image') || [];
+            availableBitrates.audio = self.player.getRepresentationsByType && self.player.getRepresentationsByType('audio') || [];
+            availableBitrates.video = self.player.getRepresentationsByType && self.player.getRepresentationsByType('video') || [];
+            availableBitrates.images = self.player.getRepresentationsByType && self.player.getRepresentationsByType('image') || [];
 
             if (availableBitrates.audio.length >= 1 || availableBitrates.video.length >= 1 || availableBitrates.images.length >= 1) {
                 contentFunc = function (element, index) {
-                    var result = isNaN(index) ? ' Auto Switch' : Math.floor(element.bitrate / 1000) + ' kbps';
+                    var result = isNaN(index) ? ' Auto Switch' : Math.floor(element.bitrateInKbit) + ' kbps';
                     result += element && element.width && element.height ? ' (' + element.width + 'x' + element.height + ')' : '';
                     return result;
                 };
@@ -576,7 +576,7 @@ var ControlBar = function (dashjsMediaPlayer, displayUTCTimeCodes) {
                     }
 
                     if (element.roles[0]) {
-                        info += '- Role: ' + element.roles[0] + ' ';
+                        info += '- Role: ' + element.roles[0].value + ' ';
                     }
 
                     if (element.codec) {
@@ -912,14 +912,14 @@ var ControlBar = function (dashjsMediaPlayer, displayUTCTimeCodes) {
                             if (item.index > 0) {
                                 cfg.streaming.abr.autoSwitchBitrate[item.mediaType] = false;
                                 self.player.updateSettings(cfg);
-                                self.player.setQualityFor(item.mediaType, item.index - 1, forceQuality);
+                                self.player.setRepresentationForTypeByIndex(item.mediaType, item.index - 1, forceQuality);
                             } else {
                                 cfg.streaming.abr.autoSwitchBitrate[item.mediaType] = true;
                                 self.player.updateSettings(cfg);
                             }
                             break;
                         case 'image-bitrate-list':
-                            player.setQualityFor(item.mediaType, item.index);
+                            player.setRepresentationForTypeByIndex(item.mediaType, item.index);
                             break;
                         case 'caption-list':
                             self.player.setTextTrack(item.index - 1);

diff --git a/deploy.js b/deploy.js
diff --git a/externals/BigInteger.js b/externals/BigInteger.js
diff --git a/externals/base64.js b/externals/base64.js
@@ -160,3 +160,5 @@ if (typeof exports !== 'undefined') {
     exports.encode = BASE64.encode;
     exports.encodeASCII = BASE64.encodeASCII;
 }
+
+export default BASE64