rollbar 2.26.3 → 2.26.4

package/.github/workflows/ci.yml

@@ -2,30 +2,52 @@ name: Rollbar.js CI
 
 on:
   push:
-    branches: [ master ]
-    tags: [ v* ]
+    branches: [master]
+    tags: [v*]
   pull_request:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   build:
     runs-on: ubuntu-20.04
+
     strategy:
       matrix:
-        node-version: [10, 12, 14, 16]
+        include:
+          - node: 14
+            npm: ^8
+          - node: 16
+            npm: ^8
+          - node: 18
+            npm: ^9
+          - node: 20
+            npm: ^10
+          - node: latest
+            npm: latest
 
     steps:
-      - uses: actions/checkout@v2
-        with:
-          submodules: recursive
+      - name: Checkout
+        uses: actions/checkout@v4
 
-      - name: Install node.js
-        uses: actions/setup-node@v2-beta
+      - name: Set up node ${{ matrix.node }}
+        uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: ${{ matrix.node }}
+
+      - name: Update npm
+        run: npm install -g npm@${{ matrix.npm }}
 
       - name: npm install
         run: npm install
 
+      - name: Lint
+        uses: wearerequired/lint-action@v2
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          prettier: true
+          eslint: true
+          eslint_args: '--max-warnings 0'
+          eslint_extensions: js
+
       - name: Run tests
         run: npm run test_ci

package/.lgtm.yml

@@ -1,8 +1,8 @@
 path_classifiers:
-    generated:
-      - release
-      - dist
-    docs:
-      - examples
-    test:
-      - test
+  generated:
+    - release
+    - dist
+  docs:
+    - examples
+  test:
+    - test

package/.prettierignore

@@ -0,0 +1,18 @@
+.github
+.next
+.yalc
+build
+bundles
+coverage
+dist
+lib
+node_modules
+out
+public
+release
+vendor
+
+# ignore prettifying examples for now due to
+# 1. there being minified snippets all over the place and
+# 2. some tests fail cause they depend on some sources being exactly as they are
+examples

package/.vscode/settings.json

@@ -0,0 +1,39 @@
+{
+  "[github-actions-workflow]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[javascript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[javascriptreact]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[json5]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[jsonc]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[typescript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "[typescriptreact]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "editor.formatOnSave": true
+  },
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.formatOnSave": true,
+  "editor.formatOnSaveMode": "file",
+  "editor.tabSize": 2,
+  "editor.codeActionsOnSave": {
+    "source.fixAll.eslint": "explicit"
+  },
+  "files.trimTrailingWhitespace": true,
+  "prettier.useEditorConfig": false
+}

package/CHANGELOG.md

@@ -24,7 +24,6 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
   The boolean configuration options `captureEmail` and `captureUsername` can be used to change
   this behaviour.
 
-
 ## v2.3.6
 
 - Updates for React Native
@@ -46,26 +45,26 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
 
 ## v2.3.2
 
-- Be defensive about navigator not existing on window 
-[#514](https://github.com/rollbar/rollbar.js/pull/514)
+- Be defensive about navigator not existing on window
+  [#514](https://github.com/rollbar/rollbar.js/pull/514)
 
-- Proper scrubbing 
-[#510](https://github.com/rollbar/rollbar.js/pull/510)
+- Proper scrubbing
+  [#510](https://github.com/rollbar/rollbar.js/pull/510)
 
-- Be defensive when accessing event properties 
-[#503](https://github.com/rollbar/rollbar.js/pull/503)
+- Be defensive when accessing event properties
+  [#503](https://github.com/rollbar/rollbar.js/pull/503)
 
 - Add environment to options type enhancement
-[#499](https://github.com/rollbar/rollbar.js/pull/499)
+  [#499](https://github.com/rollbar/rollbar.js/pull/499)
 
-- call the transform in the options on the server side too 
-[#498](https://github.com/rollbar/rollbar.js/pull/498)
+- call the transform in the options on the server side too
+  [#498](https://github.com/rollbar/rollbar.js/pull/498)
 
-- fixes errMsg.match is not a function error 
-[#492](https://github.com/rollbar/rollbar.js/pull/492)
+- fixes errMsg.match is not a function error
+  [#492](https://github.com/rollbar/rollbar.js/pull/492)
 
-- Call callback even if logging same as last error 
-[#490](https://github.com/rollbar/rollbar.js/pull/490)
+- Call callback even if logging same as last error
+  [#490](https://github.com/rollbar/rollbar.js/pull/490)
 
 ## v2.3.1
 
@@ -105,16 +104,20 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
 - Fix some typescript declaration issues
 
 ## v2.2.7
+
 - Ensure scrubbing has the right value for telemetry inputs if `scrubTelemetryInputs` is true.
 - Fix memory leak related to network requests
 
 ## v2.2.6
+
 - Fix blacklist issue
 
 ## v2.2.5
+
 - Revert `isNative` check for replacing JSON functions
 
 ## v2.2.4
+
 - Fix a "race" condition in how wait works
 - Add two options to control scrubbing of telemetry input events: `scrubTelemetryInputs` and
   `telemetryScrubber`.
@@ -126,9 +129,11 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
   rather than an integer.
 
 ## v2.2.3
+
 - Actually make collecting network telemetry on by default
 
 ## v2.2.2
+
 - Fix bug in network telemetry that was wrapping onreadystatechage in correctly
 - Enable collecting network telemetry by default
 - Fix stack overflow issue due to scrubbing of self-referential objects
@@ -137,43 +142,48 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
 - Make the typescript type declarations better
 
 ## v2.2.1
+
 - Disable collection of network telemetry data by default
 
 ## v2.2.0
+
 - Telemetry data is now collected by default in the browser. See
   [README.md](https://github.com/rollbar/rollbar.js#telemetry)
   for information and configuration details.
 - Fix some typos in the typescript declarations
 
 ## v2.1.3
+
 - Actually change version number in package.json and build all relevant files
 
 ## v2.1.2
+
 - Actually send the client-side UUID with the item
 - Fix typos in typescript declaration
 
 ## v2.1.1
 
-- Allow more flexibility of request objects 
-[#346](https://github.com/rollbar/rollbar.js/pull/346)
-- Address issue with handleItemWithError possible not calling the callback 
-[#345](https://github.com/rollbar/rollbar.js/pull/345)
-- Fixed a typo in README.md 
-[#343](https://github.com/rollbar/rollbar.js/pull/343)
-- unify configuration options 
-[#341](https://github.com/rollbar/rollbar.js/pull/341)
-- add some intervals to keep the event loop running when wait is called 
-[#338](https://github.com/rollbar/rollbar.js/pull/338)
-- [Fixes #336] Propagate message 
-[#337](https://github.com/rollbar/rollbar.js/pull/337)
-- [Fixes #269] get rid of rollbar wrapper stack frames 
-[#335](https://github.com/rollbar/rollbar.js/pull/335)
-- first pass at a typescript declaration file 
-[#334](https://github.com/rollbar/rollbar.js/pull/334)
-- Fix #331 Verbose logging was using the wrong key structure 
-[#332](https://github.com/rollbar/rollbar.js/pull/332)
+- Allow more flexibility of request objects
+  [#346](https://github.com/rollbar/rollbar.js/pull/346)
+- Address issue with handleItemWithError possible not calling the callback
+  [#345](https://github.com/rollbar/rollbar.js/pull/345)
+- Fixed a typo in README.md
+  [#343](https://github.com/rollbar/rollbar.js/pull/343)
+- unify configuration options
+  [#341](https://github.com/rollbar/rollbar.js/pull/341)
+- add some intervals to keep the event loop running when wait is called
+  [#338](https://github.com/rollbar/rollbar.js/pull/338)
+- [Fixes #336] Propagate message
+  [#337](https://github.com/rollbar/rollbar.js/pull/337)
+- [Fixes #269] get rid of rollbar wrapper stack frames
+  [#335](https://github.com/rollbar/rollbar.js/pull/335)
+- first pass at a typescript declaration file
+  [#334](https://github.com/rollbar/rollbar.js/pull/334)
+- Fix #331 Verbose logging was using the wrong key structure
+  [#332](https://github.com/rollbar/rollbar.js/pull/332)
 
 ## v2.1.0
+
 - Use the upstream version of console-polyfill: [#306](https://github.com/rollbar/rollbar.js/pull/306)
 - The verbose option still existed but didn't do anything, this adds back the functionality to
   output some information to the console: [#311](https://github.com/rollbar/rollbar.js/pull/311)
@@ -197,10 +207,12 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
   match what we allow in the browser: [#329](https://github.com/rollbar/rollbar.js/pull/329)
 
 ## v2.0.4
+
 - Fixes a small typo in the unhandled rejection handler. (pr#303)
-- Updated the no-conflict example to use the new noconflict bundle and updated docs. 
+- Updated the no-conflict example to use the new noconflict bundle and updated docs.
 
 ## v2.0.3
+
 - Decreased NPM package size by removing examples and coverage directories. (pr#276)
 - Handle unordered params on the server side. (pr#286)
 - Fixes a server-side bug which failed to report an error if the request did not contain a `headers` key. (pr#288)
@@ -210,13 +222,16 @@ The change log has moved to this repo's [GitHub Releases Page](https://github.co
 - Added a no-conflict bundle for the browser. (pr#295)
 
 ## v2.0.2
+
 - Added/updated docs on how to use rollbar.js with React, Angular 1 & 2, Ember, Backbone.js
 - Fixed a bug which was not respecting the `enabled` flag. (pr#280)
 
 ## v2.0.1
+
 - Maintenance release - No new features or bugfixes
 
 ## v2.0.0
+
 This release is the first where the library supports both browser and node environments from the
 same package. We decided to deprecate the `rollbar-browser` npm package in favor of using the
 `rollbar` npm package for all javascript environments. This single codebase now handles both
@@ -247,6 +262,7 @@ be noticable. However, there are a few cases where one might run in to trouble.
   the convenience instance located at `window.Rollbar` will be setup as this handler.
 
 **v1.9.4**
+
 - Updated to the newest version of console-polyfill (pr#244)
 - Log functions now return an object with the uuid as one of the keys (pr#236)
 - Fix issue related to Object.assign which caused problems on IE8 (pr#246)
@@ -255,16 +271,20 @@ be noticable. However, there are a few cases where one might run in to trouble.
   `[object Object]` (pr#249)
 
 **v1.9.3**
+
 - Serve rollbar.js from CDNJS
 
 **v1.9.2**
+
 - Fix bug which would break `Rollbar.wrap()` if a string was thrown. (pr#222)
 
 **v1.9.1**
+
 - Re-add rollbar.snippet.js to the Bower distribution. (pr#196)
   - This re-adds `dist/rollbar.snippet.js` to be backwards compatible with v1.8.5
 
 **v1.9.0**
+
 - Added support for arrays as custom data. (pr#194)
 - Documentation added for disabling Rollbar in the presence of ad blockers. (pr#190)
 - Added support for unhandled rejections from Promises. (pr#192)
@@ -278,21 +298,27 @@ be noticable. However, there are a few cases where one might run in to trouble.
       - `src`
 
 **v1.8.5**
+
 - Support retrying after being in offline mode. (pr#186)
 
 **v1.8.4**
+
 - Check messages body for ignored messages. (pr#180)
 
 **v1.8.3**
+
 - Fix a bug introduced in 1.8.0 where payload options were being removed by calls to `configure()`. (pr#176)
 
 **v1.8.2**
+
 - Using the latest error-stack-parser from NPM. (pr#171)
 
 **v1.8.1**
+
 - Changed the error-stack-parser dependency to use git+https. (pr#168)
 
 **v1.8.0**
+
 - Fixed a few bugs in IE9, IE8 which were not recognozing `Error` instances properly. (pr#164)
 - Changed the behavior of `.global()` to only store options that we know how to process. (pr#164)
 - Refactored the code to remove custom polyfills. (pr#164)
@@ -300,24 +326,30 @@ be noticable. However, there are a few cases where one might run in to trouble.
 - Fixed a bug in the jQuery plugin which would cause an error to be thrown in the internal `checkIgnore()` function. (pr#161)
 
 **v1.7.5**
+
 - Fix bug when checking window.onerror.belongsToShim.
 
 **v1.7.4**
+
 - Don't save shim's onerror when we are building globalnotifier.
   This fixes tests using window.onerror on a browser console
 - Fix Default endpoint on docs/configuration.md
 
 **v1.7.3**
+
 - Added a named AMD module to the list of build targets. (pr#151)
 
 **v1.7.2**
+
 - Bumped version so that NPM lists 1.7.2 as the latest, (was pointing to 1.6.0) (issue#148)
 
 **v1.7.1**
+
 - Integrated karma tests. (pr#130)
 - Added warning message for common issue with `loadFull()`
 
 **v1.7.0**
+
 - Fixed a bug that was not recognizing custom Error subclasses as valid errors. (pr#142)
 - Added documentation for the `hostWhiteList` option. (pr#138)
 - Changed the default uncaught error level to "error" instead of "warning".
@@ -326,14 +358,17 @@ be noticable. However, there are a few cases where one might run in to trouble.
   asynchronously. This option is set to `true` by default.
 
 **v1.6.1**
+
 - Updated bower.json to contain only a single .js entry. (issue#126)
 
 **v1.6.0**
+
 - Fixed a bug that caused IE 8 to not properly initialize `window.Rollbar`. (pr#129)
   - Fixed the `XDomainRequest` code to work properly in IE 8.
   - Updated error parsing to provide more useful information for IE 8 errors
 
 **v1.5.0**
+
 - Published rollbar.js to npmjs.com as rollbar-browser. (pr#127)
 - Fixes a bug where thrown non-error objects were not properly handled. (pr#125)
 - Fixes a bug that was logging an incorrect message when the notifier was disabled. (pr#124)
@@ -341,158 +376,204 @@ be noticable. However, there are a few cases where one might run in to trouble.
 - Lots of code cleanup and smaller minified file size.
 
 **v1.4.4**
+
 - Remove the `window.onload` event handler from the snippet and just create the script tag for the full rollbar.js source directly. (pr#120)
 
 **v1.4.3**
+
 - Fixed a bug that would cause the notifier to crash when run in a Selenium test. (pr#117)
 - Force the notifier to always use HTTPS when communicating with api.rollbar.com. (pr#116)
 
 **v1.4.2**
+
 - Fixed a bug that occurred in FF when Rollbar logged an internal error or if verbose mode was turned on. (pr#105)
 
 **v1.4.1**
+
 - Fixed a bug that would load the wrong AMD module if a custom "rollbar" module was already defined.
   - Customers should copy and paste the new snippet into their code.
 
 **v1.4.0**
+
 - Fix a bug, (introduced in v1.3) that caused Rollbar to generate an error when used with RequireJS.
   - Customers should copy and paste the new snippet into their code.
 
 **v1.3.0**
+
 - Add more strict JSHint options and fix errors/warnings.
 
 **v1.3.0-rc.4**
+
 - Fixes IE8 bug where JSON was not defined.
 
 **v1.3.0-rc.3**
+
 - Remove polyfill.
 
 **v1.3.0-rc.2**
+
 - Fix main values in bower.json.
 
 **v1.3.0-rc.1**
+
 - Fixes for IE8+
 
 **v1.3.0-alpha.5**
+
 - Fix rollbar.umd.min.js URL in the snippet
 - Remove sourceMappingURL comment due to browser bug concerns
 
 **v1.3.0-alpha.4**
+
 - Update CHANGELOG.md.
 
 **v1.3.0-alpha.3**
+
 - Remove repeated timer to send enqueued payloads.
 - Change argument name in Stack() to fix uglify bug.
-- Change __DEFAULT_ROLLBARJS_URL__ to use https.
-- Set window._globalRollbarOptions when calling .configure().
+- Change **DEFAULT_ROLLBARJS_URL** to use https.
+- Set window.\_globalRollbarOptions when calling .configure().
 
 **v1.3.0-alpha.2**
+
 - Update missing dist/ files.
 
 **v1.3.0-alpha.1**
+
 - Build the library using webpack.
 - Replace tracekit and use error-stack-parser.
 
 **1.2.2**
+
 - Added `nojson` distribution, for use on sites with a Content Security Policy that disallows `unsafe-eval`. (The standard distributions ship with a built-in JSON implementation, since external libraries, such as MooTools, sometimes break the brower's built-in JSON.) If you know that the built-in JSON is not being modified in your application, or you are disallowing `unsafe-eval`, use this distribution.
 
 **1.2.1**
+
 - Fixed bug where the global notifier not being used to atch event listener exceptions. (pr#70)
 
 **1.2.0**
+
 - Fixed AMD build to now return the notifier instance from the `init()` method.
 
 **1.1.16 - EDIT: This version has been removed due to finding a bug that broke backward compatibility.**
+
 - Optimized the AMD build to not create a Notifier instance until the `init()` method is called.
 
 **1.1.15**
+
 - Fix a bug where custom context functions that returned `undefined` were causing `Rollbar.wrap()` to throw an error.
 
 **1.1.14**
+
 - Fix a bug in IE8 where DOMException was being used even though it's not defined, (#62).
 
 **1.1.13**
+
 - Add `responseText` and `statusText` to the data reported by the jQuery ajax plugin, (pr#61).
 
 **1.1.12**
+
 - Fixes a bug where `DOMException` objects were not recognized as error objects. (#55).
 
 **1.1.11**
+
 - Fixes a bug where wrapped functions were crashing when a `null` callback was given to `removeEventListener()`, (pr#50).
 
 **1.1.10**
+
 - Pulls in the latest JSON-js changes that do not call `.toJSON()` if the method exists already. This was breaking because MooTools v1.2.4 sets `.toJSON()` to use a broken JSON stringify implementation.
 
 **1.1.9**
+
 - Always use the custom JSON implementation since some users are initializing a library that will overwrite a working `JSON.stringify()` with a broken one after Rollbar has checked for `JSON.stringify()` correctness.
 
 **1.1.8**
+
 - Added a callback function to `loadFull()` to support Segment.io's plugin model.
 
 **1.1.7**
+
 - Added `verbose` and `logFunction` options, (pr#42).
 
 **1.1.6**
+
 - Added a `_wrappedSource` key to exceptions caught by the `wrap()` method to record the source of the wrapped function.
 
 **1.1.5**
+
 - Added a `context` parameter to `Rollbar.wrap()`, (#26).
 - Added a `transform` option to allow the user to read/modify the payload before we send it to Rollbar, (#41 #43).
 
 **1.1.4**
+
 - Added the `enabled` flag to determine when we should enqueue payloads to be sent, (#28).
 
 **1.1.3**
+
 - Fixed a bug that was causing a stack overflow error in IE8, (#38).
 - Shaved off a few bytes from the snippet's size.
 
 **1.1.2**
+
 - Fixed a bug that was causing `Rollbar.configure()` to incorrectly handle overwriting array configuration.
 - Added in support for a `ignoredMessages` configuration option, (pr#35).
 - Fixed a bug that was causing some `EventListener` objects to not be unbound, (pr#33).
 - Updated the snippet with fixes.
 
 **1.1.1**
+
 - Fixed a bug with default rate limits. The defaults were not applied unless Rollbar.global() was called.
 
 **1.1.0**
+
 - Add support for AMD JS loaders and refactor rollbar.require.js into rollbar.amd.js and rollbar.commonjs.js.
 
 **1.0.0-rc.11**
+
 - Add support for whitelisting host names/domains, (pr#31).
 
 **1.0.0-rc.10**
+
 - Add support for using rollbar with Webpack/Browserify via `require("rollbar.require.min.js")` with examples.
 
 **1.0.0-rc.9**
+
 - Fixed a bug that caused a wrapped async handler to break if there was no callback provided.
 
 **1.0.0-rc.8**
+
 - Fixed a bug that created/used a global variable.
 
 **1.0.0-rc.7**
+
 - Change default reportLevel to `debug`. Previously, calls to `Rollbar.info` and `Rollbar.debug` were filtered out under the default configuration; now they are let through.
 
 **1.0.0-rc.6**
+
 - Fixed a bug where items were sent in reverse order when queued
 - Add `maxItems` global option. If defined, at most this many items per pageview will be sent. Default `0`, meaning "no limit".
 
 **1.0.0-rc.5**
+
 - Fix invalid payload generated when a non-Error object is passed as the error (#20)
 - Pass correct window.onerror args to original onerror (#23)
 - jQuery plugin: ignore status 0 events (#22)
 - Fix issue where callbacks to `.debug()`, etc. were not called if reportLevel filters the item (#24)
 
 **1.0.0-rc.4**
+
 - Fix snippet in IE8 (change `map` to a for loop)
 
 **1.0.0-rc.3**
+
 - Remove source maps from build process (no user-facing changes)
 
 **1.0.0-rc.2**
+
 - Send access token as a request header in browsers that support XMLHttpRequest
 
 **1.0.0-rc.1**
+
 - Fix bug where we were attempting to wrap an object instead of a function.
   - https://github.com/rollbar/rollbar.js/pull/17
 - Fix bug in jQuery plugin that wasn't passing along the jQuery object.
@@ -501,17 +582,21 @@ be noticable. However, there are a few cases where one might run in to trouble.
   - https://github.com/rollbar/rollbar.js/blob/master/docs/migration_v0_to_v1.md
 
 **1.0.0-beta9**
+
 - Fix api response JSON parsing on older browsers
 
 **1.0.0-beta8**
+
 - Fix uncaught errors being ignored in some browsers
 - Default uncaught error level now `warning` instead of `error`, also configurable
 - Wrap `addEventListener` to get more rich stack trace info for uncaught exceptions
 
 **1.0.0-beta7**
+
 - Use a custom JSON.stringify method if the existing one does not work properly.
 
 **1.0.0-beta4**
+
 - Fix some documentation bugs
 - Changes made to the snippet to put `environment` in the `payload` key.
 - Remove the default `context` value and associated logic around it being either a string or a function.
@@ -519,4 +604,5 @@ be noticable. However, there are a few cases where one might run in to trouble.
 ## Upgrade Instructions
 
 ### v1.0.x to v1.1.x
+
 1. Replace your rollbar snippet with the latest from the [rollbar.js quickstart docs](https://rollbar.com/docs/notifier/rollbar.js/) or from [the GitHub repo](https://github.com/rollbar/rollbar.js/blob/master/dist/rollbar.snippet.js).