Release braintree-web 3.112.1 source

CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 3.112.1
+
+- Venmo
+  - Fix a bug where, after redirecting in the mobile web flow, we were passing in a null context ID to a graphql API call. The fix is to set the Venmo instance's venmoPaymentContextId based on resource_id in url.
+
 ## 3.112.0
 
 - Local Payment

package.json

@@ -1,6 +1,6 @@
 {
   "name": "braintree-web",
-  "version": "3.112.0",
+  "version": "3.112.1",
   "license": "MIT",
   "main": "src/index.js",
   "private": true,

src/lib/url-params.js

@@ -0,0 +1,11 @@
+"use strict";
+
+var querystring = require("./querystring");
+
+function getUrlParams() {
+  return querystring.parse(window.location.href);
+}
+
+module.exports = {
+  getUrlParams: getUrlParams,
+};

src/venmo/venmo.js

@@ -16,6 +16,7 @@ var ExtendedPromise = require("@braintree/extended-promise");
 var getVenmoUrl = require("./shared/get-venmo-url");
 var desktopWebLogin = require("./shared/web-login-backdrop");
 var snakeCaseToCamelCase = require("../lib/snake-case-to-camel-case");
+var urlParams = require("../lib/url-params");
 
 // NEXT_MAJOR_VERSION the source code for this is actually in a
 // typescript repo called venmo-desktop, once the SDK is migrated
@@ -503,6 +504,11 @@ Venmo.prototype.hasTokenizationResult = function () {
 // when listening on a hashchange event
 Venmo.prototype._hasTokenizationResult = function (hash) {
   var params = getFragmentParameters(hash);
+  var paramsFromUrl = urlParams.getUrlParams();
+
+  if (paramsFromUrl.resource_id) {
+    this._venmoPaymentContextId = paramsFromUrl.resource_id;
+  }
 
   return (
     typeof (params.venmoSuccess || params.venmoError || params.venmoCancel) !==

test/venmo/unit/venmo.js

@@ -5,6 +5,7 @@ jest.mock("../../../src/venmo/shared/supports-venmo");
 jest.mock("../../../src/venmo/external");
 jest.mock("../../../src/lib/in-iframe");
 jest.mock("../../../src/venmo/shared/web-login-backdrop");
+jest.mock("../../../src/lib/url-params");
 
 const analytics = require("../../../src/lib/analytics");
 const { fake } = require("../../helpers");
@@ -23,6 +24,7 @@ const {
   setupDesktopWebLogin,
 } = require("../../../src/venmo/shared/web-login-backdrop");
 const venmoConstants = require("../../../src/venmo/shared/constants");
+const urlParams = require("../../../src/lib/url-params");
 
 function triggerVisibilityHandler(instance, runAllTimers = true) {
   // TODO we should have it trigger the actual
@@ -55,6 +57,7 @@ describe("Venmo", () => {
   beforeAll(() => {
     window.open = jest.fn();
     originalLocationHref = window.location.href;
+    urlParams.getUrlParams.mockReturnValue({});
   });
 
   beforeEach(() => {
@@ -2008,6 +2011,16 @@ describe("Venmo", () => {
     it("returns false when URL has no Venmo payload", () => {
       expect(venmo.hasTokenizationResult()).toBe(false);
     });
+
+    it("sets the _venmoPaymentContextId from the resource_id", () => {
+      urlParams.getUrlParams.mockReturnValue({
+        resource_id: "test-resource-id", // eslint-disable-line camelcase
+      });
+
+      venmo.hasTokenizationResult();
+
+      expect(venmo._venmoPaymentContextId).toBe("test-resource-id");
+    });
   });
 
   describe("tokenize", () => {