Skip to content

Drop deprecated Microsoft.SourceLink.GitHub 8.0.0 (CVE-2026-62900) - #1122

Merged
FlorianRappl merged 1 commit into
ElectronNET:developfrom
DYH1319:fix/drop-deprecated-sourcelink
Oct 27, 2026
Merged

FlorianRappl merged 1 commit into
ElectronNET:developfrom
DYH1319:fix/drop-deprecated-sourcelink

Conversation

@DYH1319

@DYH1319 DYH1319 commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes Microsoft.SourceLink.GitHub 8.0.0 from ElectronNET.API and ElectronNET.AspNet.

The package transitively depends on Microsoft.Build.Tasks.Git 8.0.0, which has a known vulnerability — CVE-2026-62900 / GHSA-23fw-v26w-5fgq (.NET Information Disclosure, CVSS 5.9). Per the advisory, the 8.0.0 package line has no patched NuGet version.

The fix is safe because the package is deprecated anyway: SourceLink has been built into the .NET SDK since .NET 8. With PublishRepositoryUrl=true already set in common.props, the SDK's built-in SourceLink continues to embed repository/commit metadata (e.g. +commitsha in InformationalVersion) without any package reference — verified locally on net10.0 builds.

Test plan

  • dotnet build — NU1902 warning eliminated
  • AssemblyInformationalVersion still carries +<commit> suffix (built-in SourceLink active)

@DYH1319
DYH1319 marked this pull request as ready for review October 27, 2026 15:38
@github-actions

Copy link
Copy Markdown

pr-comment: Run #155

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
1680 1518 0 0 0 162 0 4m 2s

🎉 All tests passed!

Github Test Reporter by CTRF 💚

SourceLink is built into the .NET SDK since .NET 8; the package is
deprecated and pins vulnerable Microsoft.Build.Tasks.Git 8.0.0
(CVE-2026-62900 / GHSA-23fw-v26w-5fgq). PublishRepositoryUrl=true in
common.props keeps built-in SourceLink active.
@DYH1319
DYH1319 force-pushed the fix/drop-deprecated-sourcelink branch from 24dcca0 to a7f14ff Compare October 27, 2026 15:39
@DYH1319
DYH1319 changed the base branch from main to develop October 27, 2026 15:39
@DYH1319
DYH1319 marked this pull request as draft October 27, 2026 15:39
@DYH1319
DYH1319 marked this pull request as ready for review October 27, 2026 15:41

@FlorianRappl FlorianRappl left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - imho we can safely do that without any side-effect.

@FlorianRappl FlorianRappl added this to the 1.0.0 milestone Oct 27, 2026
@FlorianRappl
FlorianRappl merged commit 8ae9d6c into ElectronNET:develop Oct 27, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants