[Nagiosplug-devel] [ nagiosplug-Bugs-1460312 ] check_http doesn't check response time on 300's

[SourceForge.net]

Bugs item #1460312, was opened at 2006-03-28 17:40
Message generated for change (Comment added) made by dermoth
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1460312&group_id=29880

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: General plugin execution
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Thomas Guyot (dermoth)
>Assigned to: Thomas Guyot (dermoth)
Summary: check_http doesn't check response time on 300's

Initial Comment:
300's return codes are accepted as OK by default, but
some checks, notably warning/critical time, aren't done
on 300 return codes.

I didn't had time to rewrite this correctly and test it
troughfully, but this simple patch will go on with
other tests when 300's are accepted as OK.

Ideally, response time should be checked when
300!=CRITICAL, and I also on when expecting a string.

----------------------------------------------------------------------

>Comment By: Thomas Guyot (dermoth)
Date: 2007-06-12 19:55

Message:
Logged In: YES 
user_id=375623
Originator: YES

Indeed this patch looks strange... It was not necessarily intended for
inclusion (just a hack), I don't use it anymore and it's been so long I
don't even remember why it wasn't "right"...

I wasn't a NP developer when I submitted it... I'll assign it to myself
and will look over it when I have time (Certainly not until august
unfortunately).

Thomas


----------------------------------------------------------------------

Comment By: Matthias Eble (psychotrahe)
Date: 2007-06-12 14:09

Message:
Logged In: YES 
user_id=1694341
Originator: NO

Hi Thomas,

I looked over the patch and i guess, this would prevent the actual
redirect.
We should also take care of changes to the time measuring behaviour and
mention changes in NEWS or similar so that people don't wonder why their
performance got worse :)
Could you bring up some time look at this?

Matthias




----------------------------------------------------------------------

Comment By: Thomas Guyot (dermoth)
Date: 2006-03-28 17:41

Message:
Logged In: YES 
user_id=375623

Forgot the check_box ;)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1460312&group_id=29880