Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix plugin-name processing in ALRM handler. #12

Merged
merged 2 commits into from Jun 22, 2016
Merged

Conversation

pdugas
Copy link
Contributor

@pdugas pdugas commented Apr 20, 2016

The regex is using the wrong case so it never matches. I've also adjusted it to allow either a hypen (-) or underscore (_).

pdugas added 2 commits Apr 20, 2016
Wrong case in ALRM regex.
Allow hypen or underscore in plugin name.
@pdugas
Copy link
Contributor Author

@pdugas pdugas commented Apr 20, 2016

Using shortname instead may be more correct.

@sni sni merged commit aa86ee0 into monitoring-plugins:master Jun 22, 2016
1 check passed
@sni
Copy link
Contributor

@sni sni commented Jun 22, 2016

thanks, merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants