fix TS problem and handle undefined credspecs

tfenster · Feb 4, 2019 · 8b52dd2 · 8b52dd2
1 parent 55314b9
commit 8b52dd2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/gui/ClientApp/package.json b/gui/ClientApp/package.json
@@ -32,8 +32,8 @@
     "core-js": "^2.5.4",
     "jquery": "3.3.1",
     "popper.js": "^1.14.3",
-    "rxjs": "^6.0.0",
-    "rxjs-compat": "^6.3.3",
+    "rxjs": "6.3.3",
+    "rxjs-compat": "6.3.3",
     "zone.js": "^0.8.26"
   },
   "devDependencies": {

diff --git a/gui/ClientApp/src/app/api.service.ts b/gui/ClientApp/src/app/api.service.ts
@@ -97,6 +97,9 @@ export class ApiService {
       env.push("password=" + guiDef.base.password);
       guiDef.base.password = ""; // forget password for storage in container labels
     }
+    if (guiDef.adv.credspec != undefined && guiDef.adv.credspec != "") {
+      guiDef.adv.credspec = "credentialspec=file://" + guiDef.adv.credspec;
+    }
 
     env.push("ExitOnError=N");
 
@@ -111,7 +114,7 @@ export class ApiService {
       TestToolkit: guiDef.adv.testToolkit,
       Network: guiDef.adv.network,
       License: guiDef.adv.license,
-      SecurityOpt: "credentialspec=file://" + guiDef.adv.credspec,
+      SecurityOpt: guiDef.adv.credspec,
       GuiDef: JSON.stringify(guiDef)
     };
     const headers = new HttpHeaders({