Merge pull request #855 from SmRiley/release/6.x

Use NullabilityInfoContext to replace Type's nullable detection in release 6.x

src/MagicOnion.Server.HttpGateway/Swagger/SwaggerDefinitionBuilder.cs

@@ -177,7 +177,7 @@ Schemas.Parameter[] BuildParameters(IDictionary<string, Schema> definitions, Xml
                     @in = "formData",
                     type = swaggerDataType,
                     description = parameterXmlComment,
-                    required = !x.IsOptional && !x.ParameterType.IsNullable(), // OpenAPI 3 has a separate definition for nullable, but for OpenAPI 2, there is only required.
+                    required = !x.IsOptional && !x.IsNullable(), // OpenAPI 3 has a separate definition for nullable, but for OpenAPI 2, there is only required.
                     @default = defaultObjectExample ?? ((x.IsOptional) ? defaultValue : null),
                     items = items,
                     @enum = enums,
@@ -488,4 +488,4 @@ protected override JsonProperty CreateProperty(MemberInfo member, MemberSerializ
 
         return property;
     }
-}
+}

src/MagicOnion.Server.HttpGateway/Utils.cs

@@ -1,11 +1,17 @@
-using System.Reflection;
+using System.Reflection;
 
 namespace MagicOnion;
 
 internal static class Utils
 {
+    private static readonly NullabilityInfoContext nullabilityInfoContext = new();
+    public static bool IsNullable(this ParameterInfo type)
+    {
+        return nullabilityInfoContext.Create(type).WriteState == NullabilityState.Nullable;
+    }
+
     public static bool IsNullable(this Type type)
     {
         return type.GetTypeInfo().IsGenericType && type.GetGenericTypeDefinition() == typeof(Nullable<>);
     }
-}
+}