Skip to content

Commit c286769

Browse files
rickhanloniigrabbou
authored andcommitted
Fix metro websocket reconnect logic
Summary: This diff fixes the reconnect logic with the metro websockets which is causing the app to not re-connect when metro crashes. To demonstrate the issue, consider the following video: {F156029086} On the left we have metro, on the right is the xcode console with some logging to show the reconnecting phase. When we kill the metro server you can see the app tries to reconnect once and that's it - when metro is started back up, you can see the notification that there are no apps running and can also see that cmd+opt+r doesn't work anymore I updated the logic to optimistically start the connection and if it's still unavailable to retry again after the timeout [iOS][Fixed] - Metro websocket reconnect logic Reviewed By: shergin Differential Revision: D14961433 fbshipit-source-id: 0569aa169dc9f538a7e4a8d04e99de39f2e9b3f9
1 parent f71357a commit c286769

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Libraries/WebSocket/RCTReconnectingWebSocket.m

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -148,9 +148,9 @@ - (void)reconnect
148148
{
149149
__weak RCTSRWebSocket *socket = _socket;
150150
dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(2 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
151-
// Only reconnect if the observer wasn't stoppped while we were waiting
152-
if (socket) {
153-
[self start];
151+
[self start];
152+
if (!socket) {
153+
[self reconnect];
154154
}
155155
});
156156
}

0 commit comments

Comments
 (0)