Skip to content

ci: make the image security scan non-blocking - #35

Merged
paco-ravenpack merged 1 commit into
masterfrom
ci/unblock-release-scan
Aug 26, 2026
Merged

ci: make the image security scan non-blocking#35
paco-ravenpack merged 1 commit into
masterfrom
ci/unblock-release-scan

Conversation

@paco-ravenpack

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #34. Switching to the official crowdstrike/fcs-action got a real error message out of the scan, and it is not a version problem — it is a permissions problem (run):

[INFO]    Version: latest
[ERROR] API returned errors:
[ERROR] Failed to get FCS download information

The action authenticates first and only then requests the CLI. OAuth succeeded against api.<region>.crowdstrike.com, so FALCON_CLIENT_ID, FALCON_CLIENT_SECRET, and FALCON_REGION are all valid. What fails is GET /csdownloads/combined/files-download/v2, which returns an errors array. The Falcon API client is missing the scope that allows downloading the FCS CLI.

That also explains the original exit code 6: the old shared action hit the v1 download endpoint, got no download_url back for the same reason, and curl then tried to resolve a host literally named null.

This unblocks publishing while the scope is granted in the Falcon console:

  • continue-on-error: true on the scan step.
  • The gating step becomes a warning instead of a hard failure.

This is temporary and should be reverted once the API client has the download scope. The image is published without a passing scan in the meantime.

Test plan

  • Merge, then recreate the v3.0.0 release to re-fire release: published
  • Confirm the scan step reports a warning rather than failing the job
  • Confirm ghcr.io/bigdata-com/bigdata-risk-analyzer:v3.0.0 and :latest are pushed
  • Confirm Fly Deploy runs afterwards
  • Follow up: grant the Falcon API client the FCS download scope, then revert this change

Made with Cursor

The Falcon API client is missing the scope for /csdownloads, so the FCS CLI
cannot be installed and the scan fails before the image is published. OAuth
succeeds against the same host, so the credentials and region are valid.

Publish the image while the scope is sorted out, surfacing the scan result as
a warning. Revert once the scan can run.

Co-authored-by: Cursor <cursoragent@cursor.com>
@paco-ravenpack
paco-ravenpack requested a review from a team as a code owner August 26, 2026 11:48

@fricigliano fricigliano left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all good

@paco-ravenpack
paco-ravenpack merged commit ddfd8c7 into master Aug 26, 2026
4 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.

2 participants