Prevent form reading if content-type is invalid

Author: usercode

src/AspNetCore.Honeypot/.editorconfig

@@ -34,31 +34,33 @@ public async Task<bool> IsTriggeredAsync(HttpContext httpContext)
             {
                 triggered = true;
             }
-
-            IFormCollection form = await httpContext.Request.ReadFormAsync();
-
-            if (triggered == false && Options.IsFieldCheckEnabled)
+            else
             {
-                //check fields
-                triggered = form.Any(x => Options.IsFieldName(x.Key) && x.Value.Any(v => string.IsNullOrEmpty(v) == false));
-            }
+                IFormCollection form = await httpContext.Request.ReadFormAsync();
 
-            if (triggered == false && Options.IsTimeCheckEnabled)
-            {
-                //check time
-                if (form.TryGetValue(Options.TimeFieldName, out StringValues timeValues))
+                if (triggered == false && Options.IsFieldCheckEnabled)
                 {
-                    if (timeValues.Count > 0 && timeValues[0] is string timeValue)
+                    //check fields
+                    triggered = form.Any(x => Options.IsFieldName(x.Key) && x.Value.Any(v => string.IsNullOrEmpty(v) == false));
+                }
+
+                if (triggered == false && Options.IsTimeCheckEnabled)
+                {
+                    //check time
+                    if (form.TryGetValue(Options.TimeFieldName, out StringValues timeValues))
                     {
-                        if (long.TryParse(timeValue, out long time))
+                        if (timeValues.Count > 0 && timeValues[0] is string timeValue)
                         {
-                            TimeSpan diff = DateTime.UtcNow - new DateTime(time, DateTimeKind.Utc);
+                            if (long.TryParse(timeValue, out long time))
+                            {
+                                TimeSpan diff = DateTime.UtcNow - new DateTime(time, DateTimeKind.Utc);
 
-                            triggered = diff < Options.MinResponseTime;
-                        }
-                        else
-                        {
-                            triggered = true; //time field doesn't contain long value.
+                                triggered = diff < Options.MinResponseTime;
+                            }
+                            else
+                            {
+                                triggered = true; //time field doesn't contain long value.
+                            }
                         }
                     }
                 }

src/AspNetCore.Honeypot/HoneypotService.cs

@@ -10,7 +10,6 @@ public class HoneypotFieldTagHelper : TagHelper
 {
     public HoneypotFieldTagHelper(IOptions<HoneypotOptions> options)
     {
-        Name = "name";
         Options = options.Value;
     }
 
@@ -22,7 +21,7 @@ public HoneypotFieldTagHelper(IOptions<HoneypotOptions> options)
     /// <summary>
     /// Name
     /// </summary>
-    public string Name { get; set; }
+    public string Name { get; set; } = "name";
 
     public override void Process(TagHelperContext context, TagHelperOutput output)
     {

src/AspNetCore.Honeypot/TagHelpers/HoneypotFieldTagHelper.cs

@@ -16,7 +16,7 @@ public HoneypotTimeTagHelper(IOptions<HoneypotOptions> options)
     /// <summary>
     /// Options
     /// </summary>
-    public HoneypotOptions Options { get; }    
+    public HoneypotOptions Options { get; }
 
     public override void Process(TagHelperContext context, TagHelperOutput output)
     {