[0.83] Add support for custom header in devsupport#55722
Draft
cortinico wants to merge 16 commits into0.83-stablefrom
Draft
[0.83] Add support for custom header in devsupport#55722cortinico wants to merge 16 commits into0.83-stablefrom
cortinico wants to merge 16 commits into0.83-stablefrom
Conversation
Summary: Pull Request resolved: #55575 The devsupport package creates multiple OkHttpClient instances across different files, each with its own connection pool and thread pool. This is wasteful and introduces bugs (thread-safety race in InspectorNetworkHelper, per-tap allocation in RedBoxContentView). This diff introduces DevSupportHttpClient, an internal object singleton with two shared clients derived from the same connection pool: - httpClient: with connect=5s, write=disabled, read=disabled (for HTTP requests) - websocketClient: derived via newBuilder() with connect=10s, write=10s, read=disabled (for WebSocket/inspector use) Subsequent diffs in this stack migrate each consumer to use these shared clients. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93480249 fbshipit-source-id: b489f5946aa0c0a5318f0fb600dec31f373b8c09
Summary: Pull Request resolved: #55576 Replace the inline OkHttpClient.Builder() in DevServerHelper with DevSupportHttpClient.httpClient. This shares the connection pool and thread pool with all other devsupport HTTP consumers. Remove the now-unused HTTP_CONNECT_TIMEOUT_MS constant and TimeUnit import. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93480251 fbshipit-source-id: b783c2b7a499d20fc12a9fca664651e99b00579d
#55577) Summary: Pull Request resolved: #55577 PackagerStatusCheck had two constructors: a no-arg one that created its own OkHttpClient, and one that accepted an external client. The no-arg constructor is used by callers in fbandroid/ (FbDevBottomSheetViewController, LocalJsPackagerManager, FbPackagerStatusCheck, FbDevModeNuxController). Instead of removing it, delegate it to `DevSupportHttpClient.httpClient` so that all callers share the same connection pool without needing code changes. Also removes the now-unused `HTTP_CONNECT_TIMEOUT_MS` constant since the timeout configuration is centralized in DevSupportHttpClient. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93480248 fbshipit-source-id: 07b3a6c01cec19a5de1604c560229622974e500e
Summary: Pull Request resolved: #55578 RedBoxContentView.OpenStackFrameTask was creating a new OkHttpClient() (with a fresh connection pool and thread pool) every time a user tapped a stack frame in the red box. Replace with the shared DevSupportHttpClient.httpClient singleton. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93480250 fbshipit-source-id: 882c218e709681e5f5fb22b4b35710d23f349c8e
Summary: Pull Request resolved: #55579 InspectorNetworkHelper used a lateinit var with an isInitialized check to lazily create an OkHttpClient. This pattern is not thread-safe — concurrent calls to loadNetworkResource could race on the initialization. Replace with the shared DevSupportHttpClient.websocketClient singleton, which is initialized at class load time and is inherently safe. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93480252 fbshipit-source-id: fefb144b3b26670954a77aa27c9b7f637290f21b
Summary: Pull Request resolved: #55580 Replace the OkHttpClient.Builder() in CxxInspectorPackagerConnection.DelegateImpl with DevSupportHttpClient.websocketClient. This shares the connection pool and thread pool with all other devsupport WebSocket consumers. Remove the now-unused TimeUnit and OkHttpClient imports. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93480223 fbshipit-source-id: ff3d742ccb4f484f4906496b1a081377a1b3fa8d
Summary: Pull Request resolved: #55581 Replace the inline OkHttpClient.Builder() in ReconnectingWebSocket with DevSupportHttpClient.websocketClient. This shares the connection pool and thread pool with all other devsupport WebSocket consumers. DevSupportHttpClient is changed from internal to public to be accessible from the packagerconnection module, and the packagerconnection BUCK target gains a dependency on the inspector target. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93481092 fbshipit-source-id: b0d6a14e59310e2f975ce92d6bab68f6f6a231da
Summary: Pull Request resolved: #55582 Add addRequestHeader/removeRequestHeader methods to DevSupportHttpClient. These allow callers to register custom HTTP headers that will be included in all requests made through both the httpClient and websocketClient. Implemented via a shared OkHttp Interceptor that reads from a ConcurrentHashMap, so headers can be safely added/removed from any thread. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D93481539 fbshipit-source-id: e8d6bc4c9d21c65a0f4d95065eaadff776933c55
Summary: Pull Request resolved: #55586 Add a thread-safe singleton that holds custom HTTP headers to be applied to all iOS devsupport network requests. This mirrors the Android `DevSupportHttpClient` from D93481539. Uses a GCD concurrent queue with barrier writes for reader-writer synchronization. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93490954 fbshipit-source-id: addb2f11025a046eedd0e296a7c398eef7211dc4
Summary: Pull Request resolved: #55607 Inject custom devsupport headers into bundle download requests. This ensures that multipart bundle fetches include any headers registered via `RCTDevSupportHttpHeaders`. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596721 fbshipit-source-id: 358b0de351677b843a70204caa101531941789b0
Summary: Pull Request resolved: #55608 Inject custom devsupport headers into the packager status check request. Changes NSURLRequest to NSMutableURLRequest to allow header mutation before the request is sent. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596717 fbshipit-source-id: 94ab3df95317cb3697d26d18b6a27750bfcf4bff
Summary: Pull Request resolved: #55609 Inject custom devsupport headers into the open-debugger POST request. This ensures the debugger launch request includes any registered custom headers. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596719 fbshipit-source-id: d4ed0a67b586b8b4e6bbd1817b7cfdf0bd75065f
Summary: Pull Request resolved: #55610 Inject custom devsupport headers into inspector resource loading requests. This ensures that inspector network fetches include any registered custom headers. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596720 fbshipit-source-id: bda4690a2d9c78729c874977357358a432e46046
Summary: Pull Request resolved: #55611 Inject custom devsupport headers into the HMR WebSocket connection. Changes from `initWithURL:` to `initWithURLRequest:` so headers can be set on the HTTP upgrade handshake. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596718 fbshipit-source-id: ab028d0e03e11f0efde000d19943693439236dbc
Summary: Pull Request resolved: #55612 Inject custom devsupport headers into the inspector CDP WebSocket connection. Changes from `initWithURL:` to `initWithURLRequest:` so headers can be set on the HTTP upgrade handshake. Changelog: [Internal] [Changed] - Reviewed By: cipolleschi Differential Revision: D93596715 fbshipit-source-id: 12dc5c991027654fdd5559daafbc27144a0fdb6b
meta-cla
bot
added
the
CLA Signed
|
Warning JavaScript API change detected This PR commits an update to
This change was flagged as: |
Job Summary for GradleTest All :: run_fantom_tests
|
…tHeaders Summary: Extract header management from the `internal` `DevSupportHttpClient` into a new public `DevSupportRequestHeaders` singleton in `devsupport.interfaces`. This allows external consumers (e.g. Expo Dev Launcher) to register custom HTTP headers that will be applied to all React Native dev-support network traffic without depending on internal APIs. `DevSupportHttpClient` now delegates to `DevSupportRequestHeaders.allHeaders()` instead of maintaining its own `ConcurrentHashMap`. A separate `request_headers` Buck target is introduced to avoid a dependency cycle (`bridge` → `inspector` → `interfaces` → `bridge`). Changelog: [Android][Added] - Public DevSupportRequestHeaders API for registering custom dev-support HTTP headers Differential Revision: D94354168
Job Summary for GradleTest All :: run_fantom_tests
|
vzaidman
approved these changes
Feb 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary:
This is a series of commits that add custom header capability to devsupport (Android & iOS) for 0.83.
Changelog:
[INTERNAL] -
Test Plan:
CI