Allow customer to change okHttpInstrumentationType in init call + updated docs

Author: FranAguilera

packages/react-native/README.md

@@ -149,6 +149,8 @@ setFeatureFlagExposure('beta_mode', false);
 
 The automatic capture of network requests can be achieved in a few different ways depending on the platform and the networking library being used.
 
+When initialization is done in JavaScript, always pass `enableNetworkInstrumentation: true` to `init(...)`. This is required on both iOS and Android.
+
 #### iOS
 
 When initializing via JS, pass `enableNetworkInstrumentation: true` as one of the options to `init`:
@@ -176,6 +178,21 @@ plugins {
 
 To find the version to use, use the same version of the Capture SDK that is being used in the React Native project. Check the `build.gradle` file in the `node_modules/@bitdrift/react-native/android` directory for the version of the Capture SDK being used.
 
+For Android, the plugin also needs instrumentation settings:
+
+```gradle
+capture {
+    instrumentation {
+        automaticOkHttpInstrumentation = true
+        okHttpInstrumentationType = OVERWRITE
+    }
+}
+```
+
+`okHttpInstrumentationType` modes:
+- `PROXY` (plugin default): preserves existing `EventListener.Factory` behavior in OkHttp clients.
+- `OVERWRITE`: replaces the listener factory and can avoid duplicate network spans in some app setups.
+
 In addition to this the plugin repository needs to be added to the `pluginManagement` block in the `settings.gradle` file:
 
 ```gradle
@@ -209,3 +226,26 @@ When using Expo to generate the project, this can be achieved by setting the `ne
   }
 }
 ```
+
+The Android plugin mode can be configured with `okHttpInstrumentationType`:
+
+```json
+{
+  "expo": {
+    "plugins": [
+      [
+        "@bitdrift/react-native",
+        {
+          "networkInstrumentation": true,
+          "okHttpInstrumentationType": "OVERWRITE"
+        }
+      ]
+    ]
+  }
+}
+```
+
+- `PROXY` (default): preserves existing `EventListener.Factory` behavior in OkHttp clients.
+- `OVERWRITE`: replaces the listener factory and can avoid duplicate network spans in some app setups.
+
+If you recently upgraded and started seeing duplicate spans on Android, set `okHttpInstrumentationType` to `OVERWRITE`.

packages/react-native/src/plugin/config.ts

@@ -7,4 +7,6 @@
 
 export default interface PluginProps {
   networkInstrumentation?: boolean;
+  // Only used when networkInstrumentation is true.
+  okHttpInstrumentationType?: 'PROXY' | 'OVERWRITE';
 }

packages/react-native/src/plugin/withAndroid.ts

@@ -19,12 +19,22 @@ const withBitdriftAppBuildGradle: ConfigPlugin<PluginProps | void> = (
       // TODO(snowp): Eventually we'd always install the plugin and gate the okhttp instrumentation
       // config on the networkInstrumentation prop.
       if (props && props.networkInstrumentation) {
+        const okHttpInstrumentationType =
+          props.okHttpInstrumentationType === 'OVERWRITE' ? 'OVERWRITE' : 'PROXY';
+
         // Add the capture-plugin at the very top of the file.
         config.modResults.contents =
           `plugins {
     id 'io.bitdrift.capture-plugin' version '0.22.12'
 }
 
+capture {
+    instrumentation {
+        automaticOkHttpInstrumentation = true
+        okHttpInstrumentationType = ${okHttpInstrumentationType}
+    }
+}
+
 ` + config.modResults.contents;
       }