Skip to content

ci: add security-scan + semgrep + dependabot#4

Merged
Cre-eD merged 4 commits intomainfrom
ci/security-scans
May 7, 2026
Merged

ci: add security-scan + semgrep + dependabot#4
Cre-eD merged 4 commits intomainfrom
ci/security-scans

Conversation

@Cre-eD
Copy link
Copy Markdown
Contributor

@Cre-eD Cre-eD commented May 7, 2026

Summary

  • Adds 4 wrapper workflows that delegate to simple-container-com/actions:
    • security-scan — TruffleHog + Syft + Trivy + Grype, sticky PR comment, status gate
    • semgrep — SC custom ruleset + optional consumer rules / registry packs, sticky PR comment, status gate
  • Adds .github/dependabot.yml for weekly bumps (github-actions + gomod where applicable, minor+patch grouped)

Trust posture

Scan jobs run on pull_request (not _target), read-only token, no secrets — safe for fork PRs. Comment posting lives in a separate workflow_run-triggered workflow that never reads PR code.

Test plan

  • First scan + comment runs end-to-end
  • Sticky PR comment appears with status table
  • No false positives on the existing codebase

Cre-eD added 2 commits May 7, 2026 16:16
Wires this repo to the org-wide reusable workflows in simple-container-com/actions:
- security-scan: TruffleHog (secrets) + Syft/CycloneDX SBOM + Trivy + Grype, sticky PR comment, status gate
- semgrep: SC custom ruleset + optional consumer rules / registry packs, sticky PR comment, status gate

Both follow the GitHub Security Lab 'preventing pwn requests' split: scan jobs run in pull_request context (read-only token, no secrets, fork-PR safe); comment posting lives in a workflow_run-triggered job that never reads PR code.

Adds .github/dependabot.yml with weekly bumps for both github-actions and gomod ecosystems, with minor+patch grouped to reduce PR noise.
@Cre-eD Cre-eD force-pushed the ci/security-scans branch from 0355054 to ce6676a Compare May 7, 2026 12:30
@Cre-eD Cre-eD self-assigned this May 7, 2026
@Cre-eD Cre-eD force-pushed the ci/security-scans branch from ce6676a to db9eebb Compare May 7, 2026 12:39
Cre-eD added 2 commits May 7, 2026 16:54
Signed-off-by: Dmitrii Creed <creeed22@gmail.com>
…otstrap

Six findings from the first Semgrep scan on this repo:

ERROR (gating, fixed by suppression with justification):
- .github/workflows/push.yaml:48,49 — bash <(curl ...) bootstrap of welder.simple-container.com/welder.sh. This is the org's own service, not third-party. Added 'nosemgrep: shell-curl-pipe-to-shell' inline with a justification comment that the long-term fix is a checksum-pinned welder release (tracked as a follow-up).

WARNING (best-practice, fixed by pinning):
- push.yaml:25 reecetech/version-increment@2023.10.2 -> @a29aa752dc3b8118a2dc2ed93faf0e95a73a9c7e (2024.10.1)
- push.yaml:41 fregante/setup-git-user@v2 -> @024bc0b8e177d7e77203b48dab6fb45666854b35 (v2.0.2)

INFO (defence-in-depth, fixed by pinning):
- push.yaml:23,40 actions/checkout@v4 -> @de0fac2e4500dabe0009e67214ff5f5447ce83dd (v6.0.2)

Signed-off-by: Dmitrii Creed <creeed22@gmail.com>
@Cre-eD Cre-eD requested a review from smecsia May 7, 2026 13:26
@Cre-eD Cre-eD merged commit e83b1ab into main May 7, 2026
7 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.

2 participants