lucasheld / ansible-uptime-kuma

Ansible collection of modules to configure Uptime Kuma
GNU General Public License v3.0
135 stars 19 forks source link

Missing mTLS choice when adding a monitor #19

Closed scr512 closed 1 year ago

scr512 commented 1 year ago

The choice in the module for adding a monitor only accepts empty, basic or ntlm. When trying to add a new monitor using mtls authentication the play fails.

A 4th choice, mtls needs to be added.

I fixed this locally: lucasheld/uptime_kuma/plugins/modules/monitor.py, line 406

From: authMethod=dict(type="str", choices=["", "basic", "ntlm"]), To: authMethod=dict(type="str", choices=["", "basic", "ntlm", "mtls"]),

lucasheld commented 1 year ago

Thank you! Fixed in https://github.com/lucasheld/ansible-uptime-kuma/commit/d359127214d5c1f73a9896015e0cef38d798cb9a.