kuhn-ruess / Checkmk-Checks

Checks and Stuff for Check_MK
MIT License
40 stars 22 forks source link

Cohesity - Add domain option for auth and allow password with dollar sign #84

Closed meni2029 closed 5 days ago

meni2029 commented 1 week ago

Hello @Bastian-Kuhn ,

About the Cohesity plugin. We need the domain for the authentication. This PR adds this option, default sets the domain to "LOCAL". Ref: https://developer.cohesity.com/apidocs/651/#/http/getting-started

Also we had issue with password including dollar sign not correctly interpreted in command line for the special agent. I added single quote to the command arguments. I hope this is ok for you to accept this PR. I changed the version to 1.3.4, but I didn't create a mkp. BR

Bastian-Kuhn commented 5 days ago

Thank you very much!