Skip to content

Conversation

cx-leonardo-fontes
Copy link
Collaborator

@cx-leonardo-fontes cx-leonardo-fontes commented Jul 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

Epic: https://checkmarx.atlassian.net/browse/AST-75500
Task: https://checkmarx.atlassian.net/browse/AST-104600

We want do add the new licensing logic under the SSCS_NEW_LICENSING_ENABLED feature flag:

  • SCS will be renamed to Repository Health
  • Enterprise Secrets will be renamed to Secret Detection
  • Repository Health and Secret Detection will have independent licenses (previously, enabling Enterprise Secrets required also having SCS).
  • The SSCS engine will run only if at least one of the two micro-engines is both enabled and licensed.

Type of Change

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

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
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@cx-leonardo-fontes cx-leonardo-fontes changed the title Add new SSCS licensing (WIP) Add new SSCS licensing (WIP) (AST-104600) Jul 24, 2025
Copy link

github-actions bot commented Jul 24, 2025

Logo
Checkmarx One – Scan Summary & Details22d34e17-3f80-4d15-ace3-16b56e0b4582

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

@cx-leonardo-fontes cx-leonardo-fontes changed the title Add new SSCS licensing (WIP) (AST-104600) Add new SSCS licensing (AST-104600) Aug 5, 2025
cx-celso-silva
cx-celso-silva previously approved these changes Aug 12, 2025
Copy link

@cx-celso-silva cx-celso-silva left a comment

Choose a reason for hiding this comment

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

looks good

@cx-leonardo-fontes cx-leonardo-fontes marked this pull request as ready for review August 14, 2025 10:59
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

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 merged commit ba50ebd into main Aug 25, 2025
9 checks passed
@cx-anurag-dalke cx-anurag-dalke deleted the feature/add-new-sscs-licensing branch August 25, 2025 10:53
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