OpenApi generation OpenApiSchemaComparer fails with null values for Properties, AllOf, or AnyOf

[FrankSzendzielarz]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

OpenApiSchemaComparer uses code like hashCode.Add(obj.Properties.Count); . In custom schema transforms where I might want to replace allof , anyof, or properties, intuitively I would set unwanted properties to null. Either the setter should prevent this, or the obj.Properties.Count should be obj.Properties?.Count??0 or otherwise take care of nulls.

Expected Behavior

Not crash when a schema property is null.

Steps To Reproduce

Custom schema transform - set Properties to null. OpenApi doc gen and Scalar result have missing elements.

Exceptions (if any)

Object reference not set to an instance of an object.

.NET Version

9

Anything else?

No response

[captainsafia]

@FrankSzendzielarz Thanks for filing this issue!

I believe as a temporary workaround you can initialize the designated properties to empty lists. The OpenAPI document writer should do the write thing and not emit them but let me know if that's not the case.

With regard to how we might solve this in the long-term, with the adoption of Microsoft.OpenApi v2, we're hoping to move forward with the comparers being implemented in the library themselves (instead of in the ASP.NET Core codebase). There's also some changes coming to the nullability/default blues for collection-based properties like AllOf and friends.

I'll take this one as a sub-issue under #58619 for now since the comparers are removed from source in #59480.