microsoft / devhomegithubextension

Dev Home GitHub Extension
MIT License
280 stars 43 forks source link

Fix Json serializer reflection #433

Closed dkbennett closed 1 month ago

dkbennett commented 1 month ago

Summary of the pull request

Sets JsonSerializer reflection default back to .Net 7 compat.  

References and relevant issues

Same fix in Azure Extension Repo: https://github.com/microsoft/DevHomeAzureExtension/pull/135

Detailed description of the pull request / Additional comments

Migration to .Net 8 changed the Json Serializer reflection default. This sets it back to .Net 7 for compat. There may be work here in the future to be able to disable this, but this will get the extension working again.

Note that this does not repro on Debug builds, must build Release to see the bug and verify fix.

Validation steps performed

PR checklist