Skip to content

Conversation

@cx-anjali-deore
Copy link
Contributor

@cx-anjali-deore cx-anjali-deore commented Jun 13, 2025

By submitting this pull request, you agree to the terms within the Checkmarx Code of Conduct. Please review the contributing guidelines for guidance on creating high-quality pull requests.

Description

Display Scaresolver error message when the required fields are not valid.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

Screenshots (if applicable)

Issue/Bug
If customer has passed ScaResolver path --sca-resolver with Source as Zip, the Scan used to get triggered from CxOne SCA scan and not using offline SCA Resolver. Offline Sca-Resolver does not support the zip source

Fix Added :
When Scan create is triggered with source as zip and scaResolver path is set

Error is displayed as : Scanning Zip files is not supported by ScaResolver. Please use non-zip source

image

@github-actions github-actions bot added bug Something isn't working fix labels Jun 13, 2025
@github-actions
Copy link

github-actions bot commented Jun 13, 2025

Logo
Checkmarx One – Scan Summary & Details7e5aa99a-855e-4a54-95b8-42e2c628fa33

Great job, no security vulnerabilities found in this Pull Request

Copy link
Collaborator

@cx-anurag-dalke cx-anurag-dalke left a comment

Choose a reason for hiding this comment

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

ok

@cx-anjali-deore cx-anjali-deore merged commit c78f6bc into main Jun 17, 2025
12 checks passed
@cx-anjali-deore cx-anjali-deore deleted the fix/ScaResolver/AST-73409 branch June 17, 2025 11:31
cx-anjali-deore added a commit that referenced this pull request Jul 3, 2025
… (AST-73409) (#1190)

* Added check for zip for scaresolver issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants