Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-3490

Websocket onClosed not called on idle timeout

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 9.0.0.Alpha1
    • 8.1.0.Final
    • Web Sockets
    • None

      When implementing a JSR-356 websocket endpoint I set

      session.setMaxIdleTimeout(5000);

      in the @OnOpen method. This means that, if no traffic is generated on the websocket in five seconds, the websocket is automatically closed.

      Now, if the websocket si correcly closed by the client calling the "close" method in JS, the corrispective @OnClose method on the endpoint is called. But if the websocket is closed because of the idle timeout, on the server the @OnClose method is not called, and also @OnError method isn't called.

      I think the correct behaviour should be to call @OnClose even when the socket is closed because of the idle timeout, so that the application has the ability to make some cleanup. Also, this is the behaviour I see when deploying in other Java web/application server.

      The test code of my endpoint is the following:

      EchoEndpoint.java
      @ServerEndpoint("/ws/echo")
      public class EchoEndpoint {
          
          @OnMessage
          public String onMessage(String message) {
              System.out.println("RECV: " + message);        
              return message;
          }
      
          @OnOpen
          public void onOpen(Session session) {
              session.setMaxIdleTimeout(5000);
              System.out.println("OPEN");
          }
      
          @OnClose
          public void onClose() {
              System.out.println("CLOSE");
          }
      
          @OnError
          public void onError(Throwable t) {
              System.out.println("ERROR");
          }  
      }
      

              sdouglas1@redhat.com Stuart Douglas (Inactive)
              ecolombo_jira Emanuele Colombo (Inactive)
              Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: