Take into account JsonRequired of STJ

[jgarciadelanoceda]

Fixes #2686
JsonRequiredAttribute was introduced in dotnet7 altough you can opt in with a nuget reference.. I do not see this an option because previous versions of dotnet are about to get out of support

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.67%. Comparing base (fdb7d6b) to head (9bd77e5).

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

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2988   +/-   ##
=======================================
  Coverage   90.66%   90.67%           
=======================================
  Files          74       74           
  Lines        2946     2948    +2     
  Branches      468      468           
=======================================
+ Hits         2671     2673    +2     
  Misses        275      275           

Flag	Coverage Δ
Linux	90.67% <100.00%> (+<0.01%)	⬆️
Windows	?
macOS	90.67% <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]

Could you add (or amend an existing) verification test for this scenario please?

[cremor]

JsonRequiredAttribute was introduced in dotnet7 altough you can opt in with a nuget reference.. I do not see this an option because previous versions of dotnet are about to get out of support

Do I understand that correctly that you only had to wrap the code in an #if because Swashbuckle references STJ 4.6.0, which doesn't contain JsonRequiredAttribute?
If so, then why doesn't Swashbuckle update to the current version of STJ? As far as I can see it the current version of STJ (8.0.4) would still support all the target frameworks that Swashbuckle.AspNetCore.SwaggerGen uses.

[martincostello]

We will review the dependencies and their versions when we tackle #2798.

Trying to use the latest version with multiple TFMs can introduce a lot of complexity.