Skip to content

Dependabot Auto-merge: Fix code injection alert#45

Merged
tangrufus merged 1 commit intomainfrom
inject
Apr 11, 2026
Merged

Dependabot Auto-merge: Fix code injection alert#45
tangrufus merged 1 commit intomainfrom
inject

Conversation

@tangrufus
Copy link
Copy Markdown
Member

No description provided.

@tangrufus tangrufus added the enhancement New feature or request label Apr 11, 2026
Copilot AI review requested due to automatic review settings April 11, 2026 18:50
@tangrufus tangrufus enabled auto-merge (squash) April 11, 2026 18:50
@tangrufus tangrufus merged commit 663e911 into main Apr 11, 2026
3 checks passed
@tangrufus tangrufus deleted the inject branch April 11, 2026 18:50
|| (inputs.minor && steps.dependabot-metadata.outputs.update-type == 'version-update:semver-minor')
|| (inputs.patch && steps.dependabot-metadata.outputs.update-type == 'version-update:semver-patch')
run: gh pr merge --auto --${{ inputs.strategy }} "$PR_URL"
run: gh pr merge --auto "--${{ inputs.strategy }}" "$PR_URL"
Copy link
Copy Markdown

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

Updates the Dependabot auto-merge reusable workflow to mitigate a potential shell/code-injection vector when passing the merge strategy into the gh pr merge command.

Changes:

  • Quote the --${{ inputs.strategy }} flag when invoking gh pr merge to prevent unsafe shell interpretation.

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

@tangrufus tangrufus added security and removed enhancement New feature or request labels Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants