Skip to content

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

Merged
merged 9 commits into from
Jul 21, 2022

Conversation

Urbansson
Copy link
Contributor

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

  • I have read the contributing guidelines.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I am following the
    contributing code guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    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.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

Further Comments

@CLAassistant
Copy link

CLAassistant commented May 30, 2022

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented May 30, 2022

Codecov Report

Merging #2496 (20af911) into master (f89d279) will increase coverage by 0.10%.
The diff coverage is 74.69%.

❗ Current head 20af911 differs from pull request most recent head 982854b. Consider uploading reports for the commit 982854b to get more accurate results

@@            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     
Impacted Files Coverage Δ
cmd/courier/watch.go 76.27% <ø> (ø)
cmd/identities/definitions.go 71.42% <ø> (+15.65%) ⬆️
driver/config/config.go 82.17% <ø> (ø)
selfservice/strategy/oidc/strategy_registration.go 60.52% <0.00%> (-1.22%) ⬇️
x/pagination.go 100.00% <ø> (ø)
selfservice/flow/login/handler.go 78.48% <16.66%> (-2.35%) ⬇️
x/http_secure_redirect.go 90.47% <66.66%> (+0.60%) ⬆️
cmd/identities/list.go 81.81% <80.00%> (+10.38%) ⬆️
cmd/daemon/serve.go 86.70% <80.64%> (+1.39%) ⬆️
cmd/identities/delete.go 94.87% <100.00%> (ø)
... and 16 more

@Urbansson Urbansson changed the title Return browser to 'return_to' when logging in without registered account using oidc. fix: return browser to 'return_to' when logging in without registered account using oidc. May 31, 2022
@aeneasr aeneasr force-pushed the oidc-redirect-to-return-to branch from 9928e7f to 23565ab Compare July 19, 2022 14:46
aeneasr
aeneasr previously approved these changes Jul 19, 2022
Copy link
Member

@aeneasr aeneasr left a 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 :)

@aeneasr aeneasr force-pushed the oidc-redirect-to-return-to branch from f637cdf to c74a815 Compare July 19, 2022 15:04
@Urbansson
Copy link
Contributor Author

Urbansson commented Jul 20, 2022

Nice work!

LGTM 🚀!

@aeneasr aeneasr force-pushed the oidc-redirect-to-return-to branch from d289b85 to 982854b Compare July 21, 2022 07:21
@aeneasr aeneasr merged commit a4194f5 into ory:master Jul 21, 2022
gi0baro pushed a commit to casavo/kratos that referenced this pull request Nov 18, 2022
peturgeorgievv pushed a commit to senteca/kratos-fork that referenced this pull request Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

returnTo query not being honoured in the seamless registration flows when using OIDC.
3 participants