Skip to content

Apply custom headers in RCTInspectorDevServerHelper#55609

Closed
cortinico wants to merge 4 commits intofacebook:mainfrom
cortinico:export-D93596719
Closed

Apply custom headers in RCTInspectorDevServerHelper#55609
cortinico wants to merge 4 commits intofacebook:mainfrom
cortinico:export-D93596719

Conversation

@cortinico
Copy link
Contributor

Summary: Inject custom devsupport headers into the open-debugger POST request. This ensures the debugger launch request includes any registered custom headers.

Differential Revision: D93596719

@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 18, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 18, 2026

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

Summary:

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.

Reviewed By: cipolleschi

Differential Revision: D93490954
Summary:

Inject custom devsupport headers into bundle download requests. This ensures that multipart bundle fetches include any headers registered via `RCTDevSupportHttpHeaders`.

Reviewed By: cipolleschi

Differential Revision: D93596721
Summary:

Inject custom devsupport headers into the packager status check request. Changes NSURLRequest to NSMutableURLRequest to allow header mutation before the request is sent.

Reviewed By: cipolleschi

Differential Revision: D93596717
Summary:

Inject custom devsupport headers into the open-debugger POST request. This ensures the debugger launch request includes any registered custom headers.

Reviewed By: cipolleschi

Differential Revision: D93596719
cortinico added a commit to cortinico/react-native that referenced this pull request Feb 18, 2026
Summary:

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
cortinico added a commit to cortinico/react-native that referenced this pull request Feb 18, 2026
Summary:

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
cortinico added a commit to cortinico/react-native that referenced this pull request Feb 18, 2026
Summary:

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
cortinico added a commit to cortinico/react-native that referenced this pull request Feb 18, 2026
Summary:

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
cortinico added a commit to cortinico/react-native that referenced this pull request Feb 18, 2026
Summary:

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
@meta-codesync meta-codesync bot closed this in 583fa27 Feb 18, 2026
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Feb 18, 2026
@meta-codesync
Copy link

meta-codesync bot commented Feb 18, 2026

This pull request has been merged in 583fa27.

cortinico added a commit that referenced this pull request Feb 24, 2026
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
cortinico added a commit that referenced this pull request Feb 27, 2026
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
cortinico added a commit that referenced this pull request Mar 4, 2026
Co-authored-by: Riccardo Cipolleschi <cipolleschi@meta.com>
resolved: #55575
resolved: #55576
resolved: #55577
Fix per-tap OkHttpClient allocation in RedBoxContentView (#55578)
resolved: #55578
Fix thread-safety race in InspectorNetworkHelper (#55579)
resolved: #55579
resolved: #55580
resolved: #55581
resolved: #55582
resolved: #55586
resolved: #55607
resolved: #55608
resolved: #55609
resolved: #55610
resolved: #55611
resolved: #55612
resolved: #55790
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.

2 participants