Websocket onclose function is not called in Android

[ttys026]

• I have reviewed the documentation
• I have searched existing issues
• I am using the latest React Native version

Websocket connection closed by server will not trigger onclose callback.

Environment

Environment:
OS: macOS High Sierra 10.13.3
Node: 7.10.1
Yarn: Not Found
npm: 5.7.1
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 2.3 AI-162.3871768

Packages: (wanted => installed)
react: 16.2.0 => 16.2.0
react-native: ^0.53.0 => 0.53.0

For websocket server-side, I use ratchetPHP

Steps to Reproduce

connect = () => {
    this.ws = new WebSocket('URL')
    this.ws.onmessage = (e) => {
      json = JSON.parse(e.data)
      console.log(json)
    }

    this.ws.onclose = (e) => {
      console.log('closed')
    }
}

componentWillReceiveProps(nextProps){
  this.connect()
}

Then close the connection on server.

Expected Behavior

onclose function should be called.

Actual Behavior

When building in Android,
close request send from client will trigger the onclose function,
but RN can't detect a connection closed by server.

[react-native-bot]

Thanks for posting this! It looks like your issue may refer to an older version of React Native. Can you reproduce the issue on the latest release, v0.55?

Thank you for your contributions.

[ttys026]

Now I update to react-native 0.55.0

Environment:
  OS: macOS High Sierra 10.13.3
  Node: 7.10.1
  Yarn: Not Found
  npm: 5.7.1
  Watchman: 4.9.0
  Xcode: Xcode 9.2 Build version 9C40b
  Android Studio: 2.3 AI-162.3871768

Packages: (wanted => installed)
  react: ^16.3.1 => 16.3.1
  react-native: ^0.55.0 => 0.55.0

The same issue is still happening.

If I close the connection from server, the onclose is not called.
If I close the connection from client, it breaks on:
WebSocketModule.close got 1 arguments, expected 3.

which is caused by b9be289

[favonianpine]

Root cause:

square/okhttp#3386

When I use okhttp to achieve WebSocket connection, when received onClosing (WebSocket webSocket, int code, String reason) callback, the call must be manually () method to close the resources.
See the source code found that if the client is passive disconnect, do not manually call the close () method will lead to onClosed (WebSocket webSocket, int code, String reason) method can not callback, the resources can not be recycled.

We need to handle WebSocket onClosing() method and call websocket.close() in it. However, there's no onClosing() in React Native Websocket library. That's why

If I close the connection from server, the onclose is not called.

It is a bug.

[Tonacatecuhtli]

Hi same issue here.

[stale]

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as "For Discussion" or "Good first issue" and I will leave it open. Thank you for your contributions.

[westcode]

This issue is still there @ latest version.

[cristiantx]

Using 0.56.0 with the same issue.
Is there any workaround to know when the WebSocket closes?

[mmazzarolo]

I can confirm the issue is still here on the latest version of RN.

I was able to fix it by adding the following listener to react-native/ReactAndroid/src/main/java/com/facebook/react/modules/websocket/WebSocketModule.java:

@Override
public void onClosing(WebSocket webSocket, int code, String reason) {
    webSocket.close(code, reason);
}