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

[Auth] Refresh Tokens expire abnormally and API respond with "invalid request" #82

Open
josephpage opened this issue Sep 4, 2017 · 7 comments
Labels

Comments

@josephpage
Copy link
Contributor

Almost every 2 days, one of the refresh_tokens expires (out of a total of ~20), and the API respond with a enigmatic :

{"error":"invalid_request"}

The only solution is the get a new refresh_token by manually enter the credentials in the consent screen. I don't have precise stats about this issue, but it is too frequent and serious enough for production environments. Happens since ... mmm... 4 months.

Just occurs this morning with a relatively new refresh_token got last week : 2017-08-30 10:37:40 UTC

If the refresh_token is expired for some reasons, the API should return a appropriate and explicit error message, to allow apps to properly handle the errors and send a renewal request to the account owner.

@josephpage
Copy link
Contributor Author

cc @alexjomin

@alexjomin
Copy link
Contributor

Do you have the HTTP code of the response ? It will be easier to point it out in the nginx log. Thank you

@josephpage
Copy link
Contributor Author

Mmm I wait for the error to happen again.
Do refresh tokens have an expiration ?

@josephpage
Copy link
Contributor Author

josephpage commented Sep 12, 2017

Mmm I remember refresh_token are valid for 48 hours (#24)
so why {"error":"invalid_request"} ? the error should be Invalid authentication according to documentation... (https://dev.xee.com/doc/api/api/v3/auth/access_token.md)

@alexjomin
Copy link
Contributor

alexjomin commented Sep 12, 2017

yes @josephpage it's look like a bug. We will give a look and let you know.

@alexjomin alexjomin added the bug label Sep 12, 2017
@josephpage
Copy link
Contributor Author

Last times it happened :

  • 9/24/2017 10:53
  • 9/21/2017 15:06
  • 9/20/2017 22:15

@josephpage
Copy link
Contributor Author

Happened many times last week and today...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants