Merge branch 'master' into SNOW-502598-async-query-exec

snowflakedb · Nov 2, 2023 · 6944e99 · 6944e99
2 parents 46748bb + 6cc2fda
commit 6944e99
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 17 deletions.
diff --git a/lib/services/sf.js b/lib/services/sf.js
@@ -1485,11 +1485,14 @@ StateRenewing.prototype.enter = function (context)
     {
       method: 'POST',
       url: '/session/token-request',
-      json:
-        {
-          'REQUEST_TYPE': 'RENEW',
-          'oldSessionToken': this.tokenInfo.getSessionToken()
-        },
+      headers: {
+        CLIENT_APP_ID: this.connectionConfig.getClientType(),
+        CLIENT_APP_VERSION: this.connectionConfig.getClientVersion(),
+      },
+      json: {
+        'REQUEST_TYPE': 'RENEW',
+        'oldSessionToken': this.tokenInfo.getSessionToken(),
+      },
       scope: this,
       callback: function (err, body)
       {

diff --git a/lib/util.js b/lib/util.js
@@ -473,7 +473,7 @@ exports.getNextSleepTime = getNextSleepTime;
  * @returns {Boolean} true if it is loginRequest, otherwise false.
  */
 exports.isLoginRequest = function (loginUrl) {
-  const endPoints = ['/v1/login-request', '/authenticator-request', '/token-request',];
+  const endPoints = ['/v1/login-request', '/authenticator-request'];
   return endPoints.some((endPoint) => loginUrl.includes(endPoint));
 }
 

diff --git a/test/unit/util_test.js b/test/unit/util_test.js
@@ -545,17 +545,7 @@ describe('Util', function ()
         testName: 'test URL with a wrong authenticator-request point',
         endPoint: '/authenticator-requ',
         result:false,
-      },
-      {
-        testName: 'test URL with a right token point',
-        endPoint: '/token-request',
-        result:true,
-      },
-      {
-        testName: 'test URL with a wrong token point',
-        endPoint: '/tokenRequest',
-        result:false,
-      },
+      }
     ];
 
     for (const {testName, endPoint,result} of testCases)