hubot-archive / hubot-pager-me

PagerDuty integration for Hubot
https://www.npmjs.com/package/hubot-pager-me
MIT License
73 stars 92 forks source link

Sanity check `/pager me N`, in case it's accidentally used to ack/resolve something #7

Open technicalpickles opened 10 years ago

technicalpickles commented 10 years ago

We've seen times when someone goes like:

hubot pager sup
hubot pager me 1234

... while attempting to acknowledge the 1234 incident. Instead, this adds a schedule override for the user for 1234 minutes. Incident numbers are much higher, which could lead to several weeks of schedule being override. This could be particularly disasterous if multiple groups are within the PagerDuty account, without overlap in skill sets.

Some fixes might include: