Make base PlatformFunction class

src/Types.d.ts

@@ -159,7 +159,7 @@ type StripeObject = {
   _requestSender: RequestSender;
   _getPropsFromConfig: (config: Record<string, unknown>) => UserProvidedConfig;
   _clientId?: string;
-  _platformFunctions: import('./platform/DefaultPlatformFunctions');
+  _platformFunctions: import('./platform/WebPlatformFunctions');
 };
 type RequestSender = {
   _request(

src/Webhooks.ts

@@ -1,6 +1,6 @@
 import utils = require('./utils');
 import _Error = require('./Error');
-import DefaultPlatformFunctions = require('./platform/DefaultPlatformFunctions');
+import WebPlatformFunctions = require('./platform/WebPlatformFunctions');
 import CryptoProvider = require('./crypto/CryptoProvider');
 const {StripeError, StripeSignatureVerificationError} = _Error;
 
@@ -60,7 +60,7 @@ type WebhookObject = {
   ) => Promise<WebhookEvent>;
   generateTestHeaderString: (opts: WebhookTestHeaderOptions) => string;
   _createCryptoProvider: () => CryptoProvider | null;
-  _platformFunctions: DefaultPlatformFunctions | null;
+  _platformFunctions: WebPlatformFunctions | null;
 };
 
 const Webhook: WebhookObject = {

src/platform/NodePlatformFunctions.ts

@@ -1,12 +1,11 @@
 import crypto = require('crypto');
 import EventEmitter = require('events');
-
-import DefaultPlatformFunctions = require('./DefaultPlatformFunctions');
+import PlatformFunctions = require('./PlatformFunctions');
 
 /**
- * Specializes DefaultPlatformFunctions using APIs available in Node.js.
+ * Specializes WebPlatformFunctions using APIs available in Node.js.
  */
-class NodePlatformFunctions extends DefaultPlatformFunctions {
+class NodePlatformFunctions extends PlatformFunctions {
   /** For mocking in tests */
   _exec: any;
   _UNAME_CACHE: Promise<string | null> | null;

src/platform/DefaultPlatformFunctions.ts → src/platform/PlatformFunctions.ts

@@ -5,12 +5,12 @@ import StripeEmitter = require('../StripeEmitter');
  * Interface encapsulating various utility functions whose
  * implementations depend on the platform / JS runtime.
  */
-class DefaultPlatformFunctions {
+class PlatformFunctions {
   /**
    * Gets uname with Node's built-in `exec` function, if available.
    */
   getUname(): Promise<string | null> {
-    return Promise.resolve(null);
+    throw new Error('getUname not implemented.');
   }
 
   /**
@@ -32,19 +32,17 @@ class DefaultPlatformFunctions {
     if (a.length !== b.length) {
       return false;
     }
-
     const len = a.length;
     let result = 0;
-
     for (let i = 0; i < len; ++i) {
       result |= a.charCodeAt(i) ^ b.charCodeAt(i);
     }
     return result === 0;
   }
 
   createEmitter(): StripeEmitter | EventEmitter {
-    throw new Error('createEmitter not implemented');
+    throw new Error('createEmitter not implemented.');
   }
 }
 
-export = DefaultPlatformFunctions;
+export = PlatformFunctions;

src/platform/WebPlatformFunctions.ts

@@ -0,0 +1,45 @@
+import EventEmitter = require('events');
+import StripeEmitter = require('../StripeEmitter');
+import PlatformFunctions = require('./PlatformFunctions');
+
+/**
+ * Specializes WebPlatformFunctions using APIs available in Web workers.
+ */
+class WebPlatformFunctions extends PlatformFunctions {
+  /** @override */
+  getUname(): Promise<string | null> {
+    return Promise.resolve(null);
+  }
+
+  /** @override */
+  uuid4(): string {
+    return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, (c) => {
+      const r = (Math.random() * 16) | 0;
+      const v = c === 'x' ? r : (r & 0x3) | 0x8;
+      return v.toString(16);
+    });
+  }
+
+  /** @override */
+  secureCompare(a: string, b: string): boolean {
+    // return early here if buffer lengths are not equal
+    if (a.length !== b.length) {
+      return false;
+    }
+
+    const len = a.length;
+    let result = 0;
+
+    for (let i = 0; i < len; ++i) {
+      result |= a.charCodeAt(i) ^ b.charCodeAt(i);
+    }
+    return result === 0;
+  }
+
+  /** @override */
+  createEmitter(): StripeEmitter | EventEmitter {
+    return new StripeEmitter(new EventTarget());
+  }
+}
+
+export = WebPlatformFunctions;

src/stripe.common.ts

@@ -55,8 +55,8 @@ Stripe.HttpClientResponse = HttpClient.HttpClientResponse;
 import CryptoProvider = require('./crypto/CryptoProvider');
 Stripe.CryptoProvider = CryptoProvider;
 
-import DefaultPlatformFunctions = require('./platform/DefaultPlatformFunctions');
-Stripe._platformFunctions = new DefaultPlatformFunctions();
+import PlatformFunctions = require('./platform/PlatformFunctions');
+Stripe._platformFunctions = new PlatformFunctions();
 
 function Stripe(
   this: StripeObject,
@@ -202,7 +202,6 @@ Stripe.prototype = {
   on: null!,
   off: null!,
   once: null!,
-  emit: null!,
   VERSION: null!,
   StripeResource: null!,
   webhooks: null!,

src/stripe.worker.ts

@@ -1,16 +1,12 @@
 import Stripe = require('./stripe.common');
-import StripeEmitter = require('./StripeEmitter');
+import WebPlatformFunctions = require('./platform/WebPlatformFunctions');
+
+Stripe._platformFunctions = new WebPlatformFunctions();
+Stripe.webhooks._platformFunctions = Stripe._platformFunctions;
 
 Stripe.createHttpClient = Stripe.createFetchHttpClient;
 Stripe.webhooks._createCryptoProvider = Stripe.createSubtleCryptoProvider;
 
-// StripeEmitter uses the Event Web API. `Event` is not available
-// in the global scope until Node 15, so we set `createEmitter` here
-// to avoid reference errors.
-Stripe._platformFunctions.createEmitter = (): StripeEmitter => {
-  return new StripeEmitter(new EventTarget());
-};
-
 module.exports = Stripe;
 
 // expose constructor as a named property to enable mocking with Sinon.JS