From 618ed98882e76bca5afdf9b4ba0df2c7ea81786d Mon Sep 17 00:00:00 2001 From: Pieter De Baets Date: Wed, 28 May 2025 05:41:51 -0700 Subject: [PATCH] Set default XMLHttpRequest trackingName to undefined (#51656) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/51656 This fixes an issue with the internal networking layer when `enableModuleArgumentNSNullConversionIOS` is enabled, as we'd try to pass `NSNull` as trackingName instead of omitting it. ` Changelog: [Internal] Reviewed By: fabriziocucci Differential Revision: D75516619 fbshipit-source-id: 91a3bba32772fdd66edde1e24b7edd977918f727 --- packages/react-native/Libraries/Network/XMLHttpRequest.js | 4 ++-- .../Libraries/__tests__/__snapshots__/public-api-test.js.snap | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/react-native/Libraries/Network/XMLHttpRequest.js b/packages/react-native/Libraries/Network/XMLHttpRequest.js index d1a73cc2ab8..9208f614fb0 100644 --- a/packages/react-native/Libraries/Network/XMLHttpRequest.js +++ b/packages/react-native/Libraries/Network/XMLHttpRequest.js @@ -170,7 +170,7 @@ class XMLHttpRequest extends EventTarget { _sent: boolean; _url: ?string = null; _timedOut: boolean = false; - _trackingName: ?string = null; + _trackingName: string | void = undefined; _incrementalEvents: boolean = false; _startTime: ?number = null; _performanceLogger: IPerformanceLogger = GlobalPerformanceLogger; @@ -534,7 +534,7 @@ class XMLHttpRequest extends EventTarget { /** * Custom extension for tracking origins of request. */ - setTrackingName(trackingName: ?string): XMLHttpRequest { + setTrackingName(trackingName: string | void): XMLHttpRequest { this._trackingName = trackingName; return this; } diff --git a/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap b/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap index b8d5903830c..fe6a31f37a1 100644 --- a/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap +++ b/packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap @@ -6009,7 +6009,7 @@ declare class XMLHttpRequest extends EventTarget { getAllResponseHeaders(): ?string; getResponseHeader(header: string): ?string; setRequestHeader(header: string, value: any): void; - setTrackingName(trackingName: ?string): XMLHttpRequest; + setTrackingName(trackingName: string | void): XMLHttpRequest; setPerformanceLogger(performanceLogger: IPerformanceLogger): XMLHttpRequest; open(method: string, url: string, async: ?boolean): void; send(data: any): void;