Fixed an issue where the initial delay was being ignored

Author: niemyjski

dist/exceptionless.js

@@ -32,9 +32,9 @@ export class ExceptionlessClient {
       this.config = new Configuration({ apiKey: <string>settingsOrApiKey, serverUrl: serverUrl });
     }
 
+    this.updateSettingsTimer(5000);
     this.config.onChanged((config) => this.updateSettingsTimer(this._timeoutId > 0 ? 5000 : 0));
     this.config.queue.onEventsPosted((events, response) =>  this.updateSettingsTimer());
-    this.updateSettingsTimer(5000);
   }
 
   public createException(exception: Error): EventBuilder {
@@ -217,7 +217,7 @@ export class ExceptionlessClient {
   }
 
   private updateSettingsTimer(initialDelay?: number) {
-    this.config.log.info('Updating settings timer');
+    this.config.log.info(`Updating settings timer with delay: ${initialDelay}`);
 
     this._timeoutId = clearTimeout(this._timeoutId);
     this._timeoutId = clearInterval(this._intervalId);

dist/exceptionless.js.map

@@ -230,7 +230,11 @@ export class Configuration implements IConfigurationSettings {
    * @param value
    */
   public set updateSettingsWhenIdleInterval(value: number) {
-    if (typeof value !== 'number' || value <= 0) {
+    if (typeof value !== 'number') {
+      return;
+    }
+
+    if (value <= 0) {
       value = -1;
     } else if (value > 0 && value < 15000) {
       value = 15000;

dist/exceptionless.min.js

@@ -91,7 +91,7 @@ export class SettingsManager {
 
       config.storage.settings.save(newSettings);
 
-      config.log.info('Updated settings');
+      config.log.info(`Updated settings: v${newSettings.version}`);
       this.changed(config);
     });
   }