Fix mute/unmute for hosts and monitors

alexcreek commented 7 years ago

Hey I'm using this library to build a hubot plugin for Datadog and patched some bugs.

muting/unmuting hosts fails
- FIX: Added the hostname to path when calling client.request(). Datadog's host api endpoint now takes the hostname to mute/umute in the url instead of the POST body. This also means params needs to be passed to client.request() with at least an empty body
muting monitors fails
- FIX: Added the missing monitorId to the client.request() call in monitor.mute(). monitorId wasn't getting passed.
unmuting monitors fails
- FIX: Added an empty params.body to monitor.unmute(). Params wasn't getting a body set and was failing at req.write(body)

brettlangdon commented 7 years ago

Nice! Thank you so much for the contribution!

brettlangdon commented 7 years ago

@alexcreek I just published 2.6.1 to npm with these changes.

Thanks again for the contribution!

alexcreek commented 7 years ago

👍

brettlangdon / node-dogapi