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

Pass additional params to OAuth provider to be used in signIn callback #716

Open
1 of 5 tasks
TiborPL opened this issue Mar 18, 2024 · 2 comments
Open
1 of 5 tasks
Labels
enhancement An improvement that needs to be added help-needed Action needed: The help of the community would be appreciated p2 Nice to have provider-authjs An issue with the authjs provider

Comments

@TiborPL
Copy link

TiborPL commented Mar 18, 2024

Describe the feature

I seached everywhere I could, looked through all the old issues, PRs but I could only find unanswered questions/requests, hacks with unoptimal solutions.
I basically would like to "reopen" this issue, at least with an clear solution if anyone has one: #303

How would you implement this?

I'm not familiar enough with the whole implementation to suggest a solution.

Additional information

  • Would you be willing to help implement this feature?

Provider

  • AuthJS
  • Local
  • Refresh
  • New Provider
@TiborPL TiborPL added enhancement An improvement that needs to be added pending labels Mar 18, 2024
@zoey-kaiser zoey-kaiser added p2 Nice to have provider-authjs An issue with the authjs provider and removed pending labels Mar 30, 2024
@zoey-kaiser
Copy link
Member

Hi @TiborPL 👋

From what I remember from the previous discussion, this limitation is due to NextAuth. We has thought about finding a way to add support through this, inside NuxtAuth, however this is sadly a pretty low priority for us at the moment.

We are currently also beginning on implementing support for authjs. I think once we have finished this migration, we could take some time to consider possible solutions ourselves. If in the meantime your or anyone else has an idea on how to implement this, we can definitely prioritize this!

@zoey-kaiser zoey-kaiser added the help-needed Action needed: The help of the community would be appreciated label Mar 30, 2024
@TiborPL
Copy link
Author

TiborPL commented Apr 2, 2024

@zoey-kaiser thank you for the thorough explanation. I understand and excited to hear the migration to authjs :)
I solved my issue so it's not urgent for me anymore. Might revisit this when the module is getting to a final version..

Thanks again for the work you've all done for this module 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement that needs to be added help-needed Action needed: The help of the community would be appreciated p2 Nice to have provider-authjs An issue with the authjs provider
Projects
None yet
Development

No branches or pull requests

2 participants