[MINOR] http: don't mark a server as failed when it returns 501/505
Those two codes can be triggered on demand by client requests.
We must not fail a server on them.
Ideally we should ignore a certain amount of status codes which do
not indicate life nor death.
diff --git a/src/proto_http.c b/src/proto_http.c
index ad44a2e..f1ec7cd 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -4503,7 +4503,11 @@
txn->status = strl2ui(msg->sol + msg->sl.st.c, msg->sl.st.c_l);
- if (txn->status >= 100 && txn->status < 500)
+ /* Adjust server's health based on status code. Note: status codes 501
+ * and 505 are triggered on demand by client request, so we must not
+ * count them as server failures.
+ */
+ if (txn->status >= 100 && (txn->status < 500 || txn->status == 501 || txn->status == 505))
health_adjust(s->srv, HANA_STATUS_HTTP_OK);
else
health_adjust(s->srv, HANA_STATUS_HTTP_STS);