Skip to content
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

fix: properly handle callback URL fallback #3402

Merged
merged 4 commits into from
Dec 8, 2021
Merged

Conversation

balazsorban44
Copy link
Member

#3399 was a bit rushed, introduced some small bugs. Testing/Fixing them here

@balazsorban44 balazsorban44 temporarily deployed to Preview December 8, 2021 17:02 Inactive
@github-actions
Copy link

github-actions bot commented Dec 8, 2021

🎉 Experimental release published on npm!

npm i next-auth@0.0.0-pr.3402.be5db520
yarn add next-auth@0.0.0-pr.3402.be5db520

@codecov-commenter
Copy link

codecov-commenter commented Dec 8, 2021

Codecov Report

Merging #3402 (fd9b5e0) into main (b0f6175) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3402      +/-   ##
==========================================
+ Coverage   13.45%   13.47%   +0.01%     
==========================================
  Files          90       90              
  Lines        1405     1403       -2     
  Branches      360      358       -2     
==========================================
  Hits          189      189              
+ Misses       1205     1203       -2     
  Partials       11       11              
Impacted Files Coverage Δ
src/core/index.ts 0.00% <0.00%> (ø)
src/core/init.ts 0.00% <ø> (ø)
src/next/index.ts 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0f6175...fd9b5e0. Read the comment docs.

@balazsorban44 balazsorban44 temporarily deployed to Preview December 8, 2021 17:08 Inactive
@balazsorban44 balazsorban44 temporarily deployed to Preview December 8, 2021 17:17 Inactive
@balazsorban44 balazsorban44 merged commit 30a0fc6 into main Dec 8, 2021
@balazsorban44 balazsorban44 deleted the fix/callback-url branch December 8, 2021 17:20
mnphpexpert added a commit to mnphpexpert/next-auth that referenced this pull request Sep 2, 2024
* fix: don't default to localhost on `host`

* fall back to `host` for `callbackUrl`

* use parsed host

* remove unnecessary type cast
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Refers to `@auth/core`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants