Skip to content

[SOAR-21948] EnableCacheValidator - Enforce enable_cache: true for any plugin defining triggers - #218

Merged
lcwiklinski-r7 merged 1 commit into
masterfrom
SOAR-21948-validator-only-apply-triggers-enable_cache-on-cloud-plugins
Sep 1, 2026
Merged

lcwiklinski-r7 merged 1 commit into
masterfrom
SOAR-21948-validator-only-apply-triggers-enable_cache-on-cloud-plugins

Conversation

@lcwiklinski-r7

Copy link
Copy Markdown
Contributor

https://rapid7.atlassian.net/browse/SOAR-21948

  • Move the "triggers require enable_cache: true" check out of CloudReadyValidator into a new EnableCacheValidator so it runs for all plugins, not just Cloud Ready ones — orchestrator-only plugins with triggers were passing validation and then failing at runtime on SDK 6.6.0+. CloudReadyValidator keeps only the cloud rule (no triggers --> enable_cache must be false).

PR Requirements

Developers, verify you have completed the following items by checking them off:

  • Unit tests written for any new or updated code
  • Version bumped within setup.py
  • Changelog entry added

@lcwiklinski-r7
lcwiklinski-r7 requested a review from a team as a code owner September 1, 2026 08:50
@snyk-io

snyk-io Bot commented Sep 1, 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

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

@lcwiklinski-r7
lcwiklinski-r7 merged commit 4906535 into master Sep 1, 2026
9 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.

3 participants