search

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

don't use case-insensitive matching #225

Closed johnseekins-pathccm closed 7 months ago

johnseekins-pathccm commented 7 months ago

This should address issues like #223 . By not doing case-insensitive matching, we allow schedules with names like "Incident Commander" to be used.

johnseekins-pathccm commented 7 months ago

This is actually really interesting. Removing case-insensitive matching works with local testing, but fails when using the Slack adapter (no response from Hubot, no errors in the logs...). Is there actually a bug in the slack adapter that breaks regex matching without /i set?