Skip to content

fix RCTBundleURLProvider crash when failing to check isPackagerRunning#55761

Closed
vzaidman wants to merge 1 commit intofacebook:mainfrom
vzaidman:export-D94382277
Closed

fix RCTBundleURLProvider crash when failing to check isPackagerRunning#55761
vzaidman wants to merge 1 commit intofacebook:mainfrom
vzaidman:export-D94382277

Conversation

@vzaidman
Copy link
Copy Markdown
Contributor

Summary:
In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the /status request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Differential Revision: D94382277

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 25, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Feb 25, 2026

@vzaidman has exported this pull request. If you are a Meta employee, you can view the originating Diff in D94382277.

vzaidman added a commit to vzaidman/react-native that referenced this pull request Feb 26, 2026
facebook#55761)

Summary:

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277
vzaidman added a commit to vzaidman/react-native that referenced this pull request Feb 26, 2026
facebook#55761)

Summary:
Pull Request resolved: facebook#55761

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277
vzaidman added a commit to vzaidman/react-native that referenced this pull request Feb 26, 2026
facebook#55761)

Summary:
Pull Request resolved: facebook#55761

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277
vzaidman added a commit to vzaidman/react-native that referenced this pull request Feb 26, 2026
facebook#55761)

Summary:

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277
facebook#55761)

Summary:
Pull Request resolved: facebook#55761

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277
@meta-codesync meta-codesync bot closed this in ec29c29 Feb 26, 2026
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Feb 26, 2026
@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @vzaidman in ec29c29

When will my fix make it into a release? | How to file a pick request?

@meta-codesync
Copy link
Copy Markdown

meta-codesync bot commented Feb 26, 2026

This pull request has been merged in ec29c29.

zoontek pushed a commit to zoontek/react-native that referenced this pull request Mar 9, 2026
facebook#55761)

Summary:
Pull Request resolved: facebook#55761

In certain situations, where there are network issues reaching the Dev Server, the app crashes because of waiting too long on a semaphore.

Instead, reduce the timeout on the `/status` request to 6 seconds, which is plenty of time to validate that the Dev Server is running, and reduce the timeout for the semaphore to 8 seconds, in case the request still won't finish by then.

Changelog: [iOS][Fixed] not crashing on network issues with connecting to DevServer

Reviewed By: cipolleschi

Differential Revision: D94382277

fbshipit-source-id: b898af6af6a5a9d7964f78a57f921327fe4c4e95
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants