Skip to content

ci: allow reviewed historical gitleaks fixtures - #25

Merged
sumitake merged 1 commit into
mainfrom
dev/codex/scheduled-gitleaks
Aug 10, 2026
Merged

ci: allow reviewed historical gitleaks fixtures#25
sumitake merged 1 commit into
mainfrom
dev/codex/scheduled-gitleaks

Conversation

@sumitake

Copy link
Copy Markdown
Owner

Summary

  • allow the five exact merge-commit fingerprints reported by the scheduled full-history Gitleaks scan
  • keep suppression limited to reviewed synthetic fixtures and deterministic test digests
  • extend the repository contract so all six approved fingerprints remain exact and no broad Gitleaks allowlist is introduced

Test plan

  • python3 -m unittest discover -s tests -p 'test_*.py' passes (290 tests)
  • Relevant updated contract test was observed failing before the allowlist change and passing afterward
  • python3 scripts/check_repository_metadata.py passes
  • python3 scripts/sanitize_public.py --tracked --history passes
  • Gitleaks 8.24.3 full-history detection reports no leaks

PUBLIC-SAFETY checklist

  • This PR contains no deployment identifiers (account IDs, zone IDs, tunnel IDs, installation/client/app IDs, or any other environment-specific identifier).
  • This PR contains no secrets (tokens, keys, credentials, passwords, or anything secret-shaped), real or expired.
  • This PR contains no real logs, real configuration, or real runtime state -- only synthetic examples.
  • I ran python3 scripts/sanitize_public.py --tracked locally and it reported sanitization passed.
  • I ran python3 scripts/check_repository_metadata.py locally and it exited 0.

Additional context

The failure was limited to the scheduled scan because that event scans merge history. Push and pull-request scans had already accepted the corresponding pre-merge fixture fingerprint.

@sumitake
sumitake merged commit 89d95de into main Aug 10, 2026
18 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