-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix Secure Supply Chain compliance issues #11548
Conversation
<!-- Warning: Never add more NuGet feeds, to be compliant this repo must only rely on this single ADO feed. --> | ||
<add key="react-native" value="https://pkgs.dev.azure.com/ms/react-native/_packaging/react-native-public/nuget/v3/index.json" /> | ||
<!-- Remove (comment out) the nuget.org feed once our ADO feed is configured correctly. See https://github.com/microsoft/react-native-windows/issues/11557 --> | ||
<add key="nuget.org" value="https://api.nuget.org/v3/index.json" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just FYI: Nuget fires requests to all feeds at the same time and first to return will be chosen. If you want ot populate our public feed you'll need ot make sure you have no nuget cache and remove this line...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, the goal is once we're ready to turn on upstream for react-native-public
then I'll remove the nuget.org line and going forward we'll be compliant.
@@ -33,6 +33,12 @@ node_modules | |||
# Optional REPL history | |||
.node_repl_history | |||
|
|||
# No NPM config to pass CFS compliance | |||
.npmrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confused about why we are ignoring lock files and .npmrc... Just like nuget.config we need to have a .npmrc
at the root
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not according to the task - I removed .npmrc and it passes.
Note: we'll need to do the same for |
This PR backports PRs microsoft#11548 and microsoft#11855 to 0.72. Fix Secure Supply Chain compliance issues (microsoft#11548) Resolves Secure Supply Chain warnings. - Bug fix (non-breaking change which fixes an issue) To meet compliance requirements. Closes microsoft#11209 Closes microsoft#10374 * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: microsoft#11557) N/A N/A CG: Move to single nuget source (microsoft#11855) * CG: Move to single nuget source * update nuget lock files * Change files
This PR backports PRs #11548 and #11855 to 0.72. # Fix Secure Supply Chain compliance issues (#11548) ## Description Resolves Secure Supply Chain warnings. ### Type of Change - Bug fix (non-breaking change which fixes an issue) ### Why To meet compliance requirements. Closes #11209 Closes #10374 ### What * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: #11557) ## Screenshots N/A ## Testing N/A # CG: Move to single nuget source (#11855) ## Description CG is failing builds due to multiple nuget feeds in nuget.config. ### Why What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves #11557
This PR backports PRs microsoft#11548 and microsoft#11855 to 0.70. Resolves Secure Supply Chain warnings. - Bug fix (non-breaking change which fixes an issue) To meet compliance requirements. Closes microsoft#11209 Closes microsoft#10374 * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: microsoft#11557) N/A N/A CG is failing builds due to multiple nuget feeds in nuget.config. What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves microsoft#11557
This PR backports PRs microsoft#11548 and microsoft#11855 to 0.72. Resolves Secure Supply Chain warnings. - Bug fix (non-breaking change which fixes an issue) To meet compliance requirements. Closes microsoft#11209 Closes microsoft#10374 * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: microsoft#11557) N/A N/A CG is failing builds due to multiple nuget feeds in nuget.config. What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves microsoft#11557
This PR backports PRs #11548 and #11855 to 0.70. # Fix Secure Supply Chain compliance issues (#11548) ## Description Resolves Secure Supply Chain warnings. ### Type of Change - Bug fix (non-breaking change which fixes an issue) ### Why To meet compliance requirements. Closes #11209 Closes #10374 ### What * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: #11557) ## Screenshots N/A ## Testing N/A # CG: Move to single nuget source (#11855) ## Description CG is failing builds due to multiple nuget feeds in nuget.config. ### Why What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves #11557
This PR backports PRs microsoft#11548 and microsoft#11855 to 0.69. Resolves Secure Supply Chain warnings. - Bug fix (non-breaking change which fixes an issue) To meet compliance requirements. Closes microsoft#11209 Closes microsoft#10374 * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: microsoft#11557) N/A N/A CG is failing builds due to multiple nuget feeds in nuget.config. What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves microsoft#11557 # Please enter the commit message for your changes. Lines starting # with '#' will be kept; you may remove them yourself if you want to. # An empty message aborts the commit. # # Author: Jon Thysell <jthysell@microsoft.com> # Date: Mon Aug 14 11:55:01 2023 -0700 # # On branch securefeed69 # Changes to be committed: # modified: .gitignore # new file: GuardianCustomConfiguration.json # modified: NuGet.Config # new file: change/react-native-windows-14a87dda-f285-4cd4-8a0f-b5aa9543cb9f.json # new file: change/react-native-windows-bfb79e47-04d8-4f4e-a72f-26b1621873d2.json # deleted: packages/e2e-test-app/.npmrc # deleted: packages/e2e-test-app/windows/NuGet.Config # deleted: packages/integration-test-app/windows/NuGet.Config # deleted: packages/playground/windows/NuGet.Config # deleted: packages/sample-apps/.npmrc # deleted: packages/sample-apps/windows/NuGet.Config # deleted: vnext/.npmrc # modified: vnext/Microsoft.ReactNative.Managed.UnitTests/packages.lock.json # modified: vnext/Microsoft.ReactNative.Managed/packages.lock.json # deleted: vnext/NuGet.Config #
This PR backports PRs #11548 and #11855 to 0.71. # Fix Secure Supply Chain compliance issues (#11548) ## Description Resolves Secure Supply Chain warnings. ### Type of Change - Bug fix (non-breaking change which fixes an issue) ### Why To meet compliance requirements. Closes #11209 Closes #10374 ### What * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: #11557) ## Screenshots N/A ## Testing N/A # CG: Move to single nuget source (#11855) ## Description CG is failing builds due to multiple nuget feeds in nuget.config. ### Why What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves #11557
This PR backports PRs #11548 and #11855 to 0.69. # Fix Secure Supply Chain compliance issues (#11548) ## Description Resolves Secure Supply Chain warnings. ### Type of Change - Bug fix (non-breaking change which fixes an issue) ### Why To meet compliance requirements. Closes #11209 Closes #10374 ### What * Removed invalid `nuget.config` configurations * Removed unnecessary `.npmrc` files * Temporarily left nuget.org feed in config (tracking: #11557) ## Screenshots N/A ## Testing N/A # CG: Move to single nuget source (#11855) ## Description CG is failing builds due to multiple nuget feeds in nuget.config. ### Why What is the motivation for this change? Add a few sentences describing the context and overall goals of the pull request's commits. Resolves #11557
This PR backports PRs microsoft#9990, microsoft#11548 and microsoft#11855 to 0.68 to resolve Secure Supply Chain warnings.
This PR backports PRs microsoft#9990, microsoft#11548 and microsoft#11855 to 0.67 to resolve Secure Supply Chain warnings.
Description
Resolves Secure Supply Chain warnings.
Type of Change
Why
To meet compliance requirements.
Closes #11209
Closes #10374
What
nuget.config
configurations.npmrc
filesScreenshots
N/A
Testing
N/A
Microsoft Reviewers: Open in CodeFlow