URL not properly URI encoded in cyberark_account

[shorbachuk]

URL not properly URI encoded in cyberark_account

[compostCY]

Thanks for the clarification. You are absolutely right. We will add this to handle the malformed uri.

[compostCY]

Please note this is working as designed in this release - safe is a required attribute needed to pass in:
safe:
description:
- The safe in the Vault where the privileged account is to be located
required: true
type: str

as stated in the doc: https://github.com/cyberark/ansible-security-automation-collection/blob/master/docs/cyberark_account.md

We may consider changing the behavior in the future release.

thanks.

[shorbachuk]

required=true requires the safe to be present but it does not require non-null as per my example. Not asking to change the required=true behavior just asking to fix the malformed uri that results

[compostCY]

Hello Shorbachuk:
Upon further review with our internal team, it seems to be an open issue with the Ansible platform itself when required is used. The discussion is here:
ansible/ansible#69190

if required, why allow someone to sent a none/empty value. This is definitely a debatable issue.

Will keep an open eye on this issue going forward.

[shorbachuk]

What is there to monitor? That issue is closed. Also confirms the behavior is as I said. I’m a cyberark customer and would like this fixed as it’s clearly a bug

why allow empty safe? Because the api doesn’t require a safe when filtering