Skip to content

mitigating script injection attacks by passing issue body as env var - #42

Merged
stefanbuck merged 1 commit into
devfrom
add-issue-body-arg
Sep 19, 2022
Merged

mitigating script injection attacks by passing issue body as env var#42
stefanbuck merged 1 commit into
devfrom
add-issue-body-arg

Conversation

@stefanbuck

Copy link
Copy Markdown
Owner

To mitigate script injection attacks, github-issue-parser v3 will require workflow authors to pass the issue body as an argument. By doing so you will follow GitHub's Good practices for mitigating script injection attacks

- uses: stefanbuck/github-issue-parser@v3id: issue-parserwith:
issue-body: ${{ github.event.issue.body }} # requiredtemplate-path: .github/ISSUE_TEMPLATE/bug-report.yml # optional but recommended

@stefanbuck
stefanbuck merged commit 857c89d into devSep 19, 2022
@stefanbuck
stefanbuck deleted the add-issue-body-arg branch September 19, 2022 22:49
stefanbuck added a commit that referenced this pull request Oct 19, 2022
…v var (#42)
BREAKING CHANGE: Add `issue-body` argument which is required from v3 onwards
To mitigate script injection attacks, github-issue-parser v3 will require workflow authors to pass the issue body as an argument. By doing so you will follow GitHub's [Good practices for mitigating script injection attacks
](https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#good-practices-for-mitigating-script-injection-attacks)
```yml
- uses: stefanbuck/github-issue-parser@v3
id: issue-parser
with:
issue-body: ${{ github.event.issue.body }} # required
template-path: .github/ISSUE_TEMPLATE/bug-report.yml # optional but recommended
```
@stefanbuckstefanbuck mentioned this pull request Nov 2, 2022
Sign up for freeto 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

@stefanbuck