修正在未正常响应时Accept包含通配符会导致异常的问题；

Author: stratosblue

src/Cuture.AspNetCore.ResponseAutoWrapper/Cuture.AspNetCore.ResponseAutoWrapper.csproj

@@ -8,7 +8,7 @@
     <IsPackable>true</IsPackable>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
 
-    <Version>1.1.0</Version>
+    <Version>1.1.1</Version>
     <Description>用于`asp.net core`的响应和异常自动包装器，使`Action`提供一致的响应内容格式</Description>
 
     <PackageIdPrefix>Cuture.AspNetCore.ResponseAutoWrapper</PackageIdPrefix>
@@ -19,7 +19,7 @@
     <RepositoryType>git</RepositoryType>
     <RepositoryUrl>$(PackageProjectUrl)</RepositoryUrl>
 
-    <PackageTags>response-wrapper autowrap autowrapper auto-wrapper response-auto-wrapper asp-net-core</PackageTags>
+    <PackageTags>response-wrapper responsewrapper response-wrap responsewrap autowrap auto-wrapper autowrapper response-auto-wrapper responseautowrapper asp-net-core</PackageTags>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)' == 'Release'">

src/Cuture.AspNetCore.ResponseAutoWrapper/ResponseAutoWrapMiddleware.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Runtime.CompilerServices;
 using System.Threading.Tasks;
 
 using Microsoft.AspNetCore.Http;
@@ -9,6 +10,7 @@
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Options;
+using Microsoft.Extensions.Primitives;
 using Microsoft.Net.Http.Headers;
 
 namespace Cuture.AspNetCore.ResponseAutoWrapper;
@@ -172,11 +174,61 @@ public async Task InvokeAsync(HttpContext context)
 
     private Task WriteResponseWithFormatterAsync(HttpContext context, object response)
     {
-        var formatterContext = new OutputFormatterWriteContext(context,
-                                                               _httpResponseStreamWriterFactory.CreateWriter,
-                                                               response.GetType(),
-                                                               response);
+        var formatterWriteContext = new OutputFormatterWriteContext(context,
+                                                                    _httpResponseStreamWriterFactory.CreateWriter,
+                                                                    response.GetType(),
+                                                                    response);
+
+        if (context.Request.Headers.TryGetValue(HeaderNames.Accept, out var acceptValues)
+            && acceptValues.Count > 0
+            && !EmptyOrHasWildcard(acceptValues))
+        {
+            return WriteResponseWithSelectFormatterAsync(context, formatterWriteContext);
+        }
+
+        // 有通配符，直接使用默认Formatter
+        return _defaultOutputFormatter.WriteAsync(formatterWriteContext);
+
+        static bool EmptyOrHasWildcard(in StringValues acceptValues)
+        {
+            // 检查通配符
+            return acceptValues.Count switch
+            {
+                1 => NullOrHasWildcard(acceptValues, 0),
+                2 => NullOrHasWildcard(acceptValues, 0) && NullOrHasWildcard(acceptValues, 1),
+                { } count => HasAnyWildcard(acceptValues, count),
+            };
+        }
 
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
+        static bool HasAnyWildcard(in StringValues acceptValues, int count)
+        {
+            for (int i = 0; i < count; i++)
+            {
+                if (NullOrHasWildcard(acceptValues, i))
+                {
+                    return true;
+                }
+            }
+            return false;
+        }
+
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
+        static bool NullOrHasWildcard(in StringValues acceptValues, in int index)
+        {
+            var value = acceptValues[index];
+            return value is null || value.Contains("*/*", StringComparison.Ordinal);
+        }
+    }
+
+    /// <summary>
+    /// 选择Formatter进行响应
+    /// </summary>
+    /// <param name="context"></param>
+    /// <param name="formatterWriteContext"></param>
+    /// <returns></returns>
+    private Task WriteResponseWithSelectFormatterAsync(HttpContext context, OutputFormatterWriteContext formatterWriteContext)
+    {
         var accepts = context.Request.Headers.TryGetValue(HeaderNames.Accept, out var acceptValue)
                       ? MediaTypeHeaderValue.TryParseList(acceptValue, out var parsedAcceptValues)
                         ? CreateMediaTypeCollection(parsedAcceptValues)
@@ -185,12 +237,12 @@ private Task WriteResponseWithFormatterAsync(HttpContext context, object respons
 
         var selectedFormatter = accepts is null
                                 ? _defaultOutputFormatter
-                                : _outputFormatterSelector.SelectFormatter(formatterContext,
-                                                                         Array.Empty<IOutputFormatter>(),
-                                                                         accepts)
-                                    ?? _defaultOutputFormatter;
+                                : _outputFormatterSelector.SelectFormatter(formatterWriteContext,
+                                                                           Array.Empty<IOutputFormatter>(),
+                                                                           accepts)
+                                  ?? _defaultOutputFormatter;
 
-        return selectedFormatter.WriteAsync(formatterContext);
+        return selectedFormatter.WriteAsync(formatterWriteContext);
     }
 
     #endregion Internal

src/Cuture.AspNetCore.ResponseAutoWrapper/ResponseAutoWrapMiddlewareOptions.cs

@@ -14,7 +14,7 @@ public class ResponseAutoWrapMiddlewareOptions
 
     /// <summary>
     /// 是否捕获异常<para/>
-    /// default is "true"
+    /// default is <see langword="true"/>
     /// </summary>
     public bool CatchExceptions { get; set; } = true;
 
@@ -29,7 +29,7 @@ public class ResponseAutoWrapMiddlewareOptions
 
     /// <summary>
     /// 是否将捕获到的异常抛出给上层中间件<para/>
-    /// default is "fasle"
+    /// default is <see langword="false"/>
     /// </summary>
     public bool ThrowCaughtExceptions { get; set; } = false;