Skip to content

Conversation

@cx-sumit-morchhale
Copy link
Contributor

@cx-sumit-morchhale cx-sumit-morchhale commented Dec 24, 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

I have implemented a new API to fetch the count of API security vulnerabilities, with support for filtering based on state, status, and severity. This implementation was done because the existing threshold mechanism for API security does not work as expected when the --filter flag is used with values for state, status, or severity.

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Documentation update

Related Issues

Link any related issues or tickets.

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)

*Before code changes:
This failed the threshold, even though there is no NEW results because already run scan with the project → it’s wrong
image
*After code changes:
command:
This passed the threshold, there is no NEW results because already run scan with the project → It is working as expected
image

Additional Notes

Add any other relevant information.

@github-actions
Copy link

github-actions bot commented Dec 24, 2025

Logo
Checkmarx One – Scan Summary & Details163bc9ee-853c-4be6-8b87-fc420d4347ee

Great job! No new security vulnerabilities introduced in this pull request

Copy link
Contributor

@cx-rahul-pidde cx-rahul-pidde left a comment

Choose a reason for hiding this comment

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

Looks Ok

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-anurag-dalke cx-anurag-dalke enabled auto-merge (squash) December 30, 2025 11:52
@cx-anurag-dalke cx-anurag-dalke merged commit 828a903 into main Dec 30, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants