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 Oauth Stuff #164

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PokeJofeJr4th
Copy link
Contributor

  • Remove package.lock from .gitignore
  • Update the seed data to use nextauth's new format

@PokeJofeJr4th PokeJofeJr4th linked an issue Sep 15, 2024 that may be closed by this pull request
@rtyocum
Copy link

rtyocum commented Sep 15, 2024

Issue Caused by Next-Auth version bump. Older versions used this in the Account database
image

Next-Auth does not allow 2 providers of the same email address. Next-Auth updated provider from google.com to google. The image above breaks oAuth.

image
This is the current version of the Google Auth in next-auth: https://github.com/nextauthjs/next-auth/blob/e67a1eb961f232eb55a8ef7cf6be783b97bd4187/packages/next-auth/src/providers/google.ts

package-lock.json was committed to prevent inadvertent version bumps, seed data was also updated to match new version

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.

Bug: Oauth sessions need to be cleared
2 participants