Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

#import <React/RCTConvert.h>
#import <React/RCTDefines.h>
#import <React/RCTDevSupportHttpHeaders.h>

#import <SocketRocket/SRWebSocket.h>

Expand Down Expand Up @@ -46,7 +47,9 @@ - (void)start
{
[self stop];
_stopped = NO;
_socket = [[SRWebSocket alloc] initWithURL:_url];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:_url];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:request];
_socket = [[SRWebSocket alloc] initWithURLRequest:request];
_socket.delegate = self;
[_socket setDelegateDispatchQueue:_delegateDispatchQueue];
[_socket open];
Expand Down
8 changes: 5 additions & 3 deletions packages/react-native/React/Base/RCTBundleURLProvider.mm
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
#import "RCTConstants.h"
#import "RCTConvert.h"
#import "RCTDefines.h"
#import "RCTDevSupportHttpHeaders.h"
#import "RCTLog.h"

#import <jsinspector-modern/InspectorFlags.h>
Expand Down Expand Up @@ -93,9 +94,10 @@ + (BOOL)isPackagerRunning:(NSString *)hostPort scheme:(NSString *)scheme
NSURL *url = [serverRootWithHostPort(hostPort, scheme) URLByAppendingPathComponent:@"status"];

NSURLSession *session = [NSURLSession sharedSession];
NSURLRequest *request = [NSURLRequest requestWithURL:url
cachePolicy:NSURLRequestUseProtocolCachePolicy
timeoutInterval:10];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:url
cachePolicy:NSURLRequestUseProtocolCachePolicy
timeoutInterval:10];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:request];
__block NSURLResponse *response;
__block NSData *data;

Expand Down
24 changes: 24 additions & 0 deletions packages/react-native/React/Base/RCTDevSupportHttpHeaders.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
/*
* Copyright (c) Meta Platforms, Inc. and affiliates.
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*/

#import <Foundation/Foundation.h>

/**
* Thread-safe singleton that holds custom HTTP headers to be applied
* to all devsupport network requests (bundle fetches, packager status
* checks, inspector and HMR WebSocket connections).
*/
@interface RCTDevSupportHttpHeaders : NSObject

+ (instancetype)sharedInstance;

- (void)addRequestHeader:(NSString *)name value:(NSString *)value;
- (void)removeRequestHeader:(NSString *)name;
- (NSDictionary<NSString *, NSString *> *)allHeaders;
- (void)applyHeadersToRequest:(NSMutableURLRequest *)request;

@end
65 changes: 65 additions & 0 deletions packages/react-native/React/Base/RCTDevSupportHttpHeaders.m
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
/*
* Copyright (c) Meta Platforms, Inc. and affiliates.
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*/

#import "RCTDevSupportHttpHeaders.h"

@implementation RCTDevSupportHttpHeaders {
NSMutableDictionary<NSString *, NSString *> *_headers;
dispatch_queue_t _queue;
}

+ (instancetype)sharedInstance
{
static RCTDevSupportHttpHeaders *sharedInstance;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
sharedInstance = [[RCTDevSupportHttpHeaders alloc] init];
});
return sharedInstance;
}

- (instancetype)init
{
if (self = [super init]) {
_headers = [NSMutableDictionary new];
_queue = dispatch_queue_create("com.facebook.react.RCTDevSupportHttpHeaders", DISPATCH_QUEUE_SERIAL);
}
return self;
}

- (void)addRequestHeader:(NSString *)name value:(NSString *)value
{
dispatch_sync(_queue, ^{
self->_headers[name] = value;
});
}

- (void)removeRequestHeader:(NSString *)name
{
dispatch_sync(_queue, ^{
[self->_headers removeObjectForKey:name];
});
}

- (NSDictionary<NSString *, NSString *> *)allHeaders
{
__block NSDictionary<NSString *, NSString *> *snapshot;
dispatch_sync(_queue, ^{
snapshot = [self->_headers copy];
});
return snapshot;
}

- (void)applyHeadersToRequest:(NSMutableURLRequest *)request
{
NSDictionary<NSString *, NSString *> *headers = [self allHeaders];
[headers enumerateKeysAndObjectsUsingBlock:^(NSString *headerName, NSString *headerValue, BOOL *stop) {
[request setValue:headerValue forHTTPHeaderField:headerName];
}];
}

@end
3 changes: 3 additions & 0 deletions packages/react-native/React/Base/RCTMultipartDataTask.m
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@

#import "RCTMultipartDataTask.h"

#import "RCTDevSupportHttpHeaders.h"

@interface RCTMultipartDataTask () <NSURLSessionDataDelegate, NSURLSessionDataDelegate>

