Skip to content

fix(core): detect Vercel without NEXTAUTH_URL#3649

Merged
balazsorban44 merged 3 commits into
mainfrom
fix/detect-vercel
Jan 16, 2022
Merged

fix(core): detect Vercel without NEXTAUTH_URL#3649
balazsorban44 merged 3 commits into
mainfrom
fix/detect-vercel

Conversation

@balazsorban44

Copy link
Copy Markdown
Contributor

Fixes #3419

Based on #3419 (comment)

@github-actions github-actions Bot added the core Refers to `@auth/core` label Jan 16, 2022
@codecov-commenter

codecov-commenter commented Jan 16, 2022

Copy link
Copy Markdown

Codecov Report

Merging #3649 (8ac3a9c) into main (aa0e820) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3649      +/-   ##
==========================================
- Coverage   13.11%   13.10%   -0.01%     
==========================================
  Files          91       91              
  Lines        1441     1442       +1     
  Branches      384      383       -1     
==========================================
  Hits          189      189              
- Misses       1238     1239       +1     
  Partials       14       14              
Impacted Files Coverage Δ
src/jwt/index.ts 0.00% <0.00%> (ø)
src/next/index.ts 0.00% <ø> (ø)
src/next/utils.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 aa0e820...8ac3a9c. Read the comment docs.

@balazsorban44 balazsorban44 temporarily deployed to Preview January 16, 2022 23:02 Inactive
@github-actions

Copy link
Copy Markdown

🎉 Experimental release published on npm!

npm i next-auth@0.0.0-pr.3649.27d8aa99
yarn add next-auth@0.0.0-pr.3649.27d8aa99

@jjorissen52

Copy link
Copy Markdown

This results in user-defined NEXTAUTH_URL being ignored in Vercel deployments.

mnphpexpert added a commit to mnphpexpert/next-auth that referenced this pull request Sep 2, 2024
* fix(core): detect Vercel without `NEXTAUTH_URL`

* chore(ts): use `any`

* chore: use `process.env.VERCEL` to detect Vercel
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.

Vercel deployment redirect URI

3 participants