Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(gh): update release #1428

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tenthirtyam
Copy link
Collaborator

Summary of Pull Request

Type of Pull Request

  • This is a bug fix.
  • This is an enhancement or feature.
  • This is a code style/formatting update.
  • This is a documentation update.
  • This is a refactoring update.
  • This is a chore update
  • This is something else.
    Please describe:

Related to Existing Issues

Test and Documentation Coverage

For bug fixes or features:

  • Tests have been completed.
  • Documentation has been added/updated.

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

@tenthirtyam tenthirtyam added chore Chore github-actions GitHub Actions labels Oct 13, 2024
@tenthirtyam tenthirtyam requested a review from annakhm October 13, 2024 16:23
@tenthirtyam tenthirtyam self-assigned this Oct 13, 2024
@tenthirtyam tenthirtyam force-pushed the chore(gh)/update-release branch 2 times, most recently from 5084653 to 4a440ec Compare October 13, 2024 16:33
@tenthirtyam tenthirtyam added needs-review Needs Review size/xs Relative Sizing: extra-small labels Oct 14, 2024
- Correct formatting.
- Pin GitHub Actions to release commit SHA.

Signed-off-by: Ryan Johnson <[email protected]>
@tenthirtyam tenthirtyam force-pushed the chore(gh)/update-release branch from 4a440ec to 1ce748d Compare October 14, 2024 19:20
@ksamoray
Copy link
Collaborator

@tenthirtyam what's the advantage of specifying the version in this way? Are you concerned that compatibility will break by GH action changes?

@tenthirtyam
Copy link
Collaborator Author

From the GH docs: "Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps mitigate the risk of a bad actor adding a backdoor to the action's repository, as they would need to generate a SHA-1 collision for a valid Git object payload. When selecting a SHA, you should verify it is from the action's repository and not a repository fork."

@ksamoray
Copy link
Collaborator

Yeah I understand that you intend to enforce a specific sub-version of the action. The question is why? As if anyone fixes anything we won't see it. Are you worried about compatibility?

@tenthirtyam
Copy link
Collaborator Author

Using a specific commit SHA when specifying a GitHub Action ensures the workflow runs against a known state of the action. This guarantees reliability, stability, and security by ensuring that the exact code specified is used, even if the version tag is modified later.

Dependabot will then automatically manage open PRs to update actions using the @<commit-sha> #vX.Y.Z format, providing detailed information on the changes in the action’s release.

@tenthirtyam tenthirtyam requested review from a team and removed request for annakhm October 28, 2024 21:33
@tenthirtyam
Copy link
Collaborator Author

@vmware/terraform-provider-nsxt-maintainers: reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore github-actions GitHub Actions needs-review Needs Review size/xs Relative Sizing: extra-small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants