Skip to content

.NET: chore: updates source link dependency due to transitive vulnerability - #8179

Merged
Roger Barreto (rogerbarreto) merged 1 commit into
mainfrom
chore/update-source-link
Sep 9, 2026
Merged

Roger Barreto (rogerbarreto) merged 1 commit into
mainfrom
chore/update-source-link

Conversation

@baywet

Copy link
Copy Markdown
Member

Updates source link dependency as it has a transitive known vulnerability which causes build failures. It's an attempt to unblock #8167 and #8166 among other PRs

@baywet Vincent Biret (baywet) self-assigned this Sep 9, 2026
Copilot AI balanced review requested due to automatic review settings September 9, 2026 10:53
@agent-framework-automation agent-framework-automation Bot added the .NET Usage: [Issues, PRs], Target: .Net label Sep 9, 2026
@github-actions github-actions Bot changed the title chore: updates source link dependency due to transitive vulnerability .NET: chore: updates source link dependency due to transitive vulnerability Sep 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The dependency-only update was fully reviewed with no unresolved issues.

Pull request overview

Updates Source Link to address a vulnerable transitive dependency and unblock related .NET dependency upgrades.

Changes:

  • Upgrades Microsoft.SourceLink.GitHub from 8.0.0 to 10.0.401.
File summaries
File Description
dotnet/Directory.Packages.props Updates the centrally managed Source Link version.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

MAF Automated Review — Iteration 1

Result: No findings
Scope: full PR (1 commit(s)): 5175f3967664
Model: gpt-5.6-sol-fast

Overview

This PR centrally upgrades the private build-time SourceLink dependency to 10.0.401 while preserving the existing symbol-package contract across all package projects. Central package management, PrivateAssets="All", warnings-as-errors builds, multi-target CI, package creation, and representative package installation constrain compatibility and dependency-leakage risk. The reviewers found no supported Critical, High, or Medium defect introduced by the change.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
No publishable findings remained after source verification for this scope.

@github-code-quality

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall line coverage in commit 5175f39 in the chore/update-source-... branch is 84%. Line coverage data for the main branch is not yet available.

Show a line coverage summary of the most covered files.
File main chore/update-source-... 5175f39 +/-
/home/runner/wo...valConverter.cs 100%
/home/runner/wo...entsProvider.cs 99%
/home/runner/wo...nticAnalyzer.cs 94%
/home/runner/wo...tClientAgent.cs 91%
/home/runner/wo...putConverter.cs 90%
/home/runner/wo...kflowBuilder.cs 90%
/home/runner/wo...kflowSession.cs 87%
/home/runner/wo...SkillsSource.cs 86%
/home/runner/wo...onExtensions.cs 81%
/home/runner/wo...ctionVisitor.cs 70%

@baywet
Vincent Biret (baywet) added this pull request to the merge queue Sep 9, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 9, 2026
@rogerbarreto
Roger Barreto (rogerbarreto) added this pull request to the merge queue Sep 9, 2026
Merged via the queue into main with commit 3183438 Sep 9, 2026
39 checks passed
This was referenced Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants