CheckPointSW / CheckPointAnsibleMgmtCollection

This Ansible collection provides control over a Check Point Management server using Check Point's web-services APIs.
https://galaxy.ansible.com/check_point/mgmt
Apache License 2.0
39 stars 30 forks source link

Not needed warning about must_change_password #99

Closed bartsikkes closed 1 year ago

bartsikkes commented 1 year ago

When I add an administrator via cp_mgmt_administrator* I get this warning:

[WARNING]: Module did not set no_log for must_change_password

I feel that warning is not needed here. The must_change_password variable doesn't contain any sensitive information, it is either true or false.

If you agree, can that be changed?

*) https://docs.ansible.com/ansible/latest/collections/check_point/mgmt/cp_mgmt_administrator_module.html

chkp-edenbr commented 1 year ago

Hi @bartsikkes , Thanks for notifying us, this is a warning generated from ansible regarding our module. We will add the missing argument in the upcoming version. Thanks, Eden

chkp-edenbr commented 1 year ago

Issue fixed in our newly released version v5.0.0.