@end
Expand Down Expand Up @@ -40,6 +42,7 @@ - (void)startTask
delegateQueue:nil];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:_url];
[request addValue:@"multipart/mixed" forHTTPHeaderField:@"Accept"];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:request];
NSURLSessionDataTask *dataTask = [session dataTaskWithRequest:request];
[dataTask resume];
[session finishTasksAndInvalidate];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@

#import <React/RCTCxxInspectorPackagerConnection.h>
#import <React/RCTDefines.h>
#import <React/RCTDevSupportHttpHeaders.h>

#import <CommonCrypto/CommonCrypto.h>
#import <jsinspector-modern/InspectorFlags.h>
Expand Down Expand Up @@ -154,6 +155,7 @@ + (void)openDebugger:(NSURL *)bundleURL withErrorMessage:(NSString *)errorMessag
escapedInspectorDeviceId]];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:url];
[request setHTTPMethod:@"POST"];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:request];

[[[NSURLSession sharedSession]
dataTaskWithRequest:request
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
*/

#import "RCTInspectorNetworkHelper.h"
#import <React/RCTDevSupportHttpHeaders.h>
#import <React/RCTLog.h>

using ListenerBlock = void (^)(RCTInspectorNetworkListener *);
Expand Down Expand Up @@ -47,6 +48,7 @@ - (void)loadNetworkResourceWithParams:(const RCTInspectorLoadNetworkResourceRequ

NSMutableURLRequest *urlRequest = [NSMutableURLRequest requestWithURL:url];
[urlRequest setHTTPMethod:@"GET"];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:urlRequest];
NSURLSessionDataTask *dataTask = [self.session dataTaskWithRequest:urlRequest];
__weak NSURLSessionDataTask *weakDataTask = dataTask;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

#if RCT_DEV || RCT_REMOTE_PROFILE

#import <React/RCTDevSupportHttpHeaders.h>
#import <React/RCTInspector.h>
#import <React/RCTInspectorPackagerConnection.h>
#import <React/RCTLog.h>
Expand Down Expand Up @@ -36,7 +37,10 @@ - (instancetype)initWithURL:(const std::string &)url delegate:(std::weak_ptr<IWe
{
if ((self = [super init]) != nullptr) {
_delegate = delegate;
_webSocket = [[SRWebSocket alloc] initWithURL:[NSURL URLWithString:NSStringFromUTF8StringView(url)]];
NSURL *requestURL = [NSURL URLWithString:NSStringFromUTF8StringView(url)];
NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:requestURL];
[[RCTDevSupportHttpHeaders sharedInstance] applyHeadersToRequest:request];
_webSocket = [[SRWebSocket alloc] initWithURLRequest:request];
_webSocket.delegate = self;
[_webSocket open];
}
Expand Down
7 changes: 7 additions & 0 deletions packages/react-native/ReactAndroid/api/ReactAndroid.api
Original file line number Diff line number Diff line change
Expand Up @@ -2212,6 +2212,13 @@ public abstract interface class com/facebook/react/devsupport/interfaces/DevSupp
public abstract fun onResume ()V
}

public final class com/facebook/react/devsupport/interfaces/DevSupportRequestHeaders {
public static final field INSTANCE Lcom/facebook/react/devsupport/interfaces/DevSupportRequestHeaders;
public static final fun addRequestHeader (Ljava/lang/String;Ljava/lang/String;)V
public static final fun allHeaders ()Ljava/util/Map;
public static final fun removeRequestHeader (Ljava/lang/String;)V
}

public abstract interface class com/facebook/react/devsupport/interfaces/ErrorCustomizer {
public abstract fun customizeErrorInfo (Landroid/util/Pair;)Landroid/util/Pair;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,9 @@ import android.os.Looper
import com.facebook.jni.HybridData
import com.facebook.proguard.annotations.DoNotStrip
import com.facebook.proguard.annotations.DoNotStripAny
import com.facebook.react.devsupport.inspector.DevSupportHttpClient
import com.facebook.soloader.SoLoader
import java.io.Closeable
import java.util.concurrent.TimeUnit
import okhttp3.OkHttpClient
import okhttp3.Request
import okhttp3.Response
import okhttp3.WebSocket
Expand Down Expand Up @@ -78,12 +77,7 @@ internal class CxxInspectorPackagerConnection(

/** Java implementation of the C++ InspectorPackagerConnectionDelegate interface. */
private class DelegateImpl {
private val httpClient =
OkHttpClient.Builder()
.connectTimeout(10, TimeUnit.SECONDS)
.writeTimeout(10, TimeUnit.SECONDS)
.readTimeout(0, TimeUnit.MINUTES) // Disable timeouts for read
.build()
private val httpClient = DevSupportHttpClient.websocketClient

private val handler = Handler(Looper.getMainLooper())

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ import com.facebook.react.bridge.ReactContext
import com.facebook.react.common.ReactConstants
import com.facebook.react.devsupport.InspectorFlags.getFuseboxEnabled
import com.facebook.react.devsupport.InspectorFlags.getIsProfilingBuild
import com.facebook.react.devsupport.inspector.DevSupportHttpClient
import com.facebook.react.devsupport.interfaces.DevBundleDownloadListener
import com.facebook.react.devsupport.interfaces.PackagerStatusCallback
import com.facebook.react.modules.debug.interfaces.DeveloperSettings
Expand All @@ -39,7 +40,6 @@ import java.io.UnsupportedEncodingException
import java.security.MessageDigest
import java.security.NoSuchAlgorithmException
import java.util.Locale
import java.util.concurrent.TimeUnit
import okhttp3.Call
import okhttp3.Callback
import okhttp3.OkHttpClient
Expand Down Expand Up @@ -86,12 +86,7 @@ public open class DevServerHelper(
MAP("map"),
}

private val client: OkHttpClient =
OkHttpClient.Builder()
.connectTimeout(HTTP_CONNECT_TIMEOUT_MS.toLong(), TimeUnit.MILLISECONDS)
.readTimeout(0, TimeUnit.MILLISECONDS)
.writeTimeout(0, TimeUnit.MILLISECONDS)
.build()
private val client: OkHttpClient = DevSupportHttpClient.httpClient
private val bundleDownloader: BundleDownloader = BundleDownloader(client)
private val packagerStatusCheck: PackagerStatusCheck = PackagerStatusCheck(client)
private val packageName: String = applicationContext.packageName
Expand Down Expand Up @@ -397,7 +392,6 @@ public open class DevServerHelper(
}

private companion object {
private const val HTTP_CONNECT_TIMEOUT_MS = 5000
private const val DEBUGGER_MSG_DISABLE = "{ \"id\":1,\"method\":\"Debugger.disable\" }"

private fun getSHA256(string: String): String {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,33 +11,20 @@ package com.facebook.react.devsupport

import com.facebook.common.logging.FLog
import com.facebook.react.common.ReactConstants
import com.facebook.react.devsupport.inspector.DevSupportHttpClient
import com.facebook.react.devsupport.interfaces.PackagerStatusCallback
import java.io.IOException
import java.util.Locale
import java.util.concurrent.TimeUnit
import okhttp3.Call
import okhttp3.Callback
import okhttp3.OkHttpClient
import okhttp3.Request
import okhttp3.Response

/** Use this class to check if the JavaScript packager is running on the provided host. */
internal class PackagerStatusCheck {
internal class PackagerStatusCheck(private val client: OkHttpClient) {

private val client: OkHttpClient

constructor() {
client =
OkHttpClient.Builder()
.connectTimeout(HTTP_CONNECT_TIMEOUT_MS.toLong(), TimeUnit.MILLISECONDS)
.readTimeout(0, TimeUnit.MILLISECONDS)
.writeTimeout(0, TimeUnit.MILLISECONDS)
.build()
}

constructor(client: OkHttpClient) {
this.client = client
}
constructor() : this(DevSupportHttpClient.httpClient)

fun run(host: String, callback: PackagerStatusCallback) {
val statusURL = createPackagerStatusURL(host)
Expand Down Expand Up @@ -92,7 +79,6 @@ internal class PackagerStatusCheck {

private companion object {
private const val PACKAGER_OK_STATUS = "packager-status:running"
private const val HTTP_CONNECT_TIMEOUT_MS = 5_000
private const val PACKAGER_STATUS_URL_TEMPLATE = "http://%s/status"

private fun createPackagerStatusURL(host: String): String =
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@ import android.widget.TextView
import com.facebook.common.logging.FLog
import com.facebook.react.R
import com.facebook.react.common.ReactConstants
import com.facebook.react.devsupport.inspector.DevSupportHttpClient
import com.facebook.react.devsupport.interfaces.DevSupportManager
import com.facebook.react.devsupport.interfaces.ErrorType
import com.facebook.react.devsupport.interfaces.RedBoxHandler
import com.facebook.react.devsupport.interfaces.StackFrame
import okhttp3.MediaType
import okhttp3.OkHttpClient
import okhttp3.Request
import okhttp3.RequestBody
import org.json.JSONObject
Expand Down Expand Up @@ -165,7 +165,7 @@ internal class RedBoxContentView(
.query(null)
.build()
.toString()
val client = OkHttpClient()
val client = DevSupportHttpClient.httpClient
for (frame in stackFrames) {
val payload = stackFrameToJson(checkNotNull(frame)).toString()
val body: RequestBody = RequestBody.create(JSON, payload)
Expand Down
Loading
Loading