Skip to content

fix(security): harden GitHub Actions workflows against expression injection#773

Merged
mnencia merged 1 commit intomainfrom
dev/expression-injection
Mar 6, 2026
Merged

fix(security): harden GitHub Actions workflows against expression injection#773
mnencia merged 1 commit intomainfrom
dev/expression-injection

Conversation

@mnencia
Copy link
Member

@mnencia mnencia commented Mar 3, 2026

Move ${{ }} expressions from run: blocks into step-level env: blocks, then reference them as properly-quoted shell variables.

Part of cloudnative-pg/cloudnative-pg#10113

Assisted-by: Claude Opus 4.6

@mnencia mnencia requested a review from a team as a code owner March 3, 2026 17:26
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Mar 3, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 4, 2026
…ection

Move ${{ }} expressions from run: blocks into step-level env: blocks,
then reference them as properly-quoted shell variables.

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
@mnencia mnencia force-pushed the dev/expression-injection branch from a15e2dc to d9b5a73 Compare March 6, 2026 08:22
@mnencia mnencia merged commit ce7b761 into main Mar 6, 2026
7 of 8 checks passed
@mnencia mnencia deleted the dev/expression-injection branch March 6, 2026 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants