Commit d8875ac
authored
chore: Pin Github Actions to Commit Hashes (#64)
This PR includes automated changes from running
[frizbee](https://github.com/stacklok/frizbee) to pin Github Actions.
For more information on what this actually means, take a look at [our
documentation on
UnpinnedActions](https://github.com/betterment/claws?tab=readme-ov-file#unpinnedactions).
Because `frizbee` will be replacing tags with their corresponding commit
hash, this change should fundamentally be a no-op. This change just
makes our code more explicit about what we're using.
Note that because we're touching Github Workflow files in this PR, there
is a chance that [Claws](https://github.com/betterment/claws), our GHA
Static Analyzer, will find other issues in those files that were
introduced before this PR was created. These findings will need to be
addressed before this PR can be merged.
The Claws documentation has a good list for how to remediate each type
of finding, but for brevity, here are some of the commonly seen ones:
* RiskyTriggers: Fires when a workflow has workflow_dispatch or
pull_request_target; remediate by leaving comments explaning why, and
put an ignore statement
([example](https://github.com/Betterment/retail/blob/92e76923f4e5d51e8386301538e383883cd5eb57/.github/workflows/create_datadog_slo.yml#L2-L4))
* UnsafeCheckout: Fires when a workflow checks out user supplied code;
remediate by leaving a comment explaining how the user supplied code is
used and confirm it isn't executed
([example](https://github.com/Betterment/linda-test/blob/6cd5954c7c1d2bdb781239b6686b3a5dcbcd6fda/.github/workflows/claws_fork_friendly.yml#L50-L54))
For other findings and how to remediate them, check the [Claws
docs!](https://github.com/betterment/claws?tab=readme-ov-file#built-in-rules)1 parent 61a0a3b commit d8875ac
2 files changed
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
11 | | - | |
| 10 | + | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
28 | 28 | | |
29 | | - | |
| 29 | + | |
30 | 30 | | |
31 | 31 | | |
32 | | - | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
0 commit comments