Skip to content

fix: remove unused Intility.Logging.AspNetCore dependency - #242

Open
Erlend Ellefsen (erlendellefsen) wants to merge 1 commit into
mainfrom
fix/remove-unused-logging-dependency
Open

Erlend Ellefsen (erlendellefsen) wants to merge 1 commit into
mainfrom
fix/remove-unused-logging-dependency

Conversation

@erlendellefsen

Copy link
Copy Markdown
Collaborator

Summary

JsonApiToolkit referenced Intility.Logging.AspNetCore, but no library code uses it. The reference pulled Microsoft.Identity.Web and vulnerable System.Security.Cryptography.Xml/Pkcs 10.0.11 (NU1902/NU1903) into every consumer. This PR removes the reference and updates the lock files.

How to test

  1. Run dotnet restore --locked-mode.
  2. Run dotnet test. No NU1902/NU1903 warnings appear.

This branch has not been deployed

No deployments
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.

1 participant