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

Need discussion about the "refresh_token_expires_in" field. #10474

Open
SH5H opened this issue Apr 3, 2024 · 2 comments
Open

Need discussion about the "refresh_token_expires_in" field. #10474

SH5H opened this issue Apr 3, 2024 · 2 comments
Labels
documentation Relates to documentation triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime.

Comments

@SH5H
Copy link

SH5H commented Apr 3, 2024

What is the improvement or update you wish to see?

Since "github" is not the only one that supports this field, it would be good to mention in the documentation which providers support this field and provide a workaround.

Is there any context that might help us understand?

I found that one of built-in providers, kakao, supports the "refresh_token_expires_in" field, so we did some additional work accordingly.

Does the docs page already exist? Please link to it.

No response

@SH5H SH5H added documentation Relates to documentation triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime. labels Apr 3, 2024
@SH5H SH5H changed the title I think there should be a discussion about the refresh_token_expires_in field. Need discussion about the refresh_token_expires_in field. Apr 3, 2024
@SH5H SH5H changed the title Need discussion about the refresh_token_expires_in field. Need discussion about the "refresh_token_expires_in" field. Apr 3, 2024
@ndom91
Copy link
Member

ndom91 commented Apr 12, 2024

What would you like to use this field for exactly? We don't document every additional field that each provider supports / returns, so I don't think that makes sense, but maybe we can come up with a more spec-compliant / supported way to do what you're trying to do 🤔

@SH5H
Copy link
Author

SH5H commented Apr 15, 2024

By default, this field throws an error when used with the Prisma adapter. Personally, I'd like to see a small mention on the Prisma adapter page about the error and workaround when using a provider that supports this field.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Relates to documentation triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime.
Projects
None yet
Development

No branches or pull requests

2 participants