From 74ef969a33272f6ba9d14f416a0f720f36dec26b Mon Sep 17 00:00:00 2001
From: mercyblitz <mercyblitz@gmail.com>
Date: Tue, 4 Feb 2020 18:42:55 +0800
Subject: [PATCH] Polish alibaba/spring-cloud-alibaba/#1196 : [Issue]
 NacosConfigHealthIndicator should not be UP in any status

---
 .../endpoint/NacosConfigHealthIndicator.java  | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/spring-cloud-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/endpoint/NacosConfigHealthIndicator.java b/spring-cloud-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/endpoint/NacosConfigHealthIndicator.java
index 7becf481..482909d8 100644
--- a/spring-cloud-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/endpoint/NacosConfigHealthIndicator.java
+++ b/spring-cloud-alibaba-nacos-config/src/main/java/com/alibaba/cloud/nacos/endpoint/NacosConfigHealthIndicator.java
@@ -20,9 +20,13 @@ import com.alibaba.nacos.api.config.ConfigService;
 
 import org.springframework.boot.actuate.health.AbstractHealthIndicator;
 import org.springframework.boot.actuate.health.Health;
+import org.springframework.boot.actuate.health.HealthIndicator;
 
 /**
+ * The {@link HealthIndicator} for Nacos Config
+ * 
  * @author xiaojing
+ * @author <a href="mailto:mercyblitz@gmail.com">Mercy</a>
  */
 public class NacosConfigHealthIndicator extends AbstractHealthIndicator {
 
@@ -34,10 +38,21 @@ public class NacosConfigHealthIndicator extends AbstractHealthIndicator {
 
 	@Override
 	protected void doHealthCheck(Health.Builder builder) throws Exception {
-		builder.up();
-
+		// Just return "UP" or "DOWN"
 		String status = configService.getServerStatus();
+		// Set the status to Builder
 		builder.status(status);
+		switch (status) {
+		case "UP":
+			builder.up();
+			break;
+		case "DOWN":
+			builder.down();
+			break;
+		default:
+			builder.unknown();
+			break;
+		}
 	}
 
 }