Add "getSourceConfig" option as an alternative to "configUrl" to get …

…sourceConfig
rudderlabs · Feb 22, 2021 · 49211f7 · 49211f7
1 parent f071344
commit 49211f7
Showing 1 changed file with 30 additions and 4 deletions.
diff --git a/analytics.js b/analytics.js
@@ -137,7 +137,9 @@ class Analytics {
   processResponse(status, response) {
     try {
       logger.debug(`===in process response=== ${status}`);
-      response = JSON.parse(response);
+      if (typeof response === 'string') {
+        response = JSON.parse(response);
+      }
       if (
         response.source.useAutoTracking &&
         !this.autoTrackHandlersRegistered
@@ -978,14 +980,38 @@ class Analytics {
         );
       }
     }
-    try {
-      getJSONTrimmed(this, configUrl, writeKey, this.processResponse);
-    } catch (error) {
+
+    function errorHandler(error) {
       handleError(error);
       if (this.autoTrackFeatureEnabled && !this.autoTrackHandlersRegistered) {
         addDomEventHandlers(this);
       }
     }
+
+    if (options && options.getSourceConfig) {
+      if (typeof options.getSourceConfig !== "function") {
+        handleError('option "getSourceConfig" must be a function');
+      } else {
+        const res = options.getSourceConfig();
+
+        if (res instanceof Promise) {
+          res
+            .then(res => this.processResponse(200, res))
+            .catch(errorHandler)
+        } else {
+          this.processResponse(200, res);
+        }
+
+        processDataInAnalyticsArray(this);
+      }
+      return;
+    }
+
+    try {
+      getJSONTrimmed(this, configUrl, writeKey, this.processResponse);
+    } catch (error) {
+      errorHandler(error)
+    }
     processDataInAnalyticsArray(this);
   }