Skip to content

Fix device access token response error codes #1889

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

Conversation

nwholloway
Copy link
Contributor

This fixes two occurrences of the Device Access Token Response returning "authorization_pending" and not the expected response.

The first is when the device token has expired (as raised in #1885). This now returns the error "expired_token".

The second is when the device token has been invalidated, but the user token has not been invalidated. This state occurs when a request has been made with the incorrect client, or after device token expiry. This now returns the error "invalid_grant".

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 31, 2025
@jgrandja jgrandja self-assigned this Feb 5, 2025
@jgrandja jgrandja added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 5, 2025
@jgrandja jgrandja added this to the 1.3.5 milestone Feb 5, 2025
@jgrandja
Copy link
Collaborator

jgrandja commented Feb 5, 2025

@nwholloway Thank you. I will get to this next week. I'm trying to wrap up a priority task by end of this week.

@jgrandja jgrandja changed the title Correct Device Access Token error responses gh-1885 Fix device access token response error codes Feb 14, 2025
jgrandja added a commit that referenced this pull request Feb 14, 2025
@jgrandja
Copy link
Collaborator

Thanks for the PR @nwholloway. This is now merged.

FYI, I added a polish commit. Please let me know if you have any questions for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants