Skip to content

CM-75484: Use the async scan flow for IaC scans - #574

Merged
cfabianski merged 3 commits into
mainfrom
CM-75484-iac-async-scan-flow
Nov 5, 2026
Merged

cfabianski merged 3 commits into
mainfrom
CM-75484-iac-async-scan-flow

Conversation

@cfabianski

@cfabianski cfabianski commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

IaC scans now use the async scan flow, like SAST, instead of the sync flow. Large IaC scans no longer stop at the sync time limit, so the CLI returns the full result.

The CLI now also requests detection rule details in chunks of 50 ids. Before, a scan batch with many distinct rules made the request URL too long (HTTP 414), and the CLI dropped the results of that batch.

Both changes have unit tests.

🤖 Generated with Claude Code

IaC scans now use the async scan flow, like SAST, instead of the sync flow.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@omer-roth omer-roth added the do not merge Used when a PR has been approved but we don't want to merge just yet label Oct 17, 2026
A large scan batch has hundreds of distinct rule ids, and sending them
all in one query string fails with HTTP 414, which drops the batch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cfabianski cfabianski removed the do not merge Used when a PR has been approved but we don't want to merge just yet label Nov 5, 2026
@cfabianski
cfabianski enabled auto-merge (squash) November 5, 2026 15:43
@cfabianski
cfabianski merged commit c6a216a into main Nov 5, 2026
25 checks passed
@cfabianski
cfabianski deleted the CM-75484-iac-async-scan-flow branch November 5, 2026 15:45
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