search

lucasheld / uptime-kuma-api

A Python wrapper for the Uptime Kuma Socket.IO API
https://uptime-kuma-api.readthedocs.io
MIT License
273 stars 23 forks source link

mTLS option is missing in Ansible collection #21

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"]),

scr512 commented 1 year ago

Wrong repo, I'll open this up in the correct place.