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

Improve gha #318

Merged
merged 5 commits into from Dec 4, 2023
Merged

Improve gha #318

merged 5 commits into from Dec 4, 2023

Conversation

kairoaraujo
Copy link
Collaborator

  • Fix the github organization name to in-toto.

  • pin GHA full-length commit SHA

It's a good practice to pin the GitHub Actions with full-length
commit sha as described in the "Security hardening for GitHub
Actions".
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions

I noticed that dependabot also checks weekly with updates.
It will maintain the updates using the hash.

Fix the github organization name to in-toto.

Signed-off-by: Kairo de Araujo <kairo.araujo@testifysec.com>
It's a good practice to pin the GitHub Actions with full-length
commit sha as described in the "Security hardening for GitHub
Actions".
https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions

I noticed that dependabot also checks weekly with updates.
It will maintain the updates using the hash.

Signed-off-by: Kairo de Araujo <kairo.araujo@testifysec.com>
@@ -71,7 +71,7 @@ jobs:
run: go install github.com/goreleaser/goreleaser@latest

- name: Run GoReleaser
uses: testifysec/witness-run-action@40aa4ef36fc431a37de7c3faebcb66513c03b934
uses: in-toto/witness-run-action@40aa4ef36fc431a37de7c3faebcb66513c03b934
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the action has not been moved to @in-toto.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I pushed it without my last amend :|

@kairoaraujo
Copy link
Collaborator Author

Should we also make it pinned?

- name: Install addlicense
run: go install github.com/google/addlicense@latest

Copy link
Member

@adityasaky adityasaky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good apart from the action URL!

@@ -56,7 +56,7 @@ jobs:
path: /tmp

- if: ${{ inputs.pre-command != '' && inputs.pull_request == false }}
uses: testifysec/witness-run-action@40aa4ef36fc431a37de7c3faebcb66513c03b934
uses: in-toto/witness-run-action@40aa4ef36fc431a37de7c3faebcb66513c03b934
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the name of the action remains an issue here, as it's not been moved to the in-toto org.

@adityasaky
Copy link
Member

Should we also make it pinned?

I think dependabot may not handle that?

@kairoaraujo
Copy link
Collaborator Author

kairoaraujo commented Dec 4, 2023

I think dependabot may not handle that?

No, it will not handle it.

@ChaosInTheCRD
Copy link
Collaborator

I think dependabot may not handle that?

No, it will not handle it.

Could we at least semver pin it? @v1.1.1 or something?

@kairoaraujo
Copy link
Collaborator Author

I see that my work here duplicates #316.

adityasaky
adityasaky previously approved these changes Dec 4, 2023
@jkjell jkjell merged commit 81bdfce into in-toto:main Dec 4, 2023
10 checks passed
Sign up for free to 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.

None yet

4 participants