cyberark / ansible-security-automation-collection

CyberArk Ansible Security Automation Collection
MIT License
60 stars 39 forks source link

Replaced concurrentSessions by concurrentSession #35

Closed linionbb closed 3 years ago

linionbb commented 3 years ago

As documented in Cyberark API v2 documentation the field is "concurrentSession" and not "concurrentSessions"

[https://docs.cyberark.com/Product-Doc/OnlineHelp/PAS/Latest/en/Content/SDK/CyberArk%20Authentication%20-%20Logon_v10.htm?tocpath=Developer%7CREST%20APIs%7CAuthentication%7CLogon%7C_____1#Bodyparameters](cyberark doc)

Desired Outcome

Allow concurrent sessions

Implemented Changes

Replaced "concurrentSessions" by "concurrentSession"

Connected Issue/Story

Resolves #[https://github.com/cyberark/ansible-security-automation-collection/issues/34]

CyberArk internal issue link: []()

Definition of Done

Replace concurrentSessions by concurrentSession

Changelog

Test coverage

Documentation

Behavior

Security

linionbb commented 3 years ago

Was fixed without evaluating / closing the PR?