search

xWyatt / check_rest_api

Nagios Core plugin to check output from a RESTful JSON API
GNU General Public License v3.0
11 stars 5 forks source link

Found small bug about CRITICAL state exit value #11

Closed kenanrizvic closed 3 years ago

kenanrizvic commented 3 years ago

Thanks on good code. It helped me make some json nagios check but also had issue when one of my check get over into critical State, nagios alarm didn't show as critical but as warning, cause it returned status 1 (WARNING). I fixed that part of code( atleast it works ok now for me :). Also original code from https://github.com/xWyatt/check_rest_api made issue when trying to compile , and it was making error when linking with argVals variable, so had to put extern src/read_input.c and define variable in src/check_rest_api.c .

If u find this usable to pull into your own code or to say that my changes are risky , or not go along logic of code, please reporte me .

Thanks one more time. Kenan R.

kenanrizvic commented 3 years ago

Fixed readme and spaces/tabs . Diff looks much more clear now.

Thanks my friend.