Skip to content

gitleaksignore: allowlist Percona MongoDB placeholders in nvidia-nvsentinel-v1.19.0 - #309

Closed
vishwanaths wants to merge 1 commit into
mainfrom
fix-gitleaks-nvsentinel-v1.19.0
Closed

gitleaksignore: allowlist Percona MongoDB placeholders in nvidia-nvsentinel-v1.19.0#309
vishwanaths wants to merge 1 commit into
mainfrom
fix-gitleaks-nvsentinel-v1.19.0

Conversation

@vishwanaths

Copy link
Copy Markdown
Collaborator

Summary

Adds two fingerprints to .gitleaksignore for the bundled Percona MongoDB (psmdb-db) chart's values.yaml in packs/nvidia-nvsentinel-v1.19.0/. Both are commented-out AWS S3/KMS backup config placeholders shipped by Percona upstream:

line 675: kmsKeyID: 1234abcd-12ab-34cd-56ef-1234567890ab
line 678: sseCustomerKey: Y3VzdG9tZXIta2V5   (base64 for "customer-key")

They are inert documentation examples inside upstream chart defaults and get flagged by the generic-api-key rule — false positives.

Unblocks the BulwarkGitLeaks check on #303.

Test plan

  • BulwarkGitLeaks passes on this PR (it does not add the leaked file itself, so should be a no-op scan)
  • After merge, PR NVIDIA NVSentinel v1.19.0 #303 re-runs BulwarkGitLeaks and passes with these fingerprints matched

…ers in nvidia-nvsentinel-v1.19.0

Add two fingerprints for the bundled psmdb-db chart's values.yaml. Both are
commented-out AWS S3/KMS backup config placeholders shipped by Percona:

  line 675: kmsKeyID: 1234abcd-12ab-34cd-56ef-1234567890ab
  line 678: sseCustomerKey: Y3VzdG9tZXIta2V5   (base64 for "customer-key")

Both are inert documentation examples inside upstream Percona chart defaults
and were flagged by the generic-api-key rule.

Unblocks the BulwarkGitLeaks check on #303.
@vishwanaths

Copy link
Copy Markdown
Collaborator Author

Superseded — the same fingerprints were pushed directly to PR #303's branch (commit c5a2066), so the fix lands with the pack itself rather than as a separate change on main.

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