Skip to content

Resolve SCA triage log errors and bump jackson-databind to 2.21.1 (AST-136208)#476

Merged
cx-atish-jadhav merged 2 commits intomainfrom
bug/AST-136208
Mar 18, 2026
Merged

Resolve SCA triage log errors and bump jackson-databind to 2.21.1 (AST-136208)#476
cx-atish-jadhav merged 2 commits intomainfrom
bug/AST-136208

Conversation

@cx-atish-jadhav
Copy link
Contributor

Description

This PR fixes SCA triage errors observed in JetBrains logs when opening SCA results from CxOne and upgrades the jackson-databind dependency to a non-vulnerable version.

Previously, the triage show/update functionality used a generic similarity-ID flow. However, SCA requires the use of the --vulnerability-identifiers flag. Due to this mismatch, the CLI returned errors such as:

“Failed showing the predicate. Vulnerabilities are required for SCA triage.”


What changed

  • Added SCA-specific CLI support in the wrapper:

    • triageScaShow(...)
    • triageScaUpdate(...)
  • Introduced a new SCA constant in CxConstants:

    • --vulnerability-identifiers
  • Added validation to handle missing or blank SCA vulnerability identifiers.

  • Added SCA triage integration coverage in PredicateTest.


Dependency update

The Maven build reported a vulnerability in the Java wrapper version 2.4.7, which depends on jackson-databind version 2.16.1 (now known to be vulnerable).

To resolve this and ensure the scan passes successfully, the jackson-databind version has been upgraded to 2.21.1 in the pom.xml of the Java wrapper.


Result

  • SCA triage now uses the correct CLI contract.
  • Noisy SCA triage failure logs are eliminated.
  • Existing non-SCA triage behavior remains unchanged.

@github-actions github-actions bot added the bug Something isn't working label Mar 18, 2026
@cx-ben-alvo
Copy link
Collaborator

cx-ben-alvo commented Mar 18, 2026

Logo
Checkmarx One – Scan Summary & Details5fe17a32-4578-4663-acde-2e26c5c3f5ca


New Issues (6) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 MEDIUM ALB Deletion Protection Disabled /positive1.tf: 15
detailsApplication Load Balancer should have deletion protection enabled
2 MEDIUM ALB Listening on HTTP /positive1.tf: 9
detailsAWS Application Load Balancer (alb) should not listen on HTTP
3 MEDIUM ALB Not Dropping Invalid Headers /positive1.tf: 15
detailsIt's considered a best practice when using Application Load Balancers to drop invalid header fields
4 LOW APT-GET Missing Flags To Avoid Manual Input /Dockerfile: 5
detailsCheck if apt-get calls use flags to avoid user manual input.
5 LOW IAM Access Analyzer Not Enabled /positive1.tf: 1
detailsIAM Access Analyzer should be enabled and configured to continuously monitor resource permissions
6 LOW Shield Advanced Not In Use /positive1.tf: 15
detailsAWS Shield Advanced should be used for Amazon Route 53 hosted zone, AWS Global Accelerator accelerator, Elastic IP Address, Elastic Load Balancing,...

Fixed Issues (1) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
HIGH Cxfa47c4e4-5ef9 Maven-com.fasterxml.jackson.core:jackson-core-2.16.1

Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

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-atish-jadhav cx-atish-jadhav marked this pull request as ready for review March 18, 2026 09:31
@cx-atish-jadhav cx-atish-jadhav merged commit 098f2ec into main Mar 18, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants