Fixed an issue where the update settings could be called multiple times.

Author: niemyjski

src/Exceptionless/ExceptionlessClient.cs

@@ -49,7 +49,7 @@ public ExceptionlessClient(ExceptionlessConfiguration configuration) {
 
             _submissionClient = new Lazy<ISubmissionClient>(() => Configuration.Resolver.GetSubmissionClient());
             _lastReferenceIdManager = new Lazy<ILastReferenceIdManager>(() => Configuration.Resolver.GetLastReferenceIdManager());
-            _updateSettingsTimer = new Timer(state => SettingsManager.UpdateSettings(Configuration), null, GetInitialSettingsDelay(), Configuration.UpdateSettingsWhenIdleInterval ?? TimeSpan.FromMilliseconds(-1));
+            _updateSettingsTimer = new Timer(OnUpdateSettings, null, GetInitialSettingsDelay(), Configuration.UpdateSettingsWhenIdleInterval ?? TimeSpan.FromMilliseconds(-1));
         }
 
         private TimeSpan GetInitialSettingsDelay() {
@@ -66,6 +66,16 @@ private void OnConfigurationChanged(object sender, EventArgs e) {
             _updateSettingsTimer.Change(!_queue.IsValueCreated ? GetInitialSettingsDelay() : interval, interval);
         }
 
+        private bool _isUpdatingSettings;
+        private void OnUpdateSettings(object state) {
+            if (_isUpdatingSettings || !Configuration.IsValid)
+                return;
+
+            _isUpdatingSettings = true;
+            SettingsManager.UpdateSettings(Configuration);
+            _isUpdatingSettings = false;
+        }
+
         public ExceptionlessConfiguration Configuration { get; private set; }
 
         /// <summary>