Skip to content

tool: make parser reject blank arguments if not supported#12620

Closed
bagder wants to merge 1 commit intomasterfrom
bagder/getstr-reject-blanks
Closed

tool: make parser reject blank arguments if not supported#12620
bagder wants to merge 1 commit intomasterfrom
bagder/getstr-reject-blanks

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Jan 2, 2024

Already in the getstr() function that clones the input argument.

Already in the getstr() function that clones the input argument.

Closes #12620
@bagder bagder force-pushed the bagder/getstr-reject-blanks branch from 170fe94 to b37c40d Compare January 3, 2024 12:09
@bagder bagder closed this in 07bcae8 Jan 3, 2024
@bagder bagder deleted the bagder/getstr-reject-blanks branch January 3, 2024 22:22
efranken added a commit to efranken/ToscaExecutionClient that referenced this pull request Jul 10, 2024
As of curl 8.6.0, curl fails if a blank arg is passed.  curl/curl#12620

Default behavior of tosca_execution_client.sh is to pass a blank --cacert to curl if --cacertificate is not specified when running the execution client script.

This modifies the execution client to not pass --cacert at all if --cacertificate is not specified when running the execution client.

If --cacertificate is specified, curl is passed "--cacert " followed by the value of the --cacertificate flag specified when running the execution client.
AgileAOTeam pushed a commit to Tricentis/ToscaExecutionClient that referenced this pull request Jul 17, 2024
As of curl 8.6.0, curl fails if a blank arg is passed.  curl/curl#12620

Default behavior of tosca_execution_client.sh is to pass a blank --cacert to curl if --cacertificate is not specified when running the execution client script.

This modifies the execution client to not pass --cacert at all if --cacertificate is not specified when running the execution client.

If --cacertificate is specified, curl is passed "--cacert " followed by the value of the --cacertificate flag specified when running the execution client.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants