Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CxFlow Sarif output fails many validation tests #1329

Closed
nleach999 opened this issue Mar 7, 2024 · 1 comment
Closed

CxFlow Sarif output fails many validation tests #1329

nleach999 opened this issue Mar 7, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@nleach999
Copy link
Contributor

Description

Attaching example SAST XML and CxFlow Sarif output. Using the Microsoft Sarif Validator on this file produces 121 validation errors.

Expected Behavior

Produce Sarif with 0 validation errors

Actual Behavior

Multiple validation errors that indicate many Sarif consumers aren't going to properly interpret the Sarif file.

Reproduction

I executed CxFlow 1.6.45 (1.6.46 is broken and --parse does not work) against the SAST XML report using this script:

java -Xms512m -Xmx2048m -jar cx-flow-1.6.45-java11.jar --parse \
    --app=Checkmarx \
    --cx-flow.bug-tracker-impl=Sarif \
    --cx-flow.bug-tracker=Sarif \
    --cx-flow.filter-severity=High,Medium,Low,Information \
    --sarif.file-path=./offline-cx.sarif \
    --checkmarx.offline=true \
    --logging.level.com=OFF \
    --logging.level.org=OFF \
    --logging.level.javax=OFF \
    --f=$1

Environment Details

CxFlow 1.6.45
Java 17

SimplyVulnerable-[master].xml.txt
offline-cx.sarif.json

@nleach999 nleach999 added the bug Something isn't working label Mar 7, 2024
@itsKedar
Copy link
Contributor

itsKedar commented Jun 5, 2024

Fixed in 1.7.01

@itsKedar itsKedar closed this as completed Jun 5, 2024
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

No branches or pull requests

2 participants