renaming launch url base to make use case more obvious

Author: adl-trey

cts/.env.example

@@ -3,7 +3,7 @@ HOSTNAME=localhost
 PLAYER_ROOT_PATH=/player
 PLAYER_BASE_URL=http://player:3398/player
 PLAYER_CONTENT_URL=http://localhost/player/content
-PLAYER_LAUNCH_URL_BASE=http://localhost/player
+# PLAYER_STANDALONE_LAUNCH_URL_BASE=http://localhost/player
 PLAYER_KEY=cts
 PLAYER_SECRET=player-secret
 PLAYER_TOKEN_SECRET=player-token-secret

cts/docker-compose.yml

@@ -104,7 +104,7 @@ services:
       - API_KEY=${PLAYER_KEY}
       - API_SECRET=${PLAYER_SECRET}
       - PLAYER_API_ROOT=${PLAYER_ROOT_PATH}
-      - PLAYER_LAUNCH_URL_BASE
+      - PLAYER_STANDALONE_LAUNCH_URL_BASE
     networks:
       - public
 

cts/service/plugins/routes/v1/sessions.js

@@ -159,6 +159,8 @@ module.exports = {
                             playerFetch = playerAuLaunchUrlParsed.searchParams.get("fetch");
                         let sessionId;
 
+                        console.log("PLAYER FETCH:", playerFetch);
+
                         try {
                             const sessionInsert = await db.insert(
                                 {
@@ -362,6 +364,8 @@ module.exports = {
                             let fetchResponse,
                                 fetchResponseBody;
 
+                            console.log("POSTING FETCH:", session.playerFetch);
+
                             try {
                                 fetchResponse = await Wreck.request(
                                     "POST",
@@ -370,6 +374,7 @@ module.exports = {
                                 fetchResponseBody = await Wreck.read(fetchResponse, {json: true});
                             }
                             catch (ex) {
+                                console.error(ex);
                                 throw Boom.internal(new Error(`Failed to request fetch url from player: ${ex}`));
                             }
 

player/.env.example

@@ -14,4 +14,4 @@ LRS_XAPI_VERSION=1.0.3
 
 ## Default is empty, add if necessary for nginx config
 PLAYER_API_ROOT=
-PLAYER_LAUNCH_URL_BASE=
+PLAYER_STANDALONE_LAUNCH_URL_BASE=

player/docker-compose.yml

@@ -41,7 +41,7 @@ services:
       - DATABASE_NAME=catapult_player
       - FIRST_TENANT_NAME
       - PLAYER_API_ROOT=${PLAYER_ROOT_PATH}
-      - PLAYER_LAUNCH_URL_BASE
+      - PLAYER_STANDALONE_LAUNCH_URL_BASE
       - HOST_PORT
   rdbms:
     image: mysql:8.0.31

player/service/plugins/routes/v1/courses.js

@@ -744,7 +744,7 @@ module.exports = {
                                 }
                             };
 
-                        let launchURLBase = (process.env.PLAYER_LAUNCH_URL_BASE || `${protocol}//${req.url.host}${rootPath}`);
+                        let launchURLBase = (process.env.PLAYER_STANDALONE_LAUNCH_URL_BASE || `${protocol}//${req.url.host}${rootPath}`);
                         let endpoint = `${launchURLBase}/lrs`;
 
                         // //Debug messages for troubleshooting host and path issuse - MB
@@ -824,7 +824,7 @@ module.exports = {
 
                         }
                         catch (ex) {
-                             
+                            console.error(ex);
                             throw Boom.internal(new Error(`Failed request to set LMS.LaunchData state document: ${ex}`));
                         }
 
@@ -918,6 +918,7 @@ module.exports = {
                             session.id = sessionInsertResult[0];
                         }
                         catch (ex) {
+                            console.error(ex);
                             throw Boom.internal(new Error(`Failed to insert session: ${ex}`));
                         }