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
feat: allow deep links in redirectTo validation #513
Conversation
a4bb915
to
59e3665
Compare
Please, can you add tests for the new schemas supported? Also, same for the go bit (file |
b67a013
to
98d8ab3
Compare
98d8ab3
to
f8240bd
Compare
case url.endsWith('/**'): | ||
break; | ||
case url.endsWith('/*'): | ||
url += '*'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will this modify the original string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will mutate the url
but here we're just looping through a copy of AUTH_ACCESS_CONTROL_ALLOWED_REDIRECT_URLS
and so the array of the allowed redirect urls won't be mutated
No description provided.