Made checkedForHttpContextProperty a field and did a R# cleanup.

src/app/SharpRaven/Data/SentryRequestFactory.cs

@@ -31,12 +31,12 @@
 using System;
 using System.Collections.Generic;
 using System.Collections.Specialized;
+using System.IO;
 using System.Linq;
 using System.Reflection;
+using System.Text;
 
 using Newtonsoft.Json;
-using System.IO;
-using System.Text;
 
 namespace SharpRaven.Data
 {
@@ -46,6 +46,22 @@ namespace SharpRaven.Data
     /// </summary>
     public class SentryRequestFactory : ISentryRequestFactory
     {
+        private static bool checkedForHttpContextProperty;
+
+        /// <summary>
+        /// Gets or sets the CurrentHttpContextProperty
+        /// </summary>
+        /// <value>
+        /// The current httpcontext property
+        /// </value>
+        internal static dynamic CurrentHttpContextProperty { get; set; }
+
+        [JsonIgnore]
+        internal static bool HasCurrentHttpContextProperty
+        {
+            get { return CurrentHttpContextProperty != null; }
+        }
+
         [JsonIgnore]
         internal static bool HasHttpContext
         {
@@ -63,7 +79,7 @@ internal static dynamic HttpContext
             get
             {
                 TryGetHttpContextPropertyFromAppDomain();
-    			
+
                 // [Meilu] If the currentHttpcontext property is not available we couldnt retrieve it, dont continue
                 if (!HasCurrentHttpContextProperty)
                     return null;
@@ -80,22 +96,6 @@ internal static dynamic HttpContext
             }
         }
 
-        /// <summary>
-        /// Gets or sets the CurrentHttpContextProperty
-        /// </summary>
-        /// <value>
-        /// The current httpcontext property
-        /// </value>
-        internal static dynamic CurrentHttpContextProperty { get; set; }
-
-        [JsonIgnore]
-        internal static bool HasCurrentHttpContextProperty
-        {
-            get { return CurrentHttpContextProperty != null; }
-        }
-
-        [JsonIgnore]
-        internal static bool CheckedForHttpContextProperty { get; set; }
 
         /// <summary>
         /// Creates a new instance of <see cref="SentryRequest"/>
@@ -107,7 +107,7 @@ public ISentryRequest Create()
             if (!HasHttpContext || HttpContext.Request == null)
                 return OnCreate(null);
 
-            var request = new SentryRequest()
+            var request = new SentryRequest
             {
                 Url = HttpContext.Request.Url.ToString(),
                 Method = HttpContext.Request.HttpMethod,
@@ -147,19 +147,16 @@ private static object BodyConvert()
                 {
                     return Convert(x => x.Request.Form);
                 }
-                else
-                {
-                    string body = String.Empty;
+                var body = string.Empty;
 
-                    using (MemoryStream stream = new MemoryStream())
-                    {
-                        HttpContext.Request.InputStream.Seek(0, SeekOrigin.Begin);
-                        HttpContext.Request.InputStream.CopyTo(stream);
-                        body = Encoding.UTF8.GetString(stream.ToArray());
-                    }
-
-                    return body;
+                using (var stream = new MemoryStream())
+                {
+                    HttpContext.Request.InputStream.Seek(0, SeekOrigin.Begin);
+                    HttpContext.Request.InputStream.CopyTo(stream);
+                    body = Encoding.UTF8.GetString(stream.ToArray());
                 }
+
+                return body;
             }
             catch (Exception exception)
             {
@@ -187,14 +184,14 @@ private static IDictionary<string, string> Convert(Func<dynamic, NameObjectColle
                     if (key == null)
                         continue;
 
-                    string stringKey = key as string ?? key.ToString();
+                    var stringKey = key as string ?? key.ToString();
 
                     // NOTE: Ignore these keys as they just add duplicate information. [asbjornu]
                     if (stringKey.StartsWith("ALL_") || stringKey.StartsWith("HTTP_"))
                         continue;
 
                     var value = collection[stringKey];
-                    string stringValue = value as string;
+                    var stringValue = value as string;
 
                     if (stringValue != null)
                     {
@@ -228,10 +225,10 @@ private static IDictionary<string, string> Convert(Func<dynamic, NameObjectColle
 
         private static void TryGetHttpContextPropertyFromAppDomain()
         {
-            if (CheckedForHttpContextProperty)
+            if (checkedForHttpContextProperty)
                 return;
 
-            CheckedForHttpContextProperty = true;
+            checkedForHttpContextProperty = true;
 
             try
             {