noibu-react-native 0.2.5 → 0.2.7

package/dist/utils/function.js

@@ -1,17 +1,21 @@
 import { __awaiter } from 'tslib';
 import { Platform } from 'react-native';
 import { parseStack } from './stacktrace-parser.js';
-import { MAX_STRING_LENGTH, MAX_BEACON_PAYLOAD_SIZE, REQUIRED_DATA_PROCESSING_URLS, PII_EMAIL_PATTERN, PII_REDACTION_REPLACEMENT_STRING, PII_DIGIT_PATTERN, JS_STACK_LINE_ATT_NAME, DEFAULT_STACK_FRAME_FIELD_VALUE, JS_STACK_METHOD_ATT_NAME, JS_STACK_FILE_ATT_NAME } from '../constants.js';
+import { MAX_STRING_LENGTH, MAX_BEACON_PAYLOAD_SIZE, PII_EMAIL_PATTERN, PII_REDACTION_REPLACEMENT_STRING } from '../constants.js';
 import { noibuLog } from './log.js';
 import { unwrapNoibuWrapped } from './object.js';
 
+// default string for stack frame fields
+const DEFAULT_STACK_FRAME_FIELD_VALUE = '_';
+const PII_DIGIT_PATTERN = /[0-9]+/g;
+const STACK_TRACE_SANITIZE_REGEXP = /(nbuGlobalPromiseRejectWrapper|InternalBytecode)/gi;
 /**
  * Returns a stack trace frame with default filed values
  */
 const getDefaultFrame = () => ({
-    [JS_STACK_LINE_ATT_NAME]: DEFAULT_STACK_FRAME_FIELD_VALUE,
-    [JS_STACK_METHOD_ATT_NAME]: DEFAULT_STACK_FRAME_FIELD_VALUE,
-    [JS_STACK_FILE_ATT_NAME]: DEFAULT_STACK_FRAME_FIELD_VALUE,
+    line: DEFAULT_STACK_FRAME_FIELD_VALUE,
+    mname: DEFAULT_STACK_FRAME_FIELD_VALUE,
+    file: DEFAULT_STACK_FRAME_FIELD_VALUE,
 });
 /**
  * returns a string that satisfies a max length
@@ -55,22 +59,15 @@ function processFrames(rawFrames) {
         return processedFrame;
     });
 }
-/**
- * Retrieves the javascript stack and message from an error event object
- * @param errObj error to extract stack from
- */
+/** Retrieves the javascript stack and message from an error event object */
 function getJSStack(errObj) {
-    let frames = [getDefaultFrame()];
     // if the errObj type is not an object or null
     // return a default frame
     if (typeof errObj !== 'object' || !errObj || !errObj.stack) {
-        return {
-            frames,
-            msg: '',
-        };
+        return { frames: [getDefaultFrame()], msg: '' };
     }
-    frames = processFrames(parseStack(errObj.stack));
     const msg = errObj.message ? getMaxSubstringAllowed(errObj.message) : '';
+    const frames = processFrames(parseStack(errObj.stack)).filter(line => !`${line.file}|${line.mname}`.match(STACK_TRACE_SANITIZE_REGEXP));
     return {
         frames,
         msg,
@@ -111,63 +108,36 @@ function stringifyJSON(jsonObject) {
     });
 }
 /**
- * Wrapper for a request, since we have to do some special handling
- * @param method
- * @param url
- * @param data
- * @param headers
- * @param timeout
- * @param sendAndForget
+ * Wrapper for a request with an ability to send-and-forget
+ * a send-and-forget request can outlive the page
  */
