acquireToken does not make callback when token is expired

[mikerains]

in adal.js:

• acquireToken.js does not have a call to this._saveItem(this.CONSTANTS.STORAGE.STATE_RENEW, expectedState, true);
• acquireTokenRedirect does have this._saveItem(this.CONSTANTS.STORAGE.STATE_RENEW, expectedState, true);
• Because it is not in storage, _matchState() doesn't find it and hence doesn't set reqest.REQUEST_TYPE = "RENEW_TOKEN" and doesn't set request.stateMatch to true.
• Because

Because request.stateMatch is not true, adal.service's handleWindowCallback() does not invoke this.context._callBackMappedToRenewStates[requestInfo.stateResponse] here:

            if (requestInfo.stateMatch) {
                if (typeof callback === 'function') {
                    if (requestInfo.requestType === this.context.REQUEST_TYPE.RENEW_TOKEN) {
                        // Idtoken or Accestoken can be renewed
                        if (requestInfo.parameters['access_token']) {
                            callback(this.context._getItem(this.context.CONSTANTS.STORAGE.ERROR_DESCRIPTION)
                                , requestInfo.parameters['access_token']);

Also, because adal's acquireToken doesn't make the callback, adal.service's refreshLoginToken

[geerzo]

Are you using 4.x? That version should have fixed this issue. You are correct match states fails but in the adal.js method getRequestInfo it has this fallback code which gets called.

                // external api requests may have many renewtoken requests for different resource
                if (!requestInfo.stateMatch && window.parent) {
                    requestInfo.requestType = this._requestType;
                    var statesInParentContext = this._renewStates;
                    for (var i = 0; i < statesInParentContext.length; i++) {
                        if (statesInParentContext[i] === requestInfo.stateResponse) {
                            requestInfo.stateMatch = true;
                            break;
                        }
                    }
                }