Fix config file parsing regression
authorMichal Klobukowski <michal.klobukowski@contractors.roche.com>
Tue, 14 Aug 2018 09:19:29 +0000 (11:19 +0200)
committerMichal Klobukowski <michal.klobukowski@contractors.roche.com>
Tue, 14 Aug 2018 09:19:29 +0000 (11:19 +0200)
Feature #14024

Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski <michal.klobukowski@contractors.roche.com>

src/common/config.ts

index 492328d986e55e3d6d2101d96f1d82f095a5690d..759a20158f9f8cafac459cd4ee2213f36c00890d 100644 (file)
@@ -13,21 +13,25 @@ export interface Config {
 
 export const fetchConfig = () => {
     return Axios
-        .get<Config>(CONFIG_URL + "?nocache=" + (new Date()).getTime())
+        .get<ConfigJSON>(CONFIG_URL + "?nocache=" + (new Date()).getTime())
         .then(response => response.data)
         .catch(() => Promise.resolve(getDefaultConfig()))
         .then(mapConfig);
 };
 
-const mapConfig = (config: Config): Config => ({
-    ...config,
-    apiHost: addProtocol(config.apiHost),
-    keepWebHost: addProtocol(config.keepWebHost)
+interface ConfigJSON {
+    API_HOST: string;
+    KEEP_WEB_HOST: string;
+}
+
+const mapConfig = (config: ConfigJSON): Config => ({
+    apiHost: addProtocol(config.API_HOST),
+    keepWebHost: addProtocol(config.KEEP_WEB_HOST)
 });
 
-const getDefaultConfig = (): Config => ({
-    apiHost: process.env.REACT_APP_ARVADOS_API_HOST || "",
-    keepWebHost: process.env.REACT_APP_ARVADOS_KEEP_WEB_HOST || ""
+const getDefaultConfig = (): ConfigJSON => ({
+    API_HOST: process.env.REACT_APP_ARVADOS_API_HOST || "",
+    KEEP_WEB_HOST: process.env.REACT_APP_ARVADOS_KEEP_WEB_HOST || ""
 });
 
 const addProtocol = (url: string) => `${window.location.protocol}//${url}`;