-function makeRequest(method, url, data, headers, timeout, sendAndForget) {
-    return __awaiter(this, void 0, void 0, function* () {
-        const ua = Object.keys(headers).findLast(k => k.toLowerCase() === 'user-agent');
-        const headersWithUa = Object.assign({}, headers);
-        if (!headers[ua]) {
-            headersWithUa['User-Agent'] = yield getUserAgent();
-        }
-        // a send-and-forget request is made by using the beacon API (fetch + keepalive)
-        if (sendAndForget) {
-            const stringData = stringifyJSON(data);
-            const currentPayloadSize = new Blob([stringData]).size;
-            // if we have a large object or fetch is not available, we skip sending the message
-            if (currentPayloadSize > MAX_BEACON_PAYLOAD_SIZE) {
-                return Promise.resolve();
-            }
-            return unwrapNoibuWrapped(fetch)(url, {
-                method: 'POST',
-                headers: headersWithUa,
-                body: stringifyJSON(data),
-                // keep alive outlives the current page, its the same as beacon
-                keepalive: true,
-            });
-        }
-        return new Promise((resolve, reject) => {
-            const xhr = new XMLHttpRequest();
-            xhr.open(method, url);
-            xhr.timeout = timeout;
-            Object.keys(headersWithUa).forEach(header => {
-                xhr.setRequestHeader(header, headers[header]);
-            });
-            xhr.onload = () => {
-                if (xhr.status >= 200 && xhr.status < 300) {
-                    resolve(xhr.response);
-                }
-                else {
-                    reject(new Error(`Custom Request failed: ${xhr.statusText}`));
-                }
-            };
-            xhr.onerror = () => {
-                reject(new Error(`Custom Request failed: ${xhr.statusText}`));
-            };
-            if (data) {
-                xhr.send(stringifyJSON(data));
-            }
-            else {
-                xhr.send();
-            }
-        });
-    });
+function postRequest(url, data, headers, sendAndForget = false, timeout = 2000) {
+    if (typeof fetch !== 'function') {
+        return Promise.resolve();
+    }
+    const body = asString(data);
+    const currentPayloadSize = new Blob([body]).size;
+    if (sendAndForget && currentPayloadSize > MAX_BEACON_PAYLOAD_SIZE) {
+        return Promise.reject("Can't queue data for transfer due to it's size.");
+    }
+    const init = {
+        method: 'POST',
+        headers,
+        body,
+        keepalive: sendAndForget,
+    };
+    let timer;
+    if (typeof AbortController !== 'undefined') {
+        const controller = new AbortController();
+        init.signal = controller.signal;
+        timer = setTimeout(() => controller.abort('Timeout exceeded'), timeout);
+    }
+    const originalFetch = unwrapNoibuWrapped(fetch);
+    const promise = originalFetch(url, init).then(() => { });
+    if (typeof AbortController !== 'undefined') {
+        promise.finally(() => clearTimeout(timer));
+    }
+    return promise;
 }
 /**
  * makes sure the url sent is a valid URL
@@ -202,20 +172,6 @@ function getUserAgent() {
         return userAgentCache;
     });
 }
-/**
- * isInvalidURLConfig will verify that Collect is being initializes with
- * the correct env vars.
- */
-function isInvalidURLConfig(urls) {
-    for (let i = 0; i < REQUIRED_DATA_PROCESSING_URLS.length; i += 1) {
-        const u = REQUIRED_DATA_PROCESSING_URLS[i];
-        if (!urls[u]) {
-            noibuLog('urlConfig invalid, reason', u);
-            return true;
-        }
-    }
-    return false;
-}
 /**
  * isNoibuJSAlreadyLoaded will verify if there are already other
  * copies of NoibuJS runnung
@@ -249,9 +205,7 @@ function asString(obj) {
  */
 function maskTextInput(text) {
     // if it has an email or digit(s), we mask the text
-    return text
-        .replace(PII_EMAIL_PATTERN, PII_REDACTION_REPLACEMENT_STRING)
-        .replace(PII_DIGIT_PATTERN, '*');
+    return text.replace(PII_EMAIL_PATTERN, PII_REDACTION_REPLACEMENT_STRING).replace(PII_DIGIT_PATTERN, '*');
 }
 /**
  * Checks if the provided object is an instance of the specified type.
@@ -266,20 +220,6 @@ function isInstanceOf(instance, type) {
         return false;
     }
 }
-/**
- * To grab the video recorder type based on the device we run the app on.
- */
-function getVideoRecorderType() {
-    return __awaiter(this, void 0, void 0, function* () {
-        if (Platform.OS === 'android') {
-            return 'AndroidNative';
-        }
-        if (Platform.OS === 'ios') {
-            return 'IOSNative';
-        }
-        return '';
-    });
-}
 /** String.trim, but safe */
 function safeTrim(text) {
     if (typeof text !== 'string') {
@@ -290,9 +230,6 @@ function safeTrim(text) {
 /**
  * Tries to get the stack trace from the given error object and returns it.
  * If the error object does not have a stack trace, an empty string is returned.
- *
- * @param {Error} error - The error object from which to retrieve the stack trace.
- * @returns {string} The stack trace of the error, if available. Otherwise, an empty string is returned.
  */
 function tryGetStackTrace(error) {
     let result = '';
@@ -306,13 +243,5 @@ function tryGetStackTrace(error) {
     }
     return result;
 }
-/**
- * Checks whether the given value is a string or an instance of String.
- * @param {*} value - The value to be checked.
- * @returns {boolean} Returns true if the value is a string or an instance of String, otherwise returns false.
- */
-function isString(value) {
-    return typeof value === 'string' || value instanceof String;
-}
 
-export { asString, getJSStack, getMaxSubstringAllowed, getUserAgent, getVideoRecorderType, isInstanceOf, isInvalidURLConfig, isNoibuJSAlreadyLoaded, isStackTrace, isString, isValidURL, makeRequest, maskTextInput, processFrames, safeTrim, stringifyJSON, tryGetStackTrace };
+export { asString, getJSStack, getMaxSubstringAllowed, getUserAgent, isInstanceOf, isNoibuJSAlreadyLoaded, isStackTrace, isValidURL, maskTextInput, postRequest, processFrames, safeTrim, stringifyJSON, tryGetStackTrace };

package/dist/utils/log.js

@@ -4,12 +4,12 @@
  * checks if was overridden and calls original console function
  */
 const getConsoleMethod = (ogProp) => {
-    // @ts-ignore
+    // @ts-expect-error __noibu_original__ is not a standard property
     return console[ogProp].__noibu_original__ || console[ogProp];
 };
-/**
- * log with level = info
- */
+/**log with level = info */
 const noibuLog = (...msgs) => getConsoleMethod('log')('Noibu', ...msgs);
+/**log with level = error */
+const noibuErr = (...msgs) => getConsoleMethod('error')('Noibu', ...msgs);
 
-export { noibuLog };
+export { noibuErr, noibuLog };

package/dist/utils/object.js

@@ -6,16 +6,13 @@
  * attributeName: the attribute key whose value will be replace
  * processingFunction: function that accepts the original value
  * and returns the newValue
- * @param  {} sourceObject
- * @param  {} attributeName
- * @param  {} processingFunction
  */
-const replace = (sourceObject, attributeName, processingFunction) => {
+function replace(sourceObject, attributeName, processingFunction) {
     if (!(attributeName in sourceObject)) {
         return;
     }
     const originalAttribute = sourceObject[attributeName];
-    const newValue = processingFunction(originalAttribute);
+    const newValue = processingFunction.call(sourceObject, originalAttribute);
     if (typeof newValue === 'function') {
         try {
             newValue.prototype = newValue.prototype || {};
@@ -40,14 +37,19 @@ const replace = (sourceObject, attributeName, processingFunction) => {
     }
     // eslint-disable-next-line no-param-reassign
     sourceObject[attributeName] = newValue;
-};
+}
 /**
  * unwraps wrapped property, so we can use it without side effects
- * @param anything
  */
 function unwrapNoibuWrapped(anything) {
     return anything.__noibu_original__ || anything;
 }
+/**
+ * Checks if the given object is wrapped
+ */
+function isNoibuWrapped(anything) {
+    return '__noibu_original__' in anything;
+}
 /**
  * Checks whether the prototype's property is writeable. If it is not,
  * checks whether the property can be made writeable. If it can, it is
@@ -55,9 +57,7 @@ function unwrapNoibuWrapped(anything) {
  * returns Whether the property on the prototype is (or is now) writeable
  */
 const propWriteableOrMadeWriteable = (proto, property) => {
-    if (!proto ||
-        !proto.hasOwnProperty ||
-        !Object.prototype.hasOwnProperty.call(proto, property)) {
+    if (!proto || !proto.hasOwnProperty || !Object.prototype.hasOwnProperty.call(proto, property)) {
         return false;
     }
     // Getting the properties that this the prototype
@@ -82,7 +82,7 @@ const propWriteableOrMadeWriteable = (proto, property) => {
  * Iterates object recursively and calls visit function
  * for each property allowing to override its value
  * @param {Object} instance An object to iterate through
- * @param {Function} visit A callback function that is called for each property
+ * @param {Function} visit Target callback function that is called for each property
  * There are 3 arguments: current object, current property and its value
  * @param {{depth: number}} limit Use limit config object to set depth of the recursion
  */
@@ -171,4 +171,4 @@ const safeFromEntries = (entries) => {
     return obj;
 };
 
-export { iterateObjectRecursively, propWriteableOrMadeWriteable, replace, safeEntries, safeFromEntries, unwrapNoibuWrapped };
+export { isNoibuWrapped, iterateObjectRecursively, propWriteableOrMadeWriteable, replace, safeEntries, safeFromEntries, unwrapNoibuWrapped };

package/dist/utils/piiRedactor.js

@@ -1,7 +1,36 @@
 import { iterateObjectRecursively } from './object.js';
-import { HTTP_PII_BLOCKING_PATTERNS, PII_REDACTION_REPLACEMENT_STRING } from '../constants.js';
+import { PII_REDACTION_REPLACEMENT_STRING, PII_EMAIL_PATTERN } from '../constants.js';
 import { stringifyJSON } from './function.js';
 
+// Regex patterns for a best-effort attempt at blocking PII in HTTP requests
+const HTTP_PII_BLOCKING_PATTERNS = [
+    // Match credit cards [https://www.regular-expressions.info/creditcard.html]
+    // Visa
+    /\b4\d{12}(?:\d{3})?\b/g,
+    // MasterCard
+    /\b(?:5[1-5]\d{2}|222[1-9]|22[3-9]\d|2[3-6]\d{2}|27[01]\d|2720)\d{12}\b/g,
+    // Amex
+    /\b3[47]\d{13}\b/g,
+    // Diners Club
+    /\b3(?:0[0-5]|[68]\d)\d{11}\b/g,
+    // Discover
+    /\b6(?:011|5\d{2})\d{12}\b/g,
+    // JCB
+    /\b(?:2131|1800|35\d{3})\d{11}\b/g,
+    // Emails [https://www.regular-expressions.info/email.html]
+    PII_EMAIL_PATTERN,
+    // US SSN with or without dashes
+    // [https://www.oreilly.com/library/view/regular-expressions-cookbook/9781449327453/ch04s12.html]
+    /\b(?!000|666)[0-8]\d{2}[-.● ]?(?!00)\d{2}[-.● ]?(?!0000)\d{4}\b/g,
+    // Canadian SIN with or without dashes [https://regexpattern.com/social-insurance-number-ca]
+    /\b(\d{3}[-.● ]?\d{3}[-.● ]?\d{3})\b/g,
+    // International phone numbers
+    // [https://www.oreilly.com/library/view/regular-expressions-cookbook/9781449327453/ch04s03.html]
+    /\+(?:\d●?){6,14}\d\b/g,
+    // US/Canada phone numbers
+    // [https://www.oreilly.com/library/view/regular-expressions-cookbook/9781449327453/ch04s02.html]
+    /(\b|\+)?(1[-.● ]?)?\(?(\d{3})\)?[-.● ]?(\d{3})[-.● ]?(\d{4})\b/g,
+];
 const fuzzyFieldsToRedact = [
     'password',
     'address',
@@ -35,8 +64,7 @@ const exactFieldsToRedact = [
  */
 function shouldRedact(field) {
     // check for exact and fuzzy matches
-    return (exactFieldsToRedact.some(v => field === v) ||
-        fuzzyFieldsToRedact.some(v => field.indexOf(v) >= 0));
+    return exactFieldsToRedact.some(v => field === v) || fuzzyFieldsToRedact.some(v => field.indexOf(v) >= 0);
 }
 /**
  * Try to parse content as a JSON object and

package/dist/utils/stacktrace-parser.js

@@ -1,4 +1,4 @@
-import { MAX_FRAMES_IN_ARRAY, MAX_STRING_LENGTH } from '../constants.js';
+import { MAX_STRING_LENGTH } from '../constants.js';
 
 /* eslint-disable require-jsdoc,prefer-destructuring,camelcase */
 // This is a loose copy of ravenjs code
@@ -15,13 +15,14 @@ import { MAX_FRAMES_IN_ARRAY, MAX_STRING_LENGTH } from '../constants.js';
 // or substantial portions of the Software.
 //
 // THE SOFTWARE IS PROVIDED 'AS IS', WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
-// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
+// INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR Target
 // PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
 // HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF
 // CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
 // OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 // global reference to slice
 const UNKNOWN_FUNCTION = '<unknown>';
+const MAX_FRAMES_IN_ARRAY = 15;
 /**
  * Safari web extensions, starting version unknown, can produce "frames-only" stacktraces.
  * What it means, is that instead of format like:
@@ -48,9 +49,7 @@ const extractSafariExtensionDetails = (func, filename) => {
     return isSafariExtension || isSafariWebExtension
         ? [
             func.indexOf('@') !== -1 ? func.split('@')[0] : UNKNOWN_FUNCTION,
-            isSafariExtension
-                ? `safari-extension:${filename}`
-                : `safari-web-extension:${filename}`,
+            isSafariExtension ? `safari-extension:${filename}` : `safari-web-extension:${filename}`,
         ]
         : [func, filename];
 };
@@ -68,8 +67,8 @@ function createFrame(filename, func, lineno, colno) {
     return frame;
 }
 // Chromium based browsers: Chrome, Brave, new Opera, new Edge
-const chromeRegex = /^\s*at (?:(.+?\)(?: \[.+\])?|.*?) ?\((?:address at )?)?(?:async )?((?:<anonymous>|[-a-z]+:|.*bundle|\/)?.*?)(?::(\d+))?(?::(\d+))?\)?\s*$/i;
-const chromeEvalRegex = /\((\S*)(?::(\d+))(?::(\d+))\)/;
+const chromeRegex = /^\s*at (?:(.+?\)(?: \[.+])?|.*?) ?\((?:address at )?)?(?:async )?((?:<anonymous>|[-a-z]+:|.*bundle|\/)?.*?)(?::(\d+))?(?::(\d+))?\)?\s*$/i;
+const chromeEvalRegex = /\((\S*):(\d+):(\d+)\)/;
 const chrome = (line) => {
     const parts = chromeRegex.exec(line);
     if (parts) {
@@ -89,20 +88,29 @@ const chrome = (line) => {
     return undefined;
 };
 function parseStack(stackString) {
-    const lines = stackString.split('\n');
-    if (lines.length > MAX_FRAMES_IN_ARRAY) {
-        return [];
-    }
-    return lines.reduce((stack, line) => {
-        if (line.length > MAX_STRING_LENGTH) {
+    if (typeof stackString === 'string') {
+        return stackString
+            .split('\n')
+            .slice(0, MAX_FRAMES_IN_ARRAY + 1)
+            .reduce((stack, line) => {
+            if (line.length > MAX_STRING_LENGTH) {
+                return stack;
+            }
+            const parseResult = chrome(line);
+            if (parseResult) {
+                stack.push(parseResult);
+            }
             return stack;
-        }
-        const parseResult = chrome(line);
-        if (parseResult) {
-            stack.push(parseResult);
-        }
-        return stack;
-    }, []);
+        }, []);
+    }
+    if (stackString.length && stackString[0].file) {
+        return stackString.map(frame => ({
+            file: frame.file || '<unknown>',
+            line: frame.lineNumber,
+            mname: `${frame.class}.${frame.methodName}`,
+        }));
+    }
+    return [];
 }
 
-export { parseStack };
+export { MAX_FRAMES_IN_ARRAY, parseStack };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "noibu-react-native",
-  "version": "0.2.5",
+  "version": "0.2.7",
   "targetNjsVersion": "1.0.104",
   "description": "React-Native SDK for NoibuJS to collect errors in React-Native applications",
   "main": "dist/entry/index.js",
@@ -16,12 +16,10 @@
     "clean": "rimraf ./dist/*",
     "build": "node ./build.js",
     "build:dev": "node ./build.watch.js",
-    "prepare": "npm run clean; npm run build;",
+    "prepack": "npm run clean; npm run build;",
     "test": "jest --coverage",
     "tsc": "tsc",
-    "lint": "eslint src -c .eslintrc.json --ext js,ts,jsx,tsx",
-    "lint_output": "eslint src -c .eslintrc.json --ext js,ts,jsx,tsx -f json > eslint_report.json",
-    "codecov": "codecov"
+    "lint": "tsc eslint.config.ts && eslint ."
   },
   "peerDependenciesMeta": {
     "react-native-navigation": {
@@ -42,13 +40,16 @@
     "react-native": ">=0.63.0",
     "react-native-navigation": ">=2.29.0",
     "react-native-url-polyfill": "^1.3.0",
-    "react-native-uuid": "^2.0.1"
+    "react-native-uuid": "^2.0.1",
+    "whatwg-fetch": "^3.6.20"
   },
   "devDependencies": {
     "@babel/core": "^7.26.0",
     "@babel/preset-env": "^7.26.0",
     "@babel/preset-typescript": "^7.26.0",
+    "@eslint/eslintrc": "^3.2.0",
     "@jest/globals": "^29.7.0",
+    "@noibu/metroplex-ts-bindings": "1.0.24",
     "@react-native-async-storage/async-storage": "^1.19.0",
     "@rollup/plugin-commonjs": "^25.0.0",
     "@rollup/plugin-json": "^6.0.0",
@@ -60,20 +61,19 @@
     "@types/node": "^20.2.3",
     "@types/react": "16.14.62",
     "@types/react-test-renderer": "^18.0.0",
-    "@typescript-eslint/eslint-plugin": "^6.21.0",
-    "@typescript-eslint/parser": "^6.21.0",
+    "@typescript-eslint/eslint-plugin": "^8.19.1",
+    "@typescript-eslint/parser": "^8.19.1",
     "babel-jest": "^29.7.0",
     "babel-plugin-transform-flow-strip-types": "^6.22.0",
     "codecov": "^3.8.3",
     "dotenv": "^16.1.3",
-    "eslint": "^8.41.0",
-    "eslint-config-airbnb": "^19.0.4",
-    "eslint-config-prettier": "^8.8.0",
-    "eslint-plugin-jsdoc": "^44.2.4",
-    "eslint-plugin-prettier": "^4.2.1",
+    "eslint": "^9.17.0",
+    "eslint-plugin-jsdoc": "^50.6.1",
+    "eslint-plugin-prettier": "^5.2.1",
+    "eslint-plugin-react": "^7.37.3",
     "fflate": "0.8.2",
     "jest": "^29.7.0",
-    "prettier": "^2.8.8",
+    "prettier": "^3.4.2",
     "react": "16.13.1",
     "react-native": "0.63.0",
     "react-native-navigation": "^7.40.3",