Skip to content

Harden workflow security#3175

Merged
ltagliaferri merged 1 commit intochainguard-dev:mainfrom
ltagliaferri:workflow-security-updates
Apr 6, 2026
Merged

Harden workflow security#3175
ltagliaferri merged 1 commit intochainguard-dev:mainfrom
ltagliaferri:workflow-security-updates

Conversation

@ltagliaferri
Copy link
Copy Markdown
Collaborator

Harden workflow security: use env vars, docker/login-action, and environment gate

  • Move all ${{ }} context values in run blocks to env variables to prevent shell injection
  • Replace echo | docker login with docker/login-action for secure credential handling
  • Add environment: documentation gate to compile-public-docs.yml
  • Switch GCS metadata generation from shell heredoc to Python json.dumps
  • Replace asset-swap release pattern in compile-docs.yml for immutable release compatibility
  • Remove persist-credentials: false from GCS workflow (needs git push)
  • Remove manual docker logout (handled by login-action post step)

Signed-off-by: ltagliaferri <lisa.tagliaferri@gmail.com>
@ltagliaferri ltagliaferri requested a review from a team as a code owner April 6, 2026 17:05
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 6, 2026

Deploy Preview for ornate-narwhal-088216 ready!

Name Link
🔨 Latest commit 99246fd
🔍 Latest deploy log https://app.netlify.com/projects/ornate-narwhal-088216/deploys/69d3e7cb7e349a000835cd7d
😎 Deploy Preview https://deploy-preview-3175--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 project configuration.

@ltagliaferri ltagliaferri merged commit 3d731c4 into chainguard-dev:main Apr 6, 2026
5 of 6 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.

1 participant