OpenAPI Feedback: services.ConfigureHttpJsonOptions was needed for casing to be picked up for JsonPropertyName in generation

[niemyjski]

Is there an existing issue for this?

• I have searched the existing issues

Describe the bug

From meta issue: I'm pretty sure I had to add services.ConfigureHttpJsonOptions to get somethings to work can't remember what but I think it for casing to be picked up for JsonPropertyName in generation. This is weird and should just work if I have services.AddJsonOptions.

Expected Behavior

These should just work out of the box.

Steps To Reproduce

Clone https://github.com/exceptionless/aspnet-core-net10-issues# and run it, hit the open api json endpoint.

Exceptions (if any)

No response

.NET Version

10 preview 6

Anything else?

MacOS Latest, Rider/VS Code

#62150

[martincostello]

From meta issue: I'm pretty sure I had to add services.ConfigureHttpJsonOptions to get somethings to work can't remember what but I think it for casing to be picked up for JsonPropertyName in generation. This is weird and should just work if I have services.AddJsonOptions.

This is by design. AddJsonOptions() is for MVC - OpenAPI uses ConfigureHttpJsonOptions() which is the configuration used by Minimal APIs. Each one has their own JSON serialization settings and defaults.

Admittedly this is confusing. There's some discussion about this in a recent ASP.NET Core Community Standup (sorry I don't have the time index for exactly where).

[dotnet-policy-service]

This issue has been resolved and has not had any activity for 1 day. It will be closed for housekeeping purposes.

See our Issue Management Policies for more information.

[niemyjski]

I'm not using minimal api's and this is not intuitive. Can we add an analyzer or something.

Also, please adjust the bot. 1 day is too short. Just think if you went on vacation for a single day and came back to all your issues closed.

[niemyjski]

I think this is the timestamp for the video: https://www.youtube.com/watch?v=lXvHXA_vuro&t=3026s