-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update NuGet packages #31316
Comments
This comment has been minimized.
This comment has been minimized.
@jaimecbernardo who can i ping for this? |
Ah, looks like we need to fix before .NET 9 hits in November. |
Regarding unused extensions, there were some that we needed to make sure dependencies follow the same versions after the flattening. |
Nobody mentioning the vulnerability issue? |
@Jay-o-Way: I'm currently looking into the BinaryFormatter deprecation / security issue. Regarding the System.Net.Http issue, could you indicate where you see the dependency? I don't see us taking any NuGet dependency on System.Net.Http in the entire solution. |
@drawbyperpetual thanks. System.Net.Http is unused in SvgPreviewHandler (FYI @zanseb) and is used in OobeWhatsNew - seemingly to create a way to link/show release notes. ( CC @jaimecbernardo and @lncubus |
@Jay-o-Way: Yes, System.Net.Http is indeed used there but not via a vulnerable NuGet package, but rather via a framework dependency on .NET Core 8. Where's the vulnerability there? |
@drawbyperpetual I just encountered the warning one day. Not an expert on the usage details 😇 |
System.Net.Http⚠️
Update 📈
Check for unused 🗑️
The text was updated successfully, but these errors were encountered: