Skip to content

Commit 507325f

Browse files
committed
build(deps): bump BouncyCastle.Cryptography 2.0.0 -> 2.6.2, drop redundant prune-flagged refs
NU1902 — Keyfactor.PKI 5.5.0 brings in BouncyCastle.Cryptography 2.0.0 transitively, which carries three known-moderate CVEs: * GHSA-8xfc-gm6g-vgpv (DH timing attack) * GHSA-m44j-cfrm-g8qc (Pkcs5S2ParametersGenerator weakness) * GHSA-v435-xc8x-wvr9 (X.509 cert verification bypass) All three are fixed in 2.4.0+; bump to the current stable 2.6.2 on both the main plugin csproj (explicit override of the transitive) and the integration tests csproj (which has a direct ref). NU1510 — System.Text.Json and System.Text.Encodings.Web were pinned explicit PackageReferences. Both .NET 8/10 SDKs now flag them as unnecessary because they're bundled or provided transitively in a form that satisfies our usage; removed both refs from CERTInext.csproj. Build + 146/146 unit tests still pass after the removal, so no real dependency was being satisfied by those direct refs. Verified live: Ping/GetProductDetails/ListOrders all PASS against the sandbox after the bump — the new BouncyCastle 2.6.2 SHA-256 path still produces a valid authKey. Remaining warnings after this commit (deferred for separate triage): seven CS8602/CS8604 nullable-deref warnings in test code only — all sit behind FluentAssertions .Should().NotBeNull() guards, so they won't actually NRE at runtime. Worth a follow-up to add `!` suppressions but not a blocker.
1 parent 63ed82f commit 507325f

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

CERTInext.IntegrationTests/CERTInext.IntegrationTests.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
</PackageReference>
2222
<PackageReference Include="FluentAssertions" Version="6.12.0" />
2323
<PackageReference Include="Xunit.SkippableFact" Version="1.4.13" />
24-
<PackageReference Include="BouncyCastle.Cryptography" Version="2.0.0" />
24+
<PackageReference Include="BouncyCastle.Cryptography" Version="2.6.2" />
2525
<!-- Suppress TFM support build warnings for transitive dependencies -->
2626
<PackageReference Include="Microsoft.Bcl.AsyncInterfaces" Version="8.0.0" />
2727
<PackageReference Include="System.IO.Pipelines" Version="8.0.0" />

CERTInext/CERTInext.csproj

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,14 @@
1414
<PackageReference Include="Keyfactor.Logging" Version="1.1.2" />
1515
<PackageReference Include="Keyfactor.PKI" Version="5.5.0" />
1616
<PackageReference Include="RestSharp" Version="112.1.0" />
17-
<PackageReference Include="System.Text.Json" Version="8.0.5" />
1817
<PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
18+
<!-- Explicit override of the transitive BouncyCastle.Cryptography 2.0.0 pulled in by
19+
Keyfactor.PKI 5.5.0. 2.0.0 has three known-moderate CVEs (GHSA-8xfc-gm6g-vgpv,
20+
GHSA-m44j-cfrm-g8qc, GHSA-v435-xc8x-wvr9), all fixed in 2.4.0+. -->
21+
<PackageReference Include="BouncyCastle.Cryptography" Version="2.6.2" />
1922
<!-- Suppress TFM support build warnings for transitive dependencies of RestSharp 112.x -->
2023
<PackageReference Include="Microsoft.Bcl.AsyncInterfaces" Version="8.0.0" />
2124
<PackageReference Include="System.IO.Pipelines" Version="8.0.0" />
22-
<PackageReference Include="System.Text.Encodings.Web" Version="8.0.0" />
2325
</ItemGroup>
2426

2527
<ItemGroup>

0 commit comments

Comments
 (0)