Merge pull request #53 from moneytree/AKL-656

[AKL-656] fix for Safari 14 issue
moneytree · Nov 27, 2020 · e211ac4 · e211ac4
2 parents 02fcb53 + 9fb69d0
commit e211ac4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/api/__tests__/authorize.test.ts b/src/api/__tests__/authorize.test.ts
@@ -68,7 +68,7 @@ describe('api', () => {
         configs: generateConfigs(),
       });
       const url = `${MY_ACCOUNT_DOMAINS.production}/oauth/authorize?${query}`;
-      expect(open).toBeCalledWith(url, '_self');
+      expect(open).toBeCalledWith(url, '_self', 'noreferrer');
     });
 
     test('with options', () => {
@@ -101,7 +101,7 @@ describe('api', () => {
         configs: generateConfigs(),
       });
       const url = `${MY_ACCOUNT_DOMAINS.production}/oauth/authorize?${query}`;
-      expect(open).toBeCalledWith(url, '_self');
+      expect(open).toBeCalledWith(url, '_self', 'noreferrer');
     });
 
     test('without window', () => {

diff --git a/src/api/authorize.ts b/src/api/authorize.ts
@@ -70,6 +70,7 @@ export default function authorize(
 
   window.open(
     `${MY_ACCOUNT_DOMAINS[mode]}/oauth/authorize?${queryString}`,
-    getIsTabValue(isNewTab)
+    getIsTabValue(isNewTab),
+    'noreferrer'
   );
 }