Remove reflection-based JSON serialization from SwaggerUIMiddleware

[VMelnalksnis]

Fixes #2550

[HakamFostok]

Can I ask a question, I saw the bug and I reviewed the PR, I know this is not my business, but I just want to wonder, if possible, why this PR is not merged yet.
I faced this problem in my Application, that led me to here.
Thank you very much for your efforts.

[TurcanStanislav]

Hi everyone. @VMelnalksnis, @bdukes, @domaindrivendev, @captainsafia, @eslamisepehr can I ask whether you will have time soon to fix the issues with the workflow and merge this PR? It would be great if you would manage to do this because this will help a lot of people. Thanks!

[VMelnalksnis]

I see that CI is failing because .NET 6 SDK is missing:

Looks like a fix for this was already provided in #2672, though I'm not sure what is the correct fix - changing the test target framework or installing the correct SDK. Either way, I can't merge PRs so someone else will have to review/merge/publish new version.

[martincostello]

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI. We'd also appreciate tests for this scenario to prevent regressions.

[VMelnalksnis]

Ok, I'll check out the current tests and see where I can add them.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.71%. Comparing base (0108842) to head (0bdd165).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2724      +/-   ##
==========================================
+ Coverage   91.69%   91.71%   +0.01%     
==========================================
  Files          91       91              
  Lines        3022     3029       +7     
  Branches      519      519              
==========================================
+ Hits         2771     2778       +7     
  Misses        251      251              

Flag	Coverage Δ
Linux	91.71% <100.00%> (+0.01%)	⬆️
Windows	91.71% <100.00%> (+0.01%)	⬆️
macOS	91.71% <100.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[martincostello]

Ok, I'll check out the current tests and see where I can add them.

I was looking at this myself yesterday and came up with this that builds on top of #2799: 880fc5f

If you'd like to cherry-pick those changes I'm happy to close my PR in favour of this one if it's making basically the same updates.

[VMelnalksnis]

I see that the PR already adds net8.0 targets to tests, and I guess fixes all AOT/reflection based issues, not just in this one place. It would make sense to go with that PR, and I'm just happy to see the issue resolved.