Skip to content

ci(secret-scan): allowlist bundle compatibility fixtures for commit b0fd15ee in .gitleaksignore - #2168

Open
yunusarda-sudo wants to merge 1 commit into
OpenAdaptAI:mainfrom
yunusarda-sudo:fix/gitleaksignore-allowlist-squashed-bundle-fixtures
Open

yunusarda-sudo wants to merge 1 commit into
OpenAdaptAI:mainfrom
yunusarda-sudo:fix/gitleaksignore-allowlist-squashed-bundle-fixtures

Conversation

@yunusarda-sudo

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

CI/CD bug fix

Summary

Commit b0fd15e (#1928) introduced bundle compatibility fixtures under packages/flow/tests/fixtures/bundle_compatibility/ and allowlisted 33 fingerprints in .gitleaksignore under pre-merge branch commit 8ccf2cf3360b7d7720af04c36eee4f8e756bdf6e.

When #1928 merged into main as commit b0fd15ee, the commit hash changed. As a result, subsequent pull requests branched from or rebased onto main trigger 33 unfiltered findings in secret-scan.yml's gitleaks step on commit b0fd15ee.

This PR adds the 33 corresponding fingerprints with commit hash b0fd15ee8584cc3b646ae9bab04b4f9acbc59292 so that secret scanning on PRs ignores these published, non-credential bundle fixtures.

Checklist

  • I have tested this change locally
  • Documentation is updated if needed
  • CI passes

Notes

Investigated and verified with Antigravity / Claude Code with local test verification (tests/test_gitleaks_metadata.py passes 20/20, local gitleaks test reports 0 findings on commit b0fd15e).

…0fd15e in .gitleaksignore

Commit b0fd15e (OpenAdaptAI#1928) introduced bundle compatibility fixtures in
packages/flow/tests/fixtures/bundle_compatibility/ and added 33 fingerprints to
.gitleaksignore under the pre-merge branch commit 8ccf2cf.
When the PR was merged into main as b0fd15e, the commit hash changed.
As a result, subsequent pull requests whose history includes b0fd15e trigger 33
unfiltered findings in secret-scan.yml's gitleaks step.

Add the 33 corresponding fingerprints with commit b0fd15e
so secret scanning ignores these published, non-credential bundle fixtures.

This branch has not been deployed

No deployments
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