Skip to content

Conversation

@cx-rahul-pidde
Copy link

Description
Updated the string replacement logic in the if(n) block to preserve single quotes in command-line parameters by modifying the .replace() function.

Why this change was made

  • Previously, both double (") and single (') quotes were being removed from the string.
  • However, some CLI arguments (like --gradle-parameters='-pUSERNAME=abc -pPASSWORD=cba') require single quotes to preserve spaces inside parameter values.
  • Removing single quotes was causing malformed or broken arguments when passed to tools like the Checkmarx SCA Resolver.
  • Now, only double quotes are stripped, while single quotes are retained to maintain correct CLI syntax.

@cx-ben-alvo
Copy link
Collaborator

Logo
Checkmarx One – Scan Summary & Detailsa6feca0d-746a-4647-857d-2acdafe80e6a

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


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

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-rahul-pidde cx-rahul-pidde merged commit 7a0275b into main Oct 10, 2025
12 of 15 checks passed
@cx-rahul-pidde cx-rahul-pidde deleted the bug/AST-115793 branch October 10, 2025 12:56
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