Update pull request template for input validation checklist (#41373)
Expand the validation checklist to include validation of untrusted data in shell scripts.
This commit is contained in:
@@ -8,7 +8,7 @@ If some of the following don't apply, delete the relevant line.
|
||||
- [ ] Changes file added for user-visible changes in `changes/`, `orbit/changes/` or `ee/fleetd-chrome/changes`.
|
||||
See [Changes files](https://github.com/fleetdm/fleet/blob/main/docs/Contributing/guides/committing-changes.md#changes-files) for more information.
|
||||
|
||||
- [ ] Input data is properly validated, `SELECT *` is avoided, SQL injection is prevented (using placeholders for values in statements), JS inline code is prevented especially for url redirects
|
||||
- [ ] Input data is properly validated, `SELECT *` is avoided, SQL injection is prevented (using placeholders for values in statements), JS inline code is prevented especially for url redirects, and untrusted data interpolated into shell scripts/commands is validated against shell metacharacters.
|
||||
- [ ] If paths of existing endpoints are modified without backwards compatibility, checked the frontend/CLI for any necessary changes
|
||||
|
||||
## Testing
|
||||
|
||||
Reference in New Issue
Block a user