Constant refresh caused by setTimeout() and addEventListener()

[martin1cerny]

First of all thank you @manfredsteyer for the great library which makes my life a lot easier!

I work on an Angular application and I recently came across the use of zone.js in Angular which is used to trigger view refresh on every event which could cause the UI to change. So, I started to investigate when our views are actually getting refreshed and I realised that angular-oauth2-oidc is causing it to refresh every 3000ms when it asks for the state of the session and than when it receives the response.

I tried to run the initialization code out of Angular like this:

zone.runOutsideAngular(() => {
    this.oauthService.configure(config);
    this.oauthService.tokenValidationHandler = new JwksValidationHandler();
});

zone.runOutsideAngular(() => {
    this.oauthService.loadDiscoveryDocument()
        .then(() => {
            this.zone.runOutsideAngular(() => {
                this.oauthService.tryLogin({
                    onTokenReceived: (info) => {
                        const state = info.state;
...

But setting silent refresh and session check happens in a different scope in asynchronous callbacks where zone.runOutsideAngular() doesn't seem to take effect.

I realised there are already parts of the code which run outside of Angular and I believe that as a library it should only cause a refresh when it emits new event (like session timeout, expired tokens etc.) but not when working internally. Am I right? Or is there a way to run it out of Angular completely?

[manfredsteyer]

This is a good point. Thanks for pointing this out. Do you care for a PR?

[martin1cerny]

I created PR shortly after the issue. And it is merged now I see :-) Thanks a lot for the great work!

[ampled]

Is this far off from being included in a release?

[martin1cerny]

I'm looking forward having this available on NPM as well.

[jeroenheijmans]

PR got merged and a few releases got out since as well. Closing the issue, let us know if it should be reopened.