From 539f4d3095412bddac0b50efe5f03cca403281fd Mon Sep 17 00:00:00 2001
From: Vincent Latombe <vincent@latombe.net>
Date: Mon, 18 Jul 2016 15:08:38 +0200
Subject: [PATCH] Calling healthcheck shouldn't trigger unauthorized log

Unauthorized calls are already logged through logUnAuthorizedRequest
method.
---
 .../elasticsearch/plugins/http/HttpBasicServer.java    | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/main/java/com/asquera/elasticsearch/plugins/http/HttpBasicServer.java b/src/main/java/com/asquera/elasticsearch/plugins/http/HttpBasicServer.java
index fe98333..c8637b8 100644
--- a/src/main/java/com/asquera/elasticsearch/plugins/http/HttpBasicServer.java
+++ b/src/main/java/com/asquera/elasticsearch/plugins/http/HttpBasicServer.java
@@ -130,14 +130,14 @@ private boolean ipAuthorized(final HttpRequest request) {
                             new XForwardedFor(xForwardedFor),
                             proxyChains);
       ipAuthorized = client.isAuthorized();
-      if (ipAuthorized) {
-        if (log) {
+      if (log) {
+        if (ipAuthorized) {
           String template = "Ip Authorized client: {}";
           Loggers.getLogger(getClass()).info(template, client);
+        } else {
+          String template = "Ip Unauthorized client: {}";
+          Loggers.getLogger(getClass()).error(template, client);
         }
-      } else {
-        String template = "Ip Unauthorized client: {}";
-        Loggers.getLogger(getClass()).error(template, client);
       }
       return ipAuthorized;
     }