Instantiates reporter only on demand

recurly · Jan 4, 2024 · 4f3b38a · 4f3b38a
1 parent 774e5c5
commit 4f3b38a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/lib/recurly.js b/lib/recurly.js
@@ -68,6 +68,7 @@ const DEFAULTS = {
     litle: { sessionId: undefined },
     braintree: { deviceData: undefined }
   },
+  report: false,
   risk: {
     threeDSecure: { preflightDeviceDataCollector: true }
   },
@@ -142,11 +143,14 @@ export class Recurly extends Emitter {
     this.readyState = 0;
     this.request = new Request({ recurly: this });
     this.config = deepAssign({}, DEFAULTS);
+
     if (options) this.configure(options);
+
     this.bankAccount = {
       token: bankAccount.token.bind(this),
       bankInfo: bankAccount.bankInfo.bind(this)
     };
+
     this.reporter = new Reporter({ recurly: this });
 
     this.Pricing = () => new SubscriptionPricing(this); // deprecated
@@ -222,6 +226,8 @@ export class Recurly extends Emitter {
    * @param {Boolean} [options.cors] Enables data transmission over XHR+CORS
    * @param {Array} [options.required] Adds additional field requirements for
    *                                   tokenization. ex: ['cvv']
+   * @param {Boolean} [options.report] Enables remote event reports
+   *
    * @public
    */
   configure (options) {
@@ -262,6 +268,10 @@ export class Recurly extends Emitter {
       deepAssign(this.config.risk, options.risk);
     }
 
+    if ('report' in options) {
+      this.config.report = options.report;
+    }
+
     if ('parent' in options) {
       this.config.parent = options.parent;
     }

diff --git a/lib/recurly/reporter.js b/lib/recurly/reporter.js
@@ -55,7 +55,7 @@ export class Reporter {
    * @private
    */
   get shouldDispatch () {
-    return this.recurly.isParent;
+    return this.recurly.isParent && this.recurly.config.report;
   }
 
   /**

diff --git a/test/unit/support/helpers.js b/test/unit/support/helpers.js
@@ -20,10 +20,12 @@ export function initRecurly (recurly, opts) {
     // prevents itinerant event logging workers from dispatching
     sinon.stub(recurly.reporter, 'send');
   }
+
   recurly.configure(merge({
     publicKey: 'test',
     api: `${window.location.protocol}//${window.location.host}/api`
   }, opts));
+
   return recurly;
 }