Fixed a bug where an ArgumentNullException would be thrown for post data that had null keys.

Author: niemyjski

Source/Platforms/Nancy/NancyRequestInfoCollector.cs

@@ -54,8 +54,7 @@ public static RequestInfo Collect(NancyContext context, IEnumerable<string> excl
         private static Dictionary<string, string> ToDictionary(this IEnumerable<KeyValuePair<string, string>> cookies, IEnumerable<string> exclusions) {
             var d = new Dictionary<string, string>();
 
-            foreach (var kv in cookies.Where(pair => !String.IsNullOrEmpty(pair.Key) && !pair.Key.AnyWildcardMatches(_ignoredCookies, true) && !pair.Key.AnyWildcardMatches(exclusions, true)))
-            {
+            foreach (var kv in cookies.Where(pair => !String.IsNullOrEmpty(pair.Key) && !pair.Key.AnyWildcardMatches(_ignoredCookies, true) && !pair.Key.AnyWildcardMatches(exclusions, true))) {
                 if (!d.ContainsKey(kv.Key))
                     d.Add(kv.Key, kv.Value);
             }
@@ -68,7 +67,7 @@ private static Dictionary<string, string> ToDictionary(this NameValueCollection
 
             var exclusionsArray = exclusions as string[] ?? exclusions.ToArray();
             foreach (string key in values.AllKeys) {
-                if (key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(exclusionsArray, true))
+                if (String.IsNullOrEmpty(key) || key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(exclusionsArray, true))
                     continue;
 
                 try {

Source/Platforms/Web/RequestInfoCollector.cs

@@ -88,7 +88,7 @@ private static Dictionary<string, string> ToDictionary(this HttpCookieCollection
             foreach (string key in cookies.AllKeys.Distinct().Where(k => !String.IsNullOrEmpty(k) && !k.AnyWildcardMatches(_ignoredCookies, true) && !k.AnyWildcardMatches(exclusions, true))) {
                 try {
                     HttpCookie cookie = cookies.Get(key);
-                    if ( cookie != null && cookie.Value != null && cookie.Value.Length < MAX_DATA_ITEM_LENGTH && !d.ContainsKey(key))
+                    if (cookie != null && cookie.Value != null && cookie.Value.Length < MAX_DATA_ITEM_LENGTH && !d.ContainsKey(key))
                         d.Add(key, cookie.Value);
                 } catch (Exception ex) {
                     if (!d.ContainsKey(key))
@@ -104,7 +104,7 @@ private static Dictionary<string, string> ToDictionary(this NameValueCollection
 
             var exclusionsArray = exclusions as string[] ?? exclusions.ToArray();
             foreach (string key in values.AllKeys) {
-                if (key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(exclusionsArray, true))
+                if (String.IsNullOrEmpty(key) || key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(exclusionsArray, true))
                     continue;
 
                 try {

Source/Platforms/WebApi/HttpRequestInfoCollector.cs

@@ -70,7 +70,7 @@ private static Dictionary<string, string> ToDictionary(this NameValueCollection
 
             var patternsToMatch = exclusions as string[] ?? exclusions.ToArray();
             foreach (string key in values.AllKeys) {
-                if (key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(patternsToMatch, true))
+                if (String.IsNullOrEmpty(key) || key.AnyWildcardMatches(_ignoredFormFields, true) || key.AnyWildcardMatches(patternsToMatch, true))
                     continue;
 
                 try {