-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: warn for incompat nextauth url and port #1596
feat: warn for incompat nextauth url and port #1596
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🦋 Changeset detectedLatest commit: b3ddb87 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Current dependencies on/for this PR: This comment was auto-generated by Graphite. |
851ad0f
to
7d26141
Compare
7d26141
to
d9e36af
Compare
console.warn( | ||
`NEXTAUTH_URL (${env.NEXTAUTH_URL}) doesn't match with PORT (${env.PORT})` | ||
); |
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.
is logging the right thing or should it be a hard throw?
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.
imo throw. logs are easy to miss, and if someone really wants to develop with a port that doesn't match their nextauth url (can't imagine this) they can always turn it off.
d9e36af
to
fbd87cb
Compare
fbd87cb
to
b4c0356
Compare
Changed to erroring @c-ehrlich |
b4c0356
to
055ff23
Compare
A new create-t3-app prerelease is available for testing. You can install this latest build in your project with: pnpm create [email protected] |
Hmm looks like Looks like they only set this if they are the one overriding the default: So to summarize, this will work if you run EDIT: actually looks like they do set it but not immediately? |
closing for now since it doesn't seem possible to achieve this without having it run on every request. would still be nice to find a way to do this in the future / if next changes their behaviour on this |
Supersedes #1591