Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[release/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo - #66115

#66115
Merged
wtgodbe merged 2 commits into
release/8.0dotnet/aspnetcore:release/8.0from
pat-migration/wif-mirror-within-azdo-release8dotnet/aspnetcore:pat-migration/wif-mirror-within-azdo-release8Copy head branch name to clipboard
Apr 8, 2026
Merged

[release/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo#66115
wtgodbe merged 2 commits into
release/8.0dotnet/aspnetcore:release/8.0from
pat-migration/wif-mirror-within-azdo-release8dotnet/aspnetcore:pat-migration/wif-mirror-within-azdo-release8Copy head branch name to clipboard

Conversation

@missymessa

Copy link
Copy Markdown
Member

Summary

Migrate the azure-pipelines-mirror-within-azdo.yml pipeline from using the dn-bot-dnceng-build-rw-code-rw PAT (from the Mirror-Credentials variable group) to the dnceng-build-rw-code-rw-wif Workload Identity Federation service connection.

This is the same change as #66074 (merged to main), ported to the release/8.0 branch.

Changes

  • Remove Mirror-Credentials variable group reference
  • New AzureCLI@2 step – mints an AzDO bearer token via az account get-access-token using the dnceng-build-rw-code-rw-wif WIF service connection and stores it as the secret pipeline variable WifAzdoToken
  • Clone step now uses header-based auth (http.extraheader) instead of PAT embedded in the URL
  • Push step now uses header-based auth as well

Related

  • Part of PAT migration work item WI 10139
  • Service connection: dnceng-build-rw-code-rw-wif (Entra app 21f66e0-bb35-4fd3-bc70-ba084d1e7a52)

@missymessa
missymessa requested review from a team and wtgodbe as code owners April 2, 2026 17:56
@missymessa missymessa added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Apr 2, 2026
Copilot AI review requested due to automatic review settings April 2, 2026 17:56
@dotnet-policy-service dotnet-policy-service Bot added this to the 8.0.x milestone Apr 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 migrates the internal AzDO “mirror-within-azdo” pipeline from using a stored PAT (via the Mirror-Credentials variable group) to using a Workload Identity Federation (WIF) service connection, aligning the release/8.0 branch with the already-merged change on main.

Changes:

  • Removed the Mirror-Credentials variable group dependency.
  • Added an AzureCLI@2 step to mint an AzDO bearer token via az account get-access-token using the dnceng-build-rw-code-rw-wif service connection and store it as a secret variable.
  • Updated git clone and git push to use header-based auth (http.https://dev.azure.com/.extraheader) instead of embedding credentials in the URL.

@missymessa

Copy link
Copy Markdown
Member Author

/azp run aspnetcore-ci

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@wtgodbe wtgodbe changed the title Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo [release/8.0] Replace dn-bot-dnceng-build-rw-code-rw PAT with WIF service connection in mirror-within-azdo Apr 8, 2026
@wtgodbe
wtgodbe merged commit 3b1b00d into release/8.0 Apr 8, 2026
25 checks passed
@wtgodbe
wtgodbe deleted the pat-migration/wif-mirror-within-azdo-release8 branch April 8, 2026 21:07
@dotnet-policy-service dotnet-policy-service Bot modified the milestones: 8.0.x, 8.0.27 Apr 8, 2026
This was referenced Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.