Netty: fix bind exception on startup

Author: jknack

jooby/src/main/java/io/jooby/Server.java

@@ -9,8 +9,10 @@
 import javax.annotation.Nullable;
 import java.io.EOFException;
 import java.io.IOException;
+import java.net.BindException;
 import java.nio.channels.ClosedChannelException;
 import java.util.List;
+import java.util.Optional;
 import java.util.concurrent.Executor;
 import java.util.concurrent.atomic.AtomicBoolean;
 
@@ -134,9 +136,27 @@ static boolean connectionLost(@Nullable Throwable cause) {
       String message = cause.getMessage();
       if (message != null) {
         String msg = message.toLowerCase();
-        return msg.contains("reset by peer") || msg.contains("broken pipe") || msg.contains("forcibly closed");
+        return msg.contains("reset by peer") || msg.contains("broken pipe") || msg
+            .contains("forcibly closed");
       }
     }
     return (cause instanceof ClosedChannelException) || (cause instanceof EOFException);
   }
+
+  /**
+   * Whenever the given exception is an address already in use. This probably won't work in none
+   * English locale systems.
+   *
+   * @param cause Exception to check.
+   * @return True address alaredy in use.
+   */
+  static boolean isAddressInUse(@Nullable Throwable cause) {
+    return (cause instanceof BindException) ||
+        (Optional.ofNullable(cause)
+            .map(Throwable::getMessage)
+            .map(String::toLowerCase)
+            .filter(msg -> msg.contains("address already in use"))
+            .isPresent()
+        );
+  }
 }

modules/jooby-jetty/src/main/java/io/jooby/jetty/Jetty.java

@@ -159,7 +159,7 @@ public class Jetty extends io.jooby.Server.Base {
 
       fireReady(applications);
     } catch (Exception x) {
-      if (x.getCause() instanceof BindException) {
+      if (io.jooby.Server.isAddressInUse(x.getCause())) {
         x = new BindException("Address already in use: " + options.getPort());
       }
       throw SneakyThrows.propagate(x);

modules/jooby-netty/src/main/java/io/jooby/netty/Netty.java

@@ -129,7 +129,7 @@ public class Netty extends Server.Base {
       throw SneakyThrows.propagate(x);
     } catch (ExecutionException x) {
       Throwable cause = x.getCause();
-      if (cause instanceof BindException) {
+      if (Server.isAddressInUse(cause)) {
         cause = new BindException("Address already in use: " + options.getPort());
       }
       throw SneakyThrows.propagate(cause);

modules/jooby-utow/src/main/java/io/jooby/utow/Utow.java

@@ -116,7 +116,7 @@ public class Utow extends Server.Base {
       return this;
     } catch (RuntimeException x) {
       Throwable cause = Optional.ofNullable(x.getCause()).orElse(x);
-      if (cause instanceof BindException) {
+      if (Server.isAddressInUse(cause)) {
         cause = new BindException("Address already in use: " + options.getPort());
       }
       throw SneakyThrows.propagate(cause);