Skip to content

CI: remove manual PowerShell install from test-proxy job (e2e-tests.yml)#703

Open
priya-kinthali wants to merge 4 commits intoactions:mainfrom
priya-kinthali:remove-powershell-manual-install-proxy-job
Open

CI: remove manual PowerShell install from test-proxy job (e2e-tests.yml)#703
priya-kinthali wants to merge 4 commits intoactions:mainfrom
priya-kinthali:remove-powershell-manual-install-proxy-job

Conversation

@priya-kinthali
Copy link
Contributor

Description:
This PR changes the test-proxy job to use mcr.microsoft.com/devcontainers/dotnet:10.0 container image instead of ubuntu:22.04 and removes the manual PowerShell installation step that used wget/dpkg, which was unsafe due to potential malicious .deb file substitution.

Related issue:
Add link to the related issue.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

Copilot AI review requested due to automatic review settings February 25, 2026 12:35
@priya-kinthali priya-kinthali requested a review from a team as a code owner February 25, 2026 12:35
Copy link

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 enhances the security of the test-proxy job in the e2e tests workflow by replacing the ubuntu:22.04 base container image with mcr.microsoft.com/devcontainers/dotnet:10.0, which comes with PowerShell pre-installed. This change eliminates the need for a manual PowerShell installation step that previously downloaded and installed packages using wget and dpkg, which posed a security risk due to potential malicious package substitution.

Changes:

  • Switched container image from ubuntu:22.04 to mcr.microsoft.com/devcontainers/dotnet:10.0
  • Removed the manual PowerShell installation step that used insecure wget/dpkg commands

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants