don’t claim dockerhost when we don’t run inside docker

jenkinsci · Sep 18, 2015 · c226479 · c226479
1 parent 1feb17d
commit c226479
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/main/java/com/cloudbees/jenkins/plugins/docker_build_env/MavenPluginHelper.java b/src/main/java/com/cloudbees/jenkins/plugins/docker_build_env/MavenPluginHelper.java
@@ -4,6 +4,9 @@
 import hudson.maven.TcpSocketHostLocator;
 
 import java.io.IOException;
+import java.net.InetAddress;
+import java.net.InetSocketAddress;
+import java.net.UnknownHostException;
 
 /**
  * @author <a href="mailto:[email protected]">Nicolas De Loof</a>
@@ -13,6 +16,11 @@ public class MavenPluginHelper extends TcpSocketHostLocator {
 
     @Override
     public String getTcpSocketHost() throws IOException {
-        return "dockerhost";
+        try {
+            InetAddress.getByName("dockerhost");
+            return "dockerhost";
+        } catch (UnknownHostException e) {
+            // we are not running inside a Docker container;
+        }
     }
 }