The method in question does not depend on "this" in any way--the Observable it accesses is referenced as "Keycloak.tokenExpiredBehaviorSubject", so there should be no need to worry about binding the method here.
There are two such references I saw in the code and this PR corrects them both. I haven't spotted any similar errors affecting refreshTokenExpiredBehaviorSubject, but am a little bit perplexed as I can't seem to find any code making reference to that Observable beyond its allocation.
Perhaps one of the two lines affected here is meant to target the refresh token and not the access token? I didn't dig that far just yet...
The callbacks for token expiration are being set to the return value from a call to pushTokenExpired() rather than a reference to the function itself.
...rather than...
The method in question does not depend on "this" in any way--the Observable it accesses is referenced as "Keycloak.tokenExpiredBehaviorSubject", so there should be no need to worry about binding the method here.
There are two such references I saw in the code and this PR corrects them both. I haven't spotted any similar errors affecting
refreshTokenExpiredBehaviorSubject, but am a little bit perplexed as I can't seem to find any code making reference to that Observable beyond its allocation.Perhaps one of the two lines affected here is meant to target the refresh token and not the access token? I didn't dig that far just yet...