Skip to content

chore: add static-check security gate (trivy-fs + gitleaks)#72

Merged
AndriyKalashnykov merged 1 commit into
masterfrom
chore/static-check-security-gate
Jun 16, 2026
Merged

chore: add static-check security gate (trivy-fs + gitleaks)#72
AndriyKalashnykov merged 1 commit into
masterfrom
chore/static-check-security-gate

Conversation

@AndriyKalashnykov

Copy link
Copy Markdown
Owner

Follow-up to #71 — adds a real static-check security gate (the deferred MEDIUM finding).

Changes

  • .mise.toml — pin aqua:aquasecurity/trivy 0.71.1 + aqua:gitleaks/gitleaks 8.30.1.
  • Makefiletrivy-fs (vuln+secret scan, HIGH/CRITICAL, exit 1), gitleaks-scan (working-tree secret scan), and a static-check composite (lint + trivy-fs + gitleaks-scan). make ci now runs static-check instead of bare lint.
  • ci.yml — new static-check job (gated on changes), added to the ci-pass aggregator's needs.

Verification

  • make static-check → exit 0 (lint ✓, trivy: 0 vulns/0 secrets ✓, gitleaks: no leaks ✓)
  • actionlint ci.yml clean ✓ · YAML valid ✓

Scanners verified clean against the current tree, so the gate is blocking (not report-only). No .trivyignore waivers needed.

Note: Checkstyle/google-java-format intentionally not added — it would force restyling the demo's existing servlet code (churn, not a security fix). Can be a separate PR if desired.

- .mise.toml: pin aqua:aquasecurity/trivy + aqua:gitleaks/gitleaks
- Makefile: add trivy-fs (vuln+secret scan), gitleaks-scan, and a
  static-check composite (lint + trivy-fs + gitleaks); ci now runs
  static-check instead of bare lint
- ci.yml: add static-check job (gated on changes), add to ci-pass needs

Both scanners verified clean against the current tree (0 vulns, 0 secrets,
no leaks), so the gate is blocking.
@AndriyKalashnykov AndriyKalashnykov merged commit 94724b7 into master Jun 16, 2026
9 checks passed
@AndriyKalashnykov AndriyKalashnykov deleted the chore/static-check-security-gate branch June 16, 2026 04:11
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