Skip to content

fix: resolve Dependency Review summary failure by patching MessagePack vulnerability#113

Merged
JerrettDavis merged 2 commits into
mainfrom
dependabot/nuget/src/Core/PokManager.Application/extensions-4b9495a5ef
Jun 15, 2026
Merged

fix: resolve Dependency Review summary failure by patching MessagePack vulnerability#113
JerrettDavis merged 2 commits into
mainfrom
dependabot/nuget/src/Core/PokManager.Application/extensions-4b9495a5ef

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes the failing GitHub Actions job Dependency Review / Review Summary (pull_request) by addressing the root cause in the Vulnerability Check job.

The failure was caused by a transitive vulnerable dependency: MessagePack 2.5.192 (GHSA-hv8m-jj95-wg3x).
To remediate this, PokManager.AppHost now explicitly references a patched version: MessagePack 2.5.301.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test update
  • Other (please describe):

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Related Issues

N/A

Screenshots (if applicable)

Not applicable.

Testing

  • dotnet build PokManager.sln
  • dotnet test PokManager.sln
  • dotnet list package --vulnerable --include-transitive

Verified that no vulnerable packages are reported after pinning MessagePack to 2.5.301.

Test Configuration:

  • OS: Ubuntu 24.04
  • .NET Version: .NET SDK 10.x
  • Build Configuration: Debug

Additional Notes

Only one file was changed:

  • src/Hosting/PokManager.AppHost/PokManager.AppHost.csproj

A direct PackageReference was added to override the vulnerable transitive MessagePack version.

Breaking Changes

None.

@dependabot @github

dependabot Bot commented on behalf of github Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

Assignees

The following users could not be added as assignees: PokManagerApi/maintainers. Either the username does not exist or it does not have the correct permissions to be added as an assignee.

Labels

The following labels could not be found: area: build, dependencies, nuget. Please create them before Dependabot can add them to a pull request.

Please fix the above issues or remove invalid values from dependabot.yml.

@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 5 package(s) with unknown licenses.
See the Details below.

License Issues

src/Core/PokManager.Application/PokManager.Application.csproj

PackageVersionLicenseIssue Type
Microsoft.Extensions.DependencyInjection.Abstractions10.0.9NullUnknown License

src/Hosting/PokManager.AppHost/PokManager.AppHost.csproj

PackageVersionLicenseIssue Type
MessagePack2.5.301NullUnknown License

src/Hosting/PokManager.ServiceDefaults/PokManager.ServiceDefaults.csproj

PackageVersionLicenseIssue Type
Microsoft.Extensions.Http.Resilience10.7.0NullUnknown License
Microsoft.Extensions.ServiceDiscovery10.7.0NullUnknown License

src/Infrastructure/PokManager.Infrastructure/PokManager.Infrastructure.csproj

PackageVersionLicenseIssue Type
Microsoft.Extensions.Hosting.Abstractions10.0.9NullUnknown License
Allowed Licenses: MIT, Apache-2.0, BSD-2-Clause, BSD-3-Clause, ISC, 0BSD

OpenSSF Scorecard

PackageVersionScoreDetails
nuget/Microsoft.Extensions.DependencyInjection.Abstractions 10.0.9 UnknownUnknown
nuget/MessagePack 2.5.301 UnknownUnknown
nuget/Microsoft.Extensions.Http.Resilience 10.7.0 🟢 6.7
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 10 issue activity found in the last 90 days -- score normalized to 10
Code-Review🟢 10all changesets reviewed
Packaging⚠️ -1packaging workflow not detected
Security-Policy🟢 10security policy file detected
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
License🟢 10license file detected
Branch-Protection🟢 5branch protection is not maximal on development and all release branches
Binary-Artifacts🟢 10no binaries found in the repo
Signed-Releases⚠️ -1no releases found
Fuzzing⚠️ 0project is not fuzzed
Pinned-Dependencies🟢 9dependency not pinned by hash detected -- score normalized to 9
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
nuget/Microsoft.Extensions.ServiceDiscovery 10.7.0 🟢 6.7
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 10 issue activity found in the last 90 days -- score normalized to 10
Code-Review🟢 10all changesets reviewed
Packaging⚠️ -1packaging workflow not detected
Security-Policy🟢 10security policy file detected
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
License🟢 10license file detected
Branch-Protection🟢 5branch protection is not maximal on development and all release branches
Binary-Artifacts🟢 10no binaries found in the repo
Signed-Releases⚠️ -1no releases found
Fuzzing⚠️ 0project is not fuzzed
Pinned-Dependencies🟢 9dependency not pinned by hash detected -- score normalized to 9
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
nuget/Microsoft.Extensions.Hosting.Abstractions 10.0.9 UnknownUnknown

Scanned Files

  • src/Core/PokManager.Application/PokManager.Application.csproj
  • src/Hosting/PokManager.AppHost/PokManager.AppHost.csproj
  • src/Hosting/PokManager.ServiceDefaults/PokManager.ServiceDefaults.csproj
  • src/Infrastructure/PokManager.Infrastructure/PokManager.Infrastructure.csproj

@JerrettDavis

Copy link
Copy Markdown
Owner

@dependabot rebase

… 3 others

Bumps Microsoft.Extensions.DependencyInjection.Abstractions from 10.0.8 to 10.0.9
Bumps Microsoft.Extensions.Hosting.Abstractions from 10.0.8 to 10.0.9
Bumps Microsoft.Extensions.Http.Resilience from 10.6.0 to 10.7.0
Bumps Microsoft.Extensions.ServiceDiscovery from 10.6.0 to 10.7.0

---
updated-dependencies:
- dependency-name: Microsoft.Extensions.DependencyInjection.Abstractions
  dependency-version: 10.0.9
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: extensions
- dependency-name: Microsoft.Extensions.Hosting.Abstractions
  dependency-version: 10.0.9
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: extensions
- dependency-name: Microsoft.Extensions.Http.Resilience
  dependency-version: 10.7.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: extensions
- dependency-name: Microsoft.Extensions.ServiceDiscovery
  dependency-version: 10.7.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: extensions
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot changed the title chore: Bump the extensions group with 4 updates chore: Bump Microsoft.Extensions.DependencyInjection.Abstractions and 3 others Jun 12, 2026
@dependabot dependabot Bot force-pushed the dependabot/nuget/src/Core/PokManager.Application/extensions-4b9495a5ef branch from c6b7f78 to 3d41d8a Compare June 12, 2026 16:34
Copilot AI changed the title chore: Bump Microsoft.Extensions.DependencyInjection.Abstractions and 3 others fix: resolve Dependency Review summary failure by patching MessagePack vulnerability Jun 12, 2026
Copilot AI requested a review from JerrettDavis June 12, 2026 16:45
@JerrettDavis JerrettDavis merged commit 0e093c6 into main Jun 15, 2026
17 checks passed
@JerrettDavis JerrettDavis deleted the dependabot/nuget/src/Core/PokManager.Application/extensions-4b9495a5ef branch June 15, 2026 23:19
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