It makes more sense to exit critical if a explicit version/protocol is requested. This would also be more consistent with other plugins. Other string matching plugins like check_snmp or check_http exit critical if the result does not match.
Signed-off-by: Sven Nierlein <[email protected]> You can merge this Pull Request by running: git pull https://github.com/sni/monitoring-plugins check_ssh_critical Or you can view, comment on it, or merge it online at: https://github.com/monitoring-plugins/monitoring-plugins/pull/1268 -- Commit Summary -- * check_ssh: change warning to critical for protocal/version errors -- File Changes -- M plugins/check_ssh.c (12) -- Patch Links -- https://github.com/monitoring-plugins/monitoring-plugins/pull/1268.patch https://github.com/monitoring-plugins/monitoring-plugins/pull/1268.diff -- Reply to this email on GitHub: https://github.com/monitoring-plugins/monitoring-plugins/pull/1268
