diff options
author | toby <smorgrav@yahoo-inc.com> | 2017-08-31 14:56:10 +0200 |
---|---|---|
committer | toby <smorgrav@yahoo-inc.com> | 2017-09-06 16:20:48 -0700 |
commit | e67d0b8657ea72d29aeb13954ea0173b6a8ef545 (patch) | |
tree | 7ea8f6454dcd083fcd6c6bc6bdc4dc4a5ff21f38 /node-repository/src/test/java/com | |
parent | 92526ea24add9baecf8e41b3c99fc3adf7b89e49 (diff) |
Avoid using ip addresses that does not resolve to a hostname
Diffstat (limited to 'node-repository/src/test/java/com')
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java index 04c0af5d98a..2deac4e70e8 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java @@ -83,4 +83,23 @@ public class NodePrioritizerTest { b.addFlavor("d2", 2, 2., 2, Flavor.Type.DOCKER_CONTAINER); return b.build(); } + + @Test + public void ignore_hosts_that_is_unresolved_in_dns() { + // Random IP that is not resolving + String hostname = NodePrioritizer.lookupHostname("34.23.12.23"); + Assert.assertNull(hostname); + + // Localhost is always resolving + hostname = NodePrioritizer.lookupHostname("127.0.0.1"); + Assert.assertNotNull(hostname); + + // IPv6 Localhost is always resolving to a hostname + hostname = NodePrioritizer.lookupHostname("::1"); + Assert.assertNotNull(hostname); + + // Unspecified IPv6 should not resolve to a hostname + hostname = NodePrioritizer.lookupHostname("::"); + Assert.assertNull(hostname); + } } |