Fixed inconsistencies in client.token (#45)

Resolves: OKTA-102903
okta · Sep 30, 2016 · e3f8679 · e3f8679
1 parent b21b9b7
commit e3f8679
Showing 1 changed file with 9 additions and 14 deletions.
diff --git a/lib/token.js b/lib/token.js
@@ -127,11 +127,11 @@ function verifyIdToken(sdk, idToken, options) {
     });
 }
 
-function refreshIdToken(sdk, opts) {
-  opts = opts || {};
-  opts.display = null;
-  opts.prompt = 'none';
-  return getToken(sdk, opts);
+function refreshIdToken(sdk, options) {
+  options = options || {};
+  options.display = null;
+  options.prompt = 'none';
+  return getToken(sdk, options);
 }
 
 function loadFrame(src) {
@@ -394,7 +394,7 @@ function buildAuthorizeUrl(sdk, oauthParams) {
  * 
  * Two main flows:
  *
- *  1) Exchange a sessionToken for an idToken
+ *  1) Exchange a sessionToken for a token
  * 
  *    Required:
  *      clientId: passed via the OktaAuth constructor or into getToken
@@ -409,7 +409,7 @@ function buildAuthorizeUrl(sdk, oauthParams) {
  *      responseMode: 'okta_post_message'
  *      display: undefined
  *
- *  2) Get an idToken from an idp
+ *  2) Get a token from an idp
  *
  *    Required:
  *      clientId: passed via the OktaAuth constructor or into getToken
@@ -440,13 +440,8 @@ function buildAuthorizeUrl(sdk, oauthParams) {
  *                                      Defaults to 'External Identity Provider User Authentication'
  */
 function getToken(sdk, oauthOptions, options) {
-  if (!oauthOptions) {
-    oauthOptions = {};
-  }
-
-  if (!options) {
-    options = {};
-  }
+  oauthOptions = oauthOptions || {};
+  options = options || {};
 
   // Default OAuth query params
   var oauthParams = getDefaultOAuthParams(sdk, oauthOptions);