Skip to content

[release/v7.4.18] Avoid calling credential provider for public feed for Wix#27664

Merged
SeeminglyScience merged 1 commit into
PowerShell:release/v7.4.18from
SeeminglyScience:backport/release/v7.4.18/27663-26757e260
Jul 8, 2026
Merged

[release/v7.4.18] Avoid calling credential provider for public feed for Wix#27664
SeeminglyScience merged 1 commit into
PowerShell:release/v7.4.18from
SeeminglyScience:backport/release/v7.4.18/27663-26757e260

Conversation

@SeeminglyScience

Copy link
Copy Markdown
Contributor

Backport of #27663 to release/v7.4.18

Triggered by @SeeminglyScience on behalf of @adityapatwardhan

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

This backport updates the Wix packaging helper so the public dotnet-eng feed is marked with CredentialProvider None on release/v7.4.18, preventing unnecessary credential provider prompts during packaging.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Validated the cherry-pick on release/v7.4.18 and ran PowerShell syntax validation on tools/wix/wix.psm1. The backport is limited to the single Wix helper change from the original PR.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

The change is narrowly scoped to a single repository registration path in tools/wix/wix.psm1 and only adjusts how the public feed is configured. It does not alter build logic outside that path.

Copilot AI review requested due to automatic review settings July 7, 2026 21:37
@SeeminglyScience
SeeminglyScience requested a review from a team as a code owner July 7, 2026 21:37
@SeeminglyScience SeeminglyScience added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Jul 7, 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 backports a packaging reliability fix to the release/v7.4.18 branch by configuring the dotnet-eng PSResourceGet repository to avoid invoking any credential provider when the feed is public (Wix packaging scenario).

Changes:

  • After registering the dotnet-eng repository, explicitly set -CredentialProvider None via Set-PSResourceRepository to prevent credential provider prompts for the public feed.

@SeeminglyScience
SeeminglyScience enabled auto-merge (squash) July 7, 2026 23:50
@SeeminglyScience
SeeminglyScience merged commit 695d7e5 into PowerShell:release/v7.4.18 Jul 8, 2026
50 of 52 checks passed
@SeeminglyScience
SeeminglyScience deleted the backport/release/v7.4.18/27663-26757e260 branch July 8, 2026 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants