Skip to content

fixes refresh scheme #99

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

Closed
wants to merge 1 commit into from
Closed

fixes refresh scheme #99

wants to merge 1 commit into from

Conversation

chriscab83
Copy link

includes grant_type 'refresh_token' on all refreshes.
updates local provider to use refreshToken.data attribute if it is included in options, defaults to 'refresh_token' if not.

updates local provider to use refreshToken.data attribute if it is included in options, defaults to 'refresh_token' if not.
Comment on lines +151 to +152
// this is required to be set in the provider or it will delete the refresh token
endpoint.body!.grant_type = 'refresh_token';
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The local scheme provider requires this to be set to refresh_token or it deletes the refresh token from the request body.

@Denoder Denoder mentioned this pull request Jan 8, 2024
@Denoder Denoder closed this Jan 8, 2024
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.

2 participants