replacing hardcoded local db info with env variables

Author: adl-trey

cts/.env.example

@@ -1,5 +1,10 @@
 HOSTNAME=localhost
 
+DB_HOST=rdbms
+DB_NAME=catapult_player
+DB_USERNAME=catapult
+DB_PASSWORD=quartz
+
 PLAYER_ROOT_PATH=/player
 PLAYER_BASE_URL=http://player:3398/player
 PLAYER_CONTENT_URL=http://localhost/player/content

cts/docker-compose.yml

@@ -60,9 +60,13 @@ services:
       - ./seeds:/usr/src/app/seeds:ro
     environment:
       - HOSTNAME
-      - DATABASE_USER=catapult
-      - DATABASE_USER_PASSWORD=quartz
-      - DATABASE_NAME=catapult_cts
+      - DB_HOST
+      - DB_NAME
+      - DB_USERNAME
+      - DB_PASSWOD
+      # - DATABASE_USER=catapult
+      # - DATABASE_USER_PASSWORD=quartz
+      # - DATABASE_NAME=catapult_cts
       - PLAYER_BASE_URL
       - PLAYER_KEY
       - PLAYER_SECRET

cts/service/index.js

@@ -65,7 +65,8 @@ const provision = async () => {
             }
         };
 
-    await waitPort({host: "rdbms", port: 3306});
+    const DB_HOST = (process.env.DB_HOST || "rdbms");
+    await waitPort({host: DB_HOST, port: 3306});
 
     const db = await require("./lib/db")();
 

cts/service/knexfile.js

@@ -13,24 +13,33 @@
     See the License for the specific language governing permissions and
     limitations under the License.
 */
+const DB_HOST = (process.env.DB_HOST || "rdbms");
+const DB_NAME = (process.env.DB_NAME || "catapult_player");
+const DB_USERNAME = (process.env.DB_USERNAME || "catapult");
+const DB_PASSWORD = (process.env.DB_PASSWORD || "quartz");
+
 const Hoek = require("@hapi/hoek"),
     waitPort = require("wait-port"),
     {
-        MYSQL_HOST: HOST = "rdbms",
+        // MYSQL_HOST: HOST = "rdbms",
         MYSQL_HOST_FILE: HOST_FILE,
-        DATABASE_USER: USER = "catapult",
+        // DATABASE_USER: USER = "catapult",
         DATABASE_USER_FILE: USER_FILE,
-        DATABASE_USER_PASSWORD: PASSWORD = "quartz",
+        // DATABASE_USER_PASSWORD: DB_PASSWORD,
         DATABASE_USER_PASSWORD_FILE: PASSWORD_FILE,
-        DATABASE_NAME: DB = "catapult_player",
+        // DATABASE_NAME: DB = "catapult_player",
         DATABASE_NAME_FILE: DB_FILE,
     } = process.env;
 
 module.exports = async () => {
-    const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : HOST,
-        user = USER_FILE ? fs.readFileSync(USER_FILE) : USER,
-        password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : PASSWORD,
-        database = DB_FILE ? fs.readFileSync(DB_FILE) : DB;
+    // const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : HOST,
+    //     user = USER_FILE ? fs.readFileSync(USER_FILE) : USER,
+    //     password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : PASSWORD,
+    //     database = DB_FILE ? fs.readFileSync(DB_FILE) : DB;
+    const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : DB_HOST,
+        user = USER_FILE ? fs.readFileSync(USER_FILE) : DB_USERNAME,
+        password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : DB_PASSWORD,
+        database = DB_FILE ? fs.readFileSync(DB_FILE) : DB_NAME;
 
     await waitPort({host, port: 3306});
 

player/.env.example

@@ -1,6 +1,11 @@
 HOSTNAME=localhost
 HOST_PORT=63398
 
+DB_HOST=rdbms
+DB_NAME=catapult_player
+DB_USERNAME=catapult
+DB_PASSWORD=quartz
+
 CONTENT_URL=http://localhost:63398/content
 
 API_KEY="some API access key"

player/docker-compose.yml

@@ -36,9 +36,13 @@ services:
       - LRS_USERNAME
       - LRS_PASSWORD
       - LRS_XAPI_VERSIONs
-      - DATABASE_USER=catapult
-      - DATABASE_USER_PASSWORD=quartz
-      - DATABASE_NAME=catapult_player
+      - DB_HOST
+      - DB_NAME
+      - DB_USERNAME
+      - DB_PASSWOD
+      # - DATABASE_USER=catapult
+      # - DATABASE_USER_PASSWORD=quartz
+      # - DATABASE_NAME=catapult_player
       - FIRST_TENANT_NAME
       - PLAYER_API_ROOT=${PLAYER_ROOT_PATH}
       - PLAYER_STANDALONE_LAUNCH_URL_BASE

player/service/index.js

@@ -65,7 +65,8 @@ const provision = async () => {
             }
         };
 
-    await waitPort({host: "rdbms", port: 3306});
+    const DB_HOST = (process.env.DB_HOST || "rdbms");
+    await waitPort({host: DB_HOST, port: 3306});
 
     const db = await require("./lib/db")();
 

player/service/knexfile.js

@@ -13,24 +13,35 @@
     See the License for the specific language governing permissions and
     limitations under the License.
 */
+
+const DB_HOST = (process.env.DB_HOST || "rdbms");
+const DB_NAME = (process.env.DB_NAME || "catapult_player");
+const DB_USERNAME = (process.env.DB_USERNAME || "catapult");
+const DB_PASSWORD = (process.env.DB_PASSWORD || "quartz");
+
 const Hoek = require("@hapi/hoek"),
     waitPort = require("wait-port"),
     {
-        MYSQL_HOST: HOST = "rdbms",
+        // MYSQL_HOST: HOST = "rdbms",
         MYSQL_HOST_FILE: HOST_FILE,
-        DATABASE_USER: USER = "catapult",
+        // DATABASE_USER: USER = "catapult",
         DATABASE_USER_FILE: USER_FILE,
-        DATABASE_USER_PASSWORD: PASSWORD = "quartz",
+        // DATABASE_USER_PASSWORD: DB_PASSWORD,
         DATABASE_USER_PASSWORD_FILE: PASSWORD_FILE,
-        DATABASE_NAME: DB = "catapult_player",
+        // DATABASE_NAME: DB = "catapult_player",
         DATABASE_NAME_FILE: DB_FILE,
     } = process.env;
 
 module.exports = async () => {
-    const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : HOST,
-        user = USER_FILE ? fs.readFileSync(USER_FILE) : USER,
-        password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : PASSWORD,
-        database = DB_FILE ? fs.readFileSync(DB_FILE) : DB;
+    // const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : HOST,
+    //     user = USER_FILE ? fs.readFileSync(USER_FILE) : USER,
+    //     password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : PASSWORD,
+    //     database = DB_FILE ? fs.readFileSync(DB_FILE) : DB;
+    const host = HOST_FILE ? fs.readFileSync(HOST_FILE) : DB_HOST,
+        user = USER_FILE ? fs.readFileSync(USER_FILE) : DB_USERNAME,
+        password = PASSWORD_FILE ? fs.readFileSync(PASSWORD_FILE) : DB_PASSWORD,
+        database = DB_FILE ? fs.readFileSync(DB_FILE) : DB_NAME;
+    
 
     await waitPort({host, port: 3306});