feat: exclude DEV server url from network logs

Author: ahmedAlaaInstabug

src/modules/Instabug.ts

@@ -633,6 +633,13 @@ export const willRedirectToStore = () => {
   NativeInstabug.willRedirectToStore();
 };
 
+/**
+ * This API has be called when changing the default Metro server port (8081) to exclude the DEV URL from network logging.
+ */
+export const setMetroDevServerPort = (port: number) => {
+  InstabugConstants.METRO_SERVER_URL = port.toString();
+};
+
 export const componentDidAppearListener = (event: ComponentDidAppearEvent) => {
   if (_isFirstScreen) {
     _lastScreen = event.componentName;

src/modules/NetworkLogger.ts

@@ -3,28 +3,18 @@ import type { RequestHandler } from '@apollo/client';
 import InstabugConstants from '../utils/InstabugConstants';
 import xhr, { NetworkData, ProgressCallback } from '../utils/XhrNetworkInterceptor';
 import { isContentTypeNotAllowed, reportNetworkLog } from '../utils/InstabugUtils';
-import { NativeModules } from 'react-native';
+import { metroDevServerPort } from './Instabug';
 
 export type { NetworkData };
 
 export type NetworkDataObfuscationHandler = (data: NetworkData) => Promise<NetworkData>;
 let _networkDataObfuscationHandler: NetworkDataObfuscationHandler | null | undefined;
 let _requestFilterExpression = 'false';
 
-const filterURL = (url: string): string => {
-  const [protocol, rest] = url.split('://');
-  return (rest || protocol).split('/')[0];
-};
-
-const getDevServerURL = (): string | null => {
-  if (NativeModules.SourceCode) {
-    const { scriptURL } = NativeModules.SourceCode;
-    if (scriptURL) {
-      return filterURL(scriptURL.toString());
-    }
-  }
-  return null;
-};
+function getPortFromUrl(url: string) {
+  const portMatch = url.match(/:(\d+)(?=\/|$)/);
+  return portMatch ? portMatch[1] : null;
+}
 
 /**
  * Sets whether network logs should be sent with bug reports.
@@ -44,10 +34,8 @@ export const setEnabled = (isEnabled: boolean) => {
           }
 
           if (__DEV__) {
-            const devServerURL = getDevServerURL();
-            const networkServerURL = filterURL(network.url);
-            // Skip logging for requests to the dev server
-            if (devServerURL === networkServerURL) {
+            const urlPort = getPortFromUrl(network.url);
+            if (urlPort === InstabugConstants.METRO_SERVER_URL) {
               return;
             }
           }

src/utils/InstabugConstants.ts

@@ -11,6 +11,7 @@ const InstabugConstants = {
     'IBG-RN: Expected key and value passed to setUserAttribute to be of type string',
   REMOVE_USER_ATTRIBUTES_ERROR_TYPE_MESSAGE:
     'IBG-RN: Expected key and value passed to removeUserAttribute to be of type string',
+  METRO_SERVER_URL: '8081',
 };
 
 export default InstabugConstants;