Solve issue 75 by adding more information to some exceptions.

djob195 · May 5, 2017 · 10a0b37 · 10a0b37
1 parent 49ee0f1
commit 10a0b37
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Microsoft.ApplicationInsights.AspNetCore/Extensions/HttpRequestExtensions.cs b/src/Microsoft.ApplicationInsights.AspNetCore/Extensions/HttpRequestExtensions.cs
@@ -23,12 +23,12 @@ public static Uri GetUri(this HttpRequest request)
 
             if (true == string.IsNullOrWhiteSpace(request.Scheme))
             {
-                throw new ArgumentException("Http request Scheme is not specified");
+                throw new ArgumentException("Http request Scheme is not specified. This generally occurs because the WebModel for the Request was not properly initialize");
             }
 
             if (false == request.Host.HasValue)
             {
-                throw new ArgumentException("Http request Host is not specified");
+                throw new ArgumentException("Http request Host is not specified. this generally occurs when Host header is not properly declare");
             }
 
             var builder = new StringBuilder();

diff --git a/...icationInsights.AspNetCore.Tests/Extensibility/Implementation/Tracing/EventSourceTests.cs b/...icationInsights.AspNetCore.Tests/Extensibility/Implementation/Tracing/EventSourceTests.cs
@@ -94,7 +94,7 @@ private static object GenerateArgument(ParameterInfo parameter)
                 return Activator.CreateInstance(parameter.ParameterType);
             }
 
-            throw new NotSupportedException("Complex types are not suppored");
+            throw new NotSupportedException("The type" + parameterType.GetTypeInfo().Name + "is not supported");
         }
 
         /// <summary>