Avoid shell injection in examples
Why:
Avoid shell injection in examples to follow suggestions from https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#understanding-the-risk-of-script-injections.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Changed examples that used shell injection. Removed a paragraph that explained the injection poorly. Clarified multi-line string syntax warning and revert to simple delimiter use in the example.
Check off the following:
-
[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.
- For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the
datadirectory.
- For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the
-
[x] For content changes, I have completed the self-review checklist.
Automatically generated comment âšī¸
This comment is automatically generated and will be overwritten every time changes are committed to this branch.
The table contains an overview of files in the content directory that have been changed in this pull request. It's provided to make it easy to review your changes on the staging site. Please note that changes to the data directory will not show up in this table.
Content directory changes
You may find it useful to copy this table into the pull request summary. There you can edit it to share links to important articles or changes and to give a high-level overview of how the changes in your pull request support the overall goals of the pull request.
| Source | Preview | Production | What Changed |
|---|---|---|---|
actions/using-workflows/workflow-commands-for-github-actions.md |
fpt ghec ghes@ 3.10 3.9 3.8 3.7 3.6 ghae |
fpt ghec ghes@ 3.10 3.9 3.8 3.7 3.6 ghae |
fpt: Free, Pro, Team ghec: GitHub Enterprise Cloud ghes: GitHub Enterprise Server ghae: GitHub AE
@soliton- Good to see another PR from you! đ
Thanks for the submission. I'll get it triaged for review. âĄ
@soliton- Quick question as I'm triaging it, you mentioned removing a paragraph - do the updates you've made render that paragraph unnecessary? Or do you have a suggestion for improving the paragraph instead of removing it altogether?
@cmwilson21 Yes, the paragraph was about the shell injection that gets removed in this PR.
@soliton- I follow now, thank you for confirming!
Is there any update on this? Anything I can do?
@soliton- Thanks for checking in! It is up on the board for review. We appreciate your patience as we are working through our backlog đ
@soliton- Thanks again. We've got a merge freeze just now due to an upcoming release, but when that's done we'll get this merged and published. đ
Thanks very much for contributing! Your pull request has been merged đ You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues :zap: