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

change errorType to invalid_grant on invalid refresh_token #1042

Merged
merged 5 commits into from
Aug 31, 2019
Merged

change errorType to invalid_grant on invalid refresh_token #1042

merged 5 commits into from
Aug 31, 2019

Conversation

omdmhd
Copy link
Contributor

@omdmhd omdmhd commented Jul 26, 2019

Fix Spec compliance according to this issue.
Now invalid_grant is returned instead of invalid_request.

@Sephster
Copy link
Member

Sephster commented Aug 31, 2019

Thanks @omdmhd - Changing this to the version 9 branch as this is BC but otherwise looks good. Cheers!

Sorry, I also noticed we should be returning a 400 response instead of 401. Will adjust that before merging

@Sephster Sephster changed the base branch from master to 9.0.0-WIP August 31, 2019 19:10
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