Skip to content

Conversation

@kevinmessiaen
Copy link
Member

Description

  • Use environment variables instead of direct interpolation for user inputs
  • Add proper quoting for environment variables in shell commands
  • Fix create-release.yml: use INPUT_VERSION env var for inputs.version
  • Fix retry-workflow.yml: use RUN_ID env var for inputs.run_id

Resolves high-severity security vulnerability in workflow variable interpolation

Related Issue

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix

Checklist

  • I've read the CODE_OF_CONDUCT.md document.
  • I've read the CONTRIBUTING.md guide.
  • I've written tests for all new methods and classes that I created.
  • I've written the docstring in Google format for all the methods and classes that I used.
  • I've updated the pdm.lock running pdm update-lock (only applicable when pyproject.toml has been
    modified)

- Use environment variables instead of direct interpolation for user inputs
- Add proper quoting for environment variables in shell commands
- Fix create-release.yml: use INPUT_VERSION env var for inputs.version
- Fix retry-workflow.yml: use RUN_ID env var for inputs.run_id

Resolves high-severity security vulnerability in workflow variable interpolation
@kevinmessiaen kevinmessiaen requested a review from mattbit October 23, 2025 10:00
@linear
Copy link

linear bot commented Oct 23, 2025

@sonarqubecloud
Copy link

@kevinmessiaen kevinmessiaen changed the title fix(security): prevent command injection in GitHub Actions workflows fix(security): prevent command injection in GitHub Actions workflows [ENG-1131] Nov 6, 2025
@linear
Copy link

linear bot commented Nov 6, 2025

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants