*: fix potential code injection - #4598
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4598 +/- ##
==========================================
+ Coverage 57.58% 57.60% +0.01%
==========================================
Files 246 246
Lines 33675 33675
==========================================
+ Hits 19392 19397 +5
Misses 11843 11843
+ Partials 2440 2435 -5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Updates the patch-release GitHub Actions workflows to reduce the risk of expression/input-driven shell injection when propagating the target_minor workflow input into bash scripts.
Changes:
- Pass
inputs.target_minorvia a stepenvvariable (TARGET_MINOR_INPUT) instead of interpolating the expression directly inside therun:script. - Update the bash assignment to use
TARGET_MINOR="$TARGET_MINOR_INPUT"across patch-release steps 1–4.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/patch-release-step1.yml | Routes target_minor through env before using it in the version bump script to reduce injection risk. |
| .github/workflows/patch-release-step2.yml | Routes target_minor through env before using it in the branch selection logic to reduce injection risk. |
| .github/workflows/patch-release-step3.yml | Routes target_minor through env before using it in the branch selection logic to reduce injection risk. |
| .github/workflows/patch-release-step4.yml | Routes target_minor through env before using it in the branch selection logic to reduce injection risk. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.



As per the title
category: bug
ticket: none