-
Notifications
You must be signed in to change notification settings - Fork 1k
fix: return browser to 'return_to' when logging in without registered account using oidc. #2496
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2496 +/- ##
==========================================
+ Coverage 75.21% 75.32% +0.10%
==========================================
Files 295 295
Lines 16379 16386 +7
==========================================
+ Hits 12320 12343 +23
+ Misses 3116 3102 -14
+ Partials 943 941 -2
|
9928e7f
to
23565ab
Compare
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.
Awesome, thank you! 🎉 Your contribution makes Ory better :)
By the way, I have made a few adjustments, PTAL :)
f637cdf
to
c74a815
Compare
Nice work! LGTM 🚀! |
…ration using oidc.
d289b85
to
982854b
Compare
… account using oidc. (ory#2496) Close ory#2444 Co-authored-by: aeneasr <[email protected]>
… account using oidc. (ory#2496) Close ory#2444 Co-authored-by: aeneasr <[email protected]>
When handling the callback from the oidc provider and starting a new registration flow to sign up the user we create a new temp request that contains the initial request parameters from the login flow.
Related issue(s)
Proposal to close #2444.
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
[email protected]) from the maintainers to push
the changes.
works.
Further Comments