NIFI-4761 Fixing test case regression.

This closes #2418.

Signed-off-by: Andy LoPresto <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/nifi/repo
Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/d9487129
Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/d9487129
Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/d9487129

Branch: refs/heads/HDF-3.1-maint
Commit: d9487129cc3abb94a6863f81c23c92e02bc1c870
Parents: c582d86
Author: Matt Gilman <[email protected]>
Authored: Fri Jan 19 12:42:47 2018 -0500
Committer: Matt Gilman <[email protected]>
Committed: Fri Jan 19 12:49:22 2018 -0500

----------------------------------------------------------------------
 .../apache/nifi/web/server/HostHeaderHandler.java    | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/nifi/blob/d9487129/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/HostHeaderHandler.java
----------------------------------------------------------------------
diff --git 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/HostHeaderHandler.java
 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/HostHeaderHandler.java
index 9a07fcb..72e8d84 100644
--- 
a/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/HostHeaderHandler.java
+++ 
b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/HostHeaderHandler.java
@@ -70,15 +70,20 @@ public class HostHeaderHandler extends ScopedHandler {
         this.serverName = Objects.requireNonNull(serverName);
         this.serverPort = serverPort;
 
-        // Default values across generic instances
         validHosts = generateDefaultHostnames(null);
-
-        // specified host:port
         validHosts.add(serverName.toLowerCase());
         validHosts.add(serverName.toLowerCase() + ":" + serverPort);
-
-        // empty is ok here
+        // Sometimes the hostname is left empty but the port is always 
populated
+        validHosts.add("localhost");
+        validHosts.add("localhost:" + serverPort);
+        // Different from customizer -- empty is ok here
         validHosts.add("");
+        try {
+            
validHosts.add(InetAddress.getLocalHost().getHostName().toLowerCase());
+            
validHosts.add(InetAddress.getLocalHost().getHostName().toLowerCase() + ":" + 
serverPort);
+        } catch (final Exception e) {
+            logger.warn("Failed to determine local hostname.", e);
+        }
 
         logger.info("Created " + this.toString());
     }

Reply via email to