Skip to content

[release/v7.5.9] Avoid calling credential provider for public feed for Wix#27665

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

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

Conversation

@SeeminglyScience

Copy link
Copy Markdown
Contributor

Backport of #27663 to release/v7.5.9

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.5.9, 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.5.9 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.

@SeeminglyScience
SeeminglyScience requested a review from a team as a code owner July 7, 2026 21:38
@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 review requested due to automatic review settings July 7, 2026 21:38

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

Backport to release/v7.5.9 that updates the WiX packaging helper to ensure the public dotnet-eng PSResourceGet feed is configured to not invoke a credential provider, avoiding unnecessary authentication prompts during packaging.

Changes:

  • After registering the dotnet-eng repository, explicitly set its credential provider to None.

Comment thread tools/wix/wix.psm1
Comment on lines 45 to 47
$respository = Get-PSResourceRepository -Name 'dotnet-eng' -ErrorAction SilentlyContinue

if (-not $respository) {
@SeeminglyScience
SeeminglyScience merged commit dfbef7b into PowerShell:release/v7.5.9 Jul 7, 2026
38 checks passed
@SeeminglyScience
SeeminglyScience deleted the backport/release/v7.5.9/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