-
Notifications
You must be signed in to change notification settings - Fork 699
Closed
Labels
bugFor tagging faulty or unexpected behavior.For tagging faulty or unexpected behavior.investigation-neededIndication that the maintainer or involved community members may need to investigate more.Indication that the maintainer or involved community members may need to investigate more.
Description
Describe the bug
If you destroy the OAuthService, for instance through destroying the angular app, the attached event listeners and the sessionCheckTimer will still continue to run.
Stackblitz example
https://angular-abivfc.stackblitz.io/
To Reproduce
Steps to reproduce the behavior:
- Go to the stackblitz. You might need to change of fix the issuer configured in app.component.ts. I have used the one in the example code.
- Log in to the identity server and redirect back to the site
- Open the console
- See error "Cannot read proprety 'contentWindow' of null" or similar error on other browsers.
Expected behavior
All timers, events and elements that the OAuthService created should be stopped or removed when the angular app is destroyed and the framework calls ngOnDestroy.
Desktop (please complete the following information):
- OS: Tested on windows 10, but will probably work the same on all platforms
- Browser Tested in chrome 78 and Firefox 71
Metadata
Metadata
Assignees
Labels
bugFor tagging faulty or unexpected behavior.For tagging faulty or unexpected behavior.investigation-neededIndication that the maintainer or involved community members may need to investigate more.Indication that the maintainer or involved community members may need to investigate more.