Added more exception handling around HttpContext.Request, since resov…

…ling it might fail. Hopefully fixes getsentry#64.
adbuker · Dec 5, 2014 · 7c98a6a · 7c98a6a
1 parent 06fc749
commit 7c98a6a
Showing 1 changed file with 32 additions and 2 deletions.
diff --git a/src/app/SharpRaven/Data/SentryRequest.cs b/src/app/SharpRaven/Data/SentryRequest.cs
@@ -166,13 +166,43 @@ public SentryUser GetUser()
             if (!HasHttpContext)
                 return null;
 
-            return new SentryUser(HttpContext.User as IPrincipal)
+            return new SentryUser(GetPrincipal())
             {
-                IpAddress = HttpContext.Request.UserHostAddress
+                IpAddress = GetIpAddress()
             };
         }
 
 
+        private static IPrincipal GetPrincipal()
+        {
+            try
+            {
+                return HttpContext.User as IPrincipal;
+            }
+            catch (Exception exception)
+            {
+                Console.WriteLine(exception);
+            }
+
+            return null;
+        }
+
+
+        private static dynamic GetIpAddress()
+        {
+            try
+            {
+                return HttpContext.Request.UserHostAddress;
+            }
+            catch (Exception exception)
+            {
+                Console.WriteLine(exception);
+            }
+
+            return null;
+        }
+
+
         private static void GetHttpContext()
         {
             if (HasHttpContext)