Skip to content

Properly escape Helix queue parameters in runtime-diagnostics - #125877

Merged
hoyosjs merged 1 commit into
mainfrom
hoyosjs-patch-2
Mar 21, 2026
Merged

Properly escape Helix queue parameters in runtime-diagnostics#125877
hoyosjs merged 1 commit into
mainfrom
hoyosjs-patch-2

Conversation

@hoyosjs

Copy link
Copy Markdown
Member

No description provided.

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @steveisok, @tommcdon, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

CopilotAI 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

Updates the runtime-diagnostics Azure Pipelines definition to ensure Helix queue values are safely passed through PowerShell/Bash invocation when they contain characters (notably ;) that would otherwise be interpreted by the shell.

Changes:

  • Quote /p:HelixTargetQueues in the “Send cDAC Dump Tests to Helix” step.
  • Quote /p:HelixTargetQueues in the “Send cDAC Dump Gen to Helix” step.
  • Quote /p:HelixTargetQueues in the “Send cDAC X-Plat Dump Tests to Helix” step.

@hoyosjs
hoyosjs merged commit 9ddca1d into mainMar 21, 2026
171 of 173 checks passed
@hoyosjs
hoyosjs deleted the hoyosjs-patch-2 branch March 21, 2026 06:39
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Apr 20, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@hoyosjs@steveisok