-
Notifications
You must be signed in to change notification settings - Fork 433
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix: Updated URL service isCallbackFromSts
#1976
Conversation
…rect URI if config says to
… config parameter
Anything else needed on this one? :) |
Hey, thanks for this PR. I did not see this, yet. Will have a look tomorrow. Thanks!!! |
projects/angular-auth-oidc-client/src/lib/utils/url/url.service.ts
Outdated
Show resolved
Hide resolved
Small suggestion for beautifying the code, otherwise this is totally fine. Thank you very much! |
Co-authored-by: Fabian Gosebrink <[email protected]>
@FabianGosebrink looks like the "commit suggestion" feature didn't do well here, so I'm fixing locally now. Just a sec :) |
@FabianGosebrink @damienbod |
Added the option to match current URL to the redirect URI when determining whether or not it's a callback.
Fixes #1935
As per the comment in the v16 changes, the default value for
checkRedirectUrlWhenCheckingIfIsCallback
is nowtrue
.A pre-push hook for Prettier ran when I pushed my changes, so around 30 of the changed files are a result of that.