Skip to content

Test#345

Draft
hjyamauchi wants to merge 1 commit into
mainfrom
hiroshi/gha
Draft

Test#345
hjyamauchi wants to merge 1 commit into
mainfrom
hiroshi/gha

Conversation

@hjyamauchi
Copy link
Copy Markdown
Contributor

No description provided.

@semgrep-code-thebrowsercompany
Copy link
Copy Markdown

Semgrep found 1 run-shell-injection finding:

Using variable interpolation ${{...}} with github context data in a run: step could allow an attacker to inject their own code into the runner. This would allow them to steal secrets and code. github context data can have arbitrary user input and should be treated as untrusted. Instead, use an intermediate environment variable with env: to store the data and use the environment variable in the run: script. Be sure to use double-quotes the environment variable, like this: "$ENVVAR".

@semgrep-code-thebrowsercompany
Copy link
Copy Markdown

Semgrep found 1 run-shell-injection finding:

Using variable interpolation ${{...}} with github context data in a run: step could allow an attacker to inject their own code into the runner. This would allow them to steal secrets and code. github context data can have arbitrary user input and should be treated as untrusted. Instead, use an intermediate environment variable with env: to store the data and use the environment variable in the run: script. Be sure to use double-quotes the environment variable, like this: "$ENVVAR".

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant