Fixes #238: Add ability to exclude Modules from getting sent with error reports (#302)

* Fixes #238: Add ability to exclude Modules from getting sent with error reports

* Fixed test

Author: niemyjski

src/Exceptionless/Configuration/ExceptionlessConfiguration.cs

@@ -54,6 +54,7 @@ public ExceptionlessConfiguration(IDependencyResolver resolver) {
             DefaultData = new DataDictionary();
             Settings = new SettingsDictionary();
             IncludePrivateInformation = true;
+            IncludeModules = true;
 
             _resolver = resolver;
 
@@ -267,6 +268,11 @@ public bool IncludePrivateInformation {
         /// NOTE: DataExclusions are applied to all Query String keys when enabled.
         /// </summary>
         public bool IncludeQueryString { get; set; }
+        
+        /// <summary>
+        /// Gets or sets a value indicating whether to include module information.
+        /// </summary>
+        public bool IncludeModules { get; set; }
 
         /// <summary>
         /// Gets or sets a value indicating whether to automatically send session start, session heartbeats and session end events.

src/Exceptionless/Extensions/ToErrorModelExtensions.cs

@@ -48,7 +48,7 @@ private static Error ToErrorModelInternal(Exception exception, ExceptionlessClie
                 Type = typeName
             };
 
-            if (!isInner)
+            if (!isInner && client.Configuration.IncludeModules)
                 error.Modules = GetLoadedModules(log);
 
             error.PopulateStackTrace(error, exception, log);

src/Exceptionless/Extensions/ToSimpleErrorModelExtensions.cs

@@ -43,7 +43,7 @@ private static SimpleError ToSimpleErrorModelInternal(this Exception exception,
                 StackTrace = exception.Demystify().StackTrace
             };
 
-            if (!isInner)
+            if (!isInner && client.Configuration.IncludeModules)
                 error.Modules = ToErrorModelExtensions.GetLoadedModules(log);
 
             var exclusions = _exceptionExclusions.Union(client.Configuration.DataExclusions).ToList();

src/Exceptionless/Storage/InMemoryObjectStorage.cs

@@ -53,7 +53,7 @@ public bool SaveObject<T>(string path, T value) where T : class {
                 }, (object)value);
 
                 if (_storage.Count > MaxObjects)
-                    _storage.Remove(_storage.OrderByDescending(kvp => kvp.Value.Item1.Created).First().Key);
+                    _storage.Remove(_storage.OrderBy(kvp => kvp.Value.Item1.Created).First().Key);
             }
 
             return true;

test/Exceptionless.Tests/Configuration/ConfigurationTests.cs

@@ -33,6 +33,7 @@ public void CanConfigureApiKeyFromClientConstructor() {
             Assert.True(client.Configuration.IncludeUserName);
             Assert.True(client.Configuration.IncludeMachineName);
             Assert.True(client.Configuration.IncludeIpAddress);
+            Assert.True(client.Configuration.IncludeHeaders);
             Assert.True(client.Configuration.IncludeCookies);
             Assert.True(client.Configuration.IncludePostData);
             Assert.True(client.Configuration.IncludeQueryString);
@@ -42,6 +43,7 @@ public void CanConfigureApiKeyFromClientConstructor() {
             Assert.False(client.Configuration.IncludeUserName);
             Assert.False(client.Configuration.IncludeMachineName);
             Assert.False(client.Configuration.IncludeIpAddress);
+            Assert.False(client.Configuration.IncludeHeaders);
             Assert.False(client.Configuration.IncludeCookies);
             Assert.False(client.Configuration.IncludePostData);
             Assert.False(client.Configuration.IncludeQueryString);
@@ -51,6 +53,7 @@ public void CanConfigureApiKeyFromClientConstructor() {
             Assert.False(client.Configuration.IncludeUserName);
             Assert.True(client.Configuration.IncludeMachineName);
             Assert.False(client.Configuration.IncludeIpAddress);
+            Assert.False(client.Configuration.IncludeHeaders);
             Assert.False(client.Configuration.IncludeCookies);
             Assert.False(client.Configuration.IncludePostData);
             Assert.False(client.Configuration.IncludeQueryString);