Skip to content

Conversation

@akoeplinger
Copy link
Member

@akoeplinger akoeplinger commented Oct 14, 2025

The api.nuget.org source will clash with the CFSClean network isolation policy and the same data is provided by AzDO now.

The api.nuget.org source will clash with the CFSClean network isolation policy
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the audit sources configuration from NuGet.config to resolve network isolation policy conflicts. The api.nuget.org source in the audit configuration was clashing with the CFSClean network isolation policy.

Key Changes

  • Removed the entire <auditSources> section from NuGet.config
  • Eliminated the reference to api.nuget.org that was causing network policy conflicts

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Oct 14, 2025
@akoeplinger akoeplinger added area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Oct 14, 2025
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@agocke agocke merged commit f9fce60 into main Oct 15, 2025
151 of 153 checks passed
@agocke agocke deleted the akoeplinger-patch-2 branch October 15, 2025 01:10
@akoeplinger
Copy link
Member Author

/backport to release/10.0

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Started backporting to release/10.0 (link to workflow run)

@akoeplinger
Copy link
Member Author

/backport to release/9.0-staging

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Started backporting to release/9.0-staging (link to workflow run)

akoeplinger added a commit that referenced this pull request Nov 7, 2025
Backport of #120696 to release/9.0-staging

/cc @akoeplinger

## Customer Impact

- [ ] Customer reported
- [X] Found internally

no customer impact, this affects our internal builds due to CFS
restrictions.

## Regression

- [ ] Yes
- [X] No


## Testing

Tested in CI.


## Risk

None. This just changes a setting for nuget package restore.

**IMPORTANT**: If this backport is for a servicing release, please
verify that:

- For .NET 8 and .NET 9: The PR target branch is `release/X.0-staging`,
not `release/X.0`.
- For .NET 10+: The PR target branch is `release/X.0` (no `-staging`
suffix).

## Package authoring no longer needed in .NET 9

**IMPORTANT**: Starting with .NET 9, you no longer need to edit a NuGet
package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older
versions.

Co-authored-by: Alexander Köplinger <[email protected]>
hoyosjs pushed a commit that referenced this pull request Nov 7, 2025
Backport of #120696 to release/10.0

/cc @akoeplinger

## Customer Impact

- [ ] Customer reported
- [x] Found internally

no customer impact, this affects our internal builds due to CFS
restrictions.

## Regression

- [ ] Yes
- [x] No

## Testing

Tested in CI.

## Risk

None. This just changes a setting for nuget package restore.

**IMPORTANT**: If this backport is for a servicing release, please
verify that:

- For .NET 8 and .NET 9: The PR target branch is `release/X.0-staging`,
not `release/X.0`.
- For .NET 10+: The PR target branch is `release/X.0` (no `-staging`
suffix).

## Package authoring no longer needed in .NET 9

**IMPORTANT**: Starting with .NET 9, you no longer need to edit a NuGet
package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older
versions.

Co-authored-by: Alexander Köplinger <[email protected]>
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants