Make STJ conditional #40

[thompson-tomo]

Closes #40

[Copilot]

Pull Request Overview

This PR updates the NATS.Jwt project file by making the inclusion of the System.Text.Json package conditional to the netstandard2.0 target framework.

• Moves the System.Text.Json package reference into a conditional ItemGroup for netstandard2.0
• Removes the unconditional reference from the global ItemGroup

Comments suppressed due to low confidence (1)

src/NATS.Jwt/NATS.Jwt.csproj:26

• Moving the System.Text.Json package reference into a conditional ItemGroup ensures it’s only included for netstandard2.0. Please confirm that no other target frameworks require a direct dependency on System.Text.Json.

    <PackageReference Include="System.Text.Json" Version="8.0.5"/>

[mtmk]

LGTM thanks @thompson-tomo

[mtmk]

• Moving the System.Text.Json package reference into a conditional ItemGroup ensures it’s only included for netstandard2.0. Please confirm that no other target frameworks require a direct dependency on System.Text.Json.

    <PackageReference Include="System.Text.Json" Version="8.0.5"/>

STJ is included in new .NET SDKs already. the reason we added this was due to a CVE at the time. Since SDKs are patched by now we can assume users should've installed the fixed SDKs by now.