Fix response handling

snowflakedb · Nov 8, 2023 · faedff3 · faedff3
1 parent 645faeb
commit faedff3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 10 deletions.
diff --git a/lib/authentication/auth_okta.js b/lib/authentication/auth_okta.js
@@ -56,7 +56,7 @@ function auth_okta(password, region, account, clientType, clientVersion, httpCli
     var tokenUrl;
     await step1(authenticator, serviceName, account, username).then((response) =>
     {
-      const responseData = JSON.parse(response['data']);
+      const responseData = response['data'];
       const success = responseData['success'];
       const errorCode = responseData['code'];
       const errorMessage = responseData['message'];
@@ -78,7 +78,7 @@ function auth_okta(password, region, account, clientType, clientVersion, httpCli
     var oneTimeToken;
     await step3(tokenUrl, username, password).then((response) =>
     {
-        var data = JSON.parse(response['data']);
+        var data = response['data'];
 
         if (data['sessionToken']) {
             oneTimeToken = data['sessionToken'];

diff --git a/test/unit/authentication/authentication_test.js b/test/unit/authentication/authentication_test.js
@@ -398,20 +398,20 @@ describe('okta authentication', function ()
         {
           json =
           {
-            data: JSON.stringify( {
+            data: {
               success: true,
               data: {
                 ssoUrl: mockssoUrl,
                 tokenUrl: mockTokenUrl
               }
-            })
+            }
           }
         }
         if (url === mockTokenUrl)
         {
           json =
           {
-            data: JSON.stringify(mockCookieToken)
+            data: mockCookieToken
           }
         }
         return json;
@@ -471,14 +471,14 @@ describe('okta authentication', function ()
         {
           json =
           {
-            data: JSON.stringify({
+            data: {
               success: true,
               data:
               {
                 ssoUrl: mockssoUrl,
                 tokenUrl: 'abcd'
               }
-            })
+            }
           }
         }
         return json;
@@ -514,21 +514,21 @@ describe('okta authentication', function ()
         {
           json =
           {
-            data: JSON.stringify({
+            data: {
               success: true,
               data:
               {
                 ssoUrl: mockssoUrl,
                 tokenUrl: mockTokenUrl
               }
-            })
+            }
           }
         }
         if (url === mockTokenUrl)
         {
           json =
           {
-            data: JSON.stringify(mockCookieToken)
+            data: mockCookieToken
           }
         }
         return json;