websocket: WebSocketConfigurer#onClose not called

- fixed for undertow and jetty.
- netty doesn't fire the client disconnected event
- fix #3111

Author: jknack

modules/jooby-jetty/src/main/java/io/jooby/internal/jetty/JettyWebSocket.java

@@ -121,11 +121,9 @@ public void onWebSocketText(String message) {
 
   @Override
   public void onWebSocketClose(int statusCode, String reason) {
-    if (onCloseCallback != null) {
-      handleClose(
-          WebSocketCloseStatus.valueOf(statusCode)
-              .orElseGet(() -> new WebSocketCloseStatus(statusCode, reason)));
-    }
+    handleClose(
+        WebSocketCloseStatus.valueOf(statusCode)
+            .orElseGet(() -> new WebSocketCloseStatus(statusCode, reason)));
   }
 
   @Override

modules/jooby-undertow/src/main/java/io/jooby/internal/undertow/UndertowWebSocket.java

@@ -359,11 +359,9 @@ protected void onError(WebSocketChannel channel, Throwable x) {
 
   @Override
   protected void onCloseMessage(CloseMessage cm, WebSocketChannel channel) {
-    if (isOpen()) {
-      handleClose(
-          WebSocketCloseStatus.valueOf(cm.getCode())
-              .orElseGet(() -> new WebSocketCloseStatus(cm.getCode(), cm.getReason())));
-    }
+    handleClose(
+        WebSocketCloseStatus.valueOf(cm.getCode())
+            .orElseGet(() -> new WebSocketCloseStatus(cm.getCode(), cm.getReason())));
   }
 
   private void handleClose(WebSocketCloseStatus status) {