Skip to content

SK-3017-gitleaks-detection-multiple-secret-exposures-identified - #336

Merged
Devesh-Skyflow merged 1 commit into
mainfrom
devesh/SK-3017
Aug 31, 2026
Merged

SK-3017-gitleaks-detection-multiple-secret-exposures-identified#336
Devesh-Skyflow merged 1 commit into
mainfrom
devesh/SK-3017

Conversation

@Devesh-Skyflow

@Devesh-Skyflow Devesh-Skyflow commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Why:

The org-wide Gitleaks scan flagged 9 generic-api-key findings — UUID-format sample/reveal tokens used in README.md's documentation code blocks and in a reveal-label test file. These are illustrative example values, not real credentials, but the findings need to be resolved to close out the security review.

Note: the repo was restructured since the original scan (README.md was split into Skyflow/README.md and SkyflowFlowVault/README.md; the test file moved under SkyflowFlowVault/Tests/). This PR's fingerprints target the current paths/line numbers on main for the same flagged values.

Goal:

Suppress the false-positive findings without altering the visible README/test content, by adding a .gitleaksignore file with fingerprints for the specific findings — Gitleaks' native, built-in suppression mechanism for exactly this case.

Ref: same pattern as skyflow-react-native#157.

Jira: SK-3017

🤖 Generated with Claude Code

…E/tests

Adds .gitleaksignore with fingerprints for the 9 flagged generic-api-key
findings — UUID-format sample/reveal tokens used in Skyflow/README.md's
documentation code blocks and in RevealSkyflowLabelViewTests.swift, not
real credentials.

Note: the repo was restructured since the original scan (README.md split
into Skyflow/README.md and SkyflowFlowVault/README.md; the test file moved
under SkyflowFlowVault/Tests/). Fingerprints below target the current
paths/line numbers on main for the same flagged values.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Gitleaks Findings: No secrets detected. Safe to proceed!

@Devesh-Skyflow
Devesh-Skyflow merged commit 4684417 into main Aug 31, 2026
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