From 16f156125c694cdd4dc72f08eec2587d8842d82f Mon Sep 17 00:00:00 2001 From: yush1ga Date: Fri, 30 Sep 2016 07:25:40 +0900 Subject: [PATCH] Revise null check of ServiceConfigureUtils (#47) --- .../com/yahoo/pulsar/broker/ServiceConfigurationUtils.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pulsar-broker-common/src/main/java/com/yahoo/pulsar/broker/ServiceConfigurationUtils.java b/pulsar-broker-common/src/main/java/com/yahoo/pulsar/broker/ServiceConfigurationUtils.java index a7c0141eafe..30ad82911ef 100644 --- a/pulsar-broker-common/src/main/java/com/yahoo/pulsar/broker/ServiceConfigurationUtils.java +++ b/pulsar-broker-common/src/main/java/com/yahoo/pulsar/broker/ServiceConfigurationUtils.java @@ -21,12 +21,14 @@ import org.slf4j.LoggerFactory; import java.net.InetAddress; import java.net.UnknownHostException; +import static org.apache.commons.lang3.StringUtils.isBlank; + public class ServiceConfigurationUtils { private static final Logger LOG = LoggerFactory.getLogger(ServiceConfigurationUtils.class); public static String getDefaultOrConfiguredAddress(String configuredAddress) { - if ( configuredAddress == null ) { + if (isBlank(configuredAddress)) { return unsafeLocalhostResolve(); } return configuredAddress; -- GitLab