Merge pull request #68 from Cox-Automotive/bugfix-dynamic-roles

fixes a bug with the getDynamicValues query param
Cox-Automotive · Apr 15, 2021 · d5a9364 · d5a9364
2 parents bcccb70 + 56c1c5a
commit d5a9364
Show file tree

Hide file tree

Showing 6 changed files with 87 additions and 45 deletions.
diff --git a/dist/alks.cjs.js b/dist/alks.cjs.js
@@ -1,6 +1,6 @@
 'use strict';
 
-var version = "1.11.2";
+var version = "1.11.3";
 
 const Buffer = require('buffer').Buffer;
 const fetch = require('node-fetch');
@@ -224,7 +224,7 @@ class alks {
     * })
     */
   getAllAWSRoleTypes(props) {
-    const url = props.getDynamicValues ? 'allAwsRoleTypes' : 'allAwsRoleTypes?getDynamicValues=true';
+    const url = props.getDynamicValues ? 'allAwsRoleTypes?getDynamicValues=true' : 'allAwsRoleTypes';
     return(this._doFetch(url, props, 'GET').then(results => results.roleTypes))
   }
 
@@ -835,7 +835,7 @@ class alks {
       delete opts.sessionToken;
     }
 
-    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}/`, {
+    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}`, {
       method, headers, body: method == 'GET' ? undefined : JSON.stringify(opts)
     });
 

diff --git a/dist/alks.esm.js b/dist/alks.esm.js
@@ -1,4 +1,4 @@
-var version = "1.11.2";
+var version = "1.11.3";
 
 const fetch = window.fetch.bind(window);
 
@@ -221,7 +221,7 @@ class alks {
     * })
     */
   getAllAWSRoleTypes(props) {
-    const url = props.getDynamicValues ? 'allAwsRoleTypes' : 'allAwsRoleTypes?getDynamicValues=true';
+    const url = props.getDynamicValues ? 'allAwsRoleTypes?getDynamicValues=true' : 'allAwsRoleTypes';
     return(this._doFetch(url, props, 'GET').then(results => results.roleTypes))
   }
 
@@ -832,7 +832,7 @@ class alks {
       delete opts.sessionToken;
     }
 
-    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}/`, {
+    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}`, {
       method, headers, body: method == 'GET' ? undefined : JSON.stringify(opts)
     });
 

diff --git a/dist/alks.min.js b/dist/alks.min.js
diff --git a/dist/alks.umd.js b/dist/alks.umd.js
@@ -4,7 +4,7 @@
   (global = global || self, global.alks = factory());
 }(this, function () { 'use strict';
 
-  var version = "1.11.2";
+  var version = "1.11.3";
 
   var fetch = window.fetch.bind(window);
 
@@ -227,7 +227,7 @@
     * })
     */
   alks.prototype.getAllAWSRoleTypes = function getAllAWSRoleTypes (props) {
-    var url = props.getDynamicValues ? 'allAwsRoleTypes' : 'allAwsRoleTypes?getDynamicValues=true';
+    var url = props.getDynamicValues ? 'allAwsRoleTypes?getDynamicValues=true' : 'allAwsRoleTypes';
     return(this._doFetch(url, props, 'GET').then(function (results) { return results.roleTypes; }))
   };
 
@@ -829,7 +829,7 @@
       delete opts.sessionToken;
     }
 
-    var responsePromise = opts._fetch(((opts.baseUrl) + "/" + path + "/"), {
+    var responsePromise = opts._fetch(((opts.baseUrl) + "/" + path), {
       method: method, headers: headers, body: method == 'GET' ? undefined : JSON.stringify(opts)
     });
 

diff --git a/src/alks.js b/src/alks.js
@@ -834,7 +834,7 @@ class alks {
       delete opts.sessionToken
     }
 
-    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}/`, {
+    var responsePromise = opts._fetch(`${opts.baseUrl}/${path}`, {
       method, headers, body: method == 'GET' ? undefined : JSON.stringify(opts)
     })