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

Add gha-creds-*.json ignore #1644

Merged
merged 2 commits into from
Jun 28, 2024
Merged

Conversation

tstromberg
Copy link
Contributor

@tstromberg tstromberg commented Jun 27, 2024

Type of change

Internal bugfix.

What should this PR do?

Prevent short-term token leakage.

Why are we making this change?

It's required to securely use the google-github-actions/auth documentation: https://github.com/google-github-actions/auth - though no technical control is in place to ensure the existence of this entry.

See google-github-actions/auth#298 for more context.

What are the acceptance criteria?

That it doesn't break tests.

How should this PR be tested?

@tstromberg tstromberg requested a review from a team as a code owner June 27, 2024 11:22
Copy link

netlify bot commented Jun 27, 2024

Deploy Preview for ornate-narwhal-088216 ready!

Name Link
🔨 Latest commit d97fa01
🔍 Latest deploy log https://app.netlify.com/sites/ornate-narwhal-088216/deploys/667e16a796be670008df1889
😎 Deploy Preview https://deploy-preview-1644--ornate-narwhal-088216.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: ltagliaferri <lisa.tagliaferri@gmail.com>
@ltagliaferri ltagliaferri merged commit 0f0f487 into chainguard-dev:main Jun 28, 2024
1 check 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

3 participants