Skip to content

feat: allow secrets exclusions [PS-715] - #7113

Merged
ancasnyk merged 2 commits into
mainfrom
feat/PS-715/secrets-exclusions
Aug 12, 2026
Merged

feat: allow secrets exclusions [PS-715]#7113
ancasnyk merged 2 commits into
mainfrom
feat/PS-715/secrets-exclusions

Conversation

@ancasnyk

@ancasnyk ancasnyk commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: docs: document .snyk exclusions for Snyk Secrets (PS-772) user-docs#1610)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

Updates the secrets extension to a new version which supports setting .snyk exclusions for secrets

Where should the reviewer start?

How should this be manually tested?

run snyk secrets test in a folder with a .snyk file that contains a secrets section under exclude in the .snyk file

What's the product update that needs to be communicated to CLI users?

Users are able to exclude files from secrets scanning by adding them to a new "excludes.secrets" section in the .snyk file

@ancasnyk
ancasnyk requested a review from a team as a code owner August 11, 2026 12:27
@snyk-io

snyk-io Bot commented Aug 11, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@snyk-pr-review-bot

This comment has been minimized.

@CatalinSnyk

Copy link
Copy Markdown
Contributor

Suggestion: Could we maybe add some acceptance test? I think it could be something similar to

it('should support .snyk file filtering', async () => {

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️ There are multiple commits on your branch, please squash them locally before merging!

Generated by 🚫 dangerJS against 21cef7d

@snyk-pr-review-bot

This comment has been minimized.

@ancasnyk
ancasnyk force-pushed the feat/PS-715/secrets-exclusions branch from 7754bb8 to 7082dfe Compare August 11, 2026 15:34
@snyk-pr-review-bot

This comment has been minimized.

Comment thread test/jest/acceptance/snyk-secrets/snyk-secrets-test-user-journey.spec.ts Outdated
@ancasnyk
ancasnyk force-pushed the feat/PS-715/secrets-exclusions branch from 7082dfe to 29cca7b Compare August 11, 2026 18:38
@snyk-pr-review-bot

This comment has been minimized.

@ancasnyk
ancasnyk force-pushed the feat/PS-715/secrets-exclusions branch from 29cca7b to 917bbc5 Compare August 11, 2026 18:54
@snyk-pr-review-bot

This comment has been minimized.

@snyk-pr-review-bot

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 13 relevant code sections from 7 files (average relevance: 1.00)

🤖 Repository instructions applied (from AGENTS.md)

@ancasnyk
ancasnyk merged commit 47321b3 into main Aug 12, 2026
10 checks passed
@ancasnyk
ancasnyk deleted the feat/PS-715/secrets-exclusions branch August 12, 2026 07:08
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.

2 participants