Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Commit

Permalink
[nuget.config] remove public nuget.org feed (#13)
Browse files Browse the repository at this point in the history
Context: https://medium.com/@alex.birsan/dependency-confusion-4a5d60fec610
Context: https://azure.microsoft.com/en-us/resources/3-ways-to-mitigate-risk-using-private-package-feeds/
Context: https://devdiv.visualstudio.com/DevDiv/_wiki/wikis/DevDiv.wiki/12676/ncident-help-for-Substitution-attack-risk-from-multiple-package-feeds

There is a Package Substitution Attack inherent in NuGet, whereby
if multiple package sources provide packages with the same name,
it is indeterminate which package source will provide the package.

To fix this repo, we don't actually need `nuget.org` at all.
  • Loading branch information
jonathanpeppers authored Feb 9, 2021
1 parent 25b0189 commit 9428620
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion samples/nuget.config
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
<add key="release" value="../bin/Release/" />
<add key="dotnet6" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet6/nuget/v3/index.json" />
<add key="xamarin" value="https://pkgs.dev.azure.com/azure-public/vside/_packaging/xamarin-impl/nuget/v3/index.json" />
<add key="nuget" value="https://api.nuget.org/v3/index.json" />
</packageSources>
<config>
<add key="globalPackagesFolder" value="../packages" />
Expand Down

0 comments on commit 9428620

Please sign in to comment.