net-mgmt/xymon-server: Remove httpresult patch

Upstream has confirmed they do not intend to revert the change to the
httpresult categorization which resulted in 301 Permanent redirects
being classfied as Yellow (warning) instead of Green (ok)

The logic is that a 301 redirect is a permanent redirect and you should
not be monitoring it but the actual content it is redirecting to.

The old behavior existed for many years, but is considered faulty
logic. If you have "yellow" surprises on your dashboard, please fix your
monitoring targets. I do not wish to divert FreeBSD's Xymon behavior
from upstream's intentions.
This commit is contained in:
Mark Felder 2015-11-25 17:37:37 +00:00
parent b4970521eb
commit 3caf0ad9bb
Notes: svn2git 2021-03-31 03:12:20 +00:00
svn path=/head/; revision=402405
2 changed files with 1 additions and 11 deletions

View File

@ -2,7 +2,7 @@
PORTNAME= xymon
PORTVERSION= 4.3.24
PORTREVISION= 0
PORTREVISION= 1
CATEGORIES= net-mgmt www
MASTER_SITES= SF/xymon/Xymon/${PORTVERSION}
PKGNAMESUFFIX= -server${PKGNAMESUFFIX2}

View File

@ -1,10 +0,0 @@
--- xymonnet/httpresult.c.orig 2015-11-24 15:59:24 UTC
+++ xymonnet/httpresult.c
@@ -44,6 +44,7 @@ static int statuscolor(testedhost_t *h,
switch(status) {
case 200: /* OK - most common case */
+ case 301: /* Permanent Redirect */
case 302: /* Temp Redirect */
case 303: /* See Other */
case 307: /* Temp Redirect (HTTP 1.1) */