Skip to content

Vorpal custom path implementation (AST-133948)#1426

Merged
cx-anurag-dalke merged 19 commits intomainfrom
other/customVorpal-AST-133948
Feb 13, 2026
Merged

Vorpal custom path implementation (AST-133948)#1426
cx-anurag-dalke merged 19 commits intomainfrom
other/customVorpal-AST-133948

Conversation

@cx-anjali-deore
Copy link
Contributor

@cx-anjali-deore cx-anjali-deore commented Feb 11, 2026

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

Provide the ability to user to give custom location directory PATH where ASCA is installed (vorpal).

ASCA-Specific Flag: --asca-location
This flag is specific to the scan asca command e.g scan asca --asca-location "C:/drive "
CX_ASCA_LOCATION env variable is bind to this flag

If the directory does not exists or the vorpal executable does not exists then throw exclusive error . No fallback mechanism

Global Flag: --optional-flags

This flag accepts multiple key-value parameters separated by semicolons (;).

Format
--optional-flags "key1=value1;key2=value2"

e.g --optional-flags "asca-location=C:\drive\vorpal"

Behavior
The asca-location key specifies the directory where Vorpal (ASCA) is already installed.

CX_OPTIONAL_FLAGS is the env variable binded to this flag

Precedence Order
The following precedence must be enforced:

  1. --asca-location flag
  2. CX_ASCA_PATH environment variable
  3. --optional-flags (asca-location key)
  4. CX_OPTIONAL_FLAGS environment variable

Type of Change

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

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)
  • Any dependent changes have been merged and published in downstream modules
  • I have updated the CLI help for new/changed functionality in this PR (if applicable)
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

Screenshots (if applicable)

Add screenshots to help explain your changes.

Additional Notes

Add any other relevant information.

@github-actions
Copy link

github-actions bot commented Feb 11, 2026

Logo
Checkmarx One – Scan Summary & Details0d0a9098-4abc-4e36-a642-e90de6ade663

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

@cx-anjali-deore cx-anjali-deore changed the title (AST-133948) Vorpal Custom Path implementation Vorpal Custom Path implementation (AST-133948) Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal Custom Path implementation (AST-133948) Vorpal custom path implementation (AST-133948) Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal custom path implementation (AST-133948) Vorpal custom path implementation(AST-133948) Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal custom path implementation(AST-133948) Vorpal custom path implementation(AST-133948) . Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal custom path implementation(AST-133948) . Vorpal custom path implementation (AST-00000) Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal custom path implementation (AST-00000) Vorpal custom path implementation (AST-00000) Feb 11, 2026
@cx-anjali-deore cx-anjali-deore changed the title Vorpal custom path implementation (AST-00000) Vorpal custom path implementation (AST-133948) Feb 11, 2026
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 d1e0da4 into main Feb 13, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants