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

cached tokens are never expired #3

Open
nikhilbobb opened this issue Jan 2, 2015 · 4 comments
Open

cached tokens are never expired #3

nikhilbobb opened this issue Jan 2, 2015 · 4 comments
Labels

Comments

@nikhilbobb
Copy link

Cached tokens do not expire and are used after their expiration. This leads to an error being thrown, and the API request failing.

For example, I make a request at 11am, and a token is generated with a (default) 1 hour expiration. I then make another request at 2pm, and an expired token is used to make the request.

I realize I can clear the tokens, but a simple one liner can check for expiry.

@lukejagodzinski
Copy link
Owner

Yes in deed, token expires but it isn't checked if it expired so new token is not generated. I know about it and I'm planing to fix it. For now I suggest generating token on every request or just do manual check. I will notify you when I fix it.

@duinzaghi
Copy link

Hello, I'm using your packages to request Google Calendar API. I make sure everything work fine before. But in now, it's only work in localhost. In deployment server, it's always return "Invalid Credential". I think my issues relate to the expiration token. Can you suggest any idea to resolved this issues? Thanks!

@lukejagodzinski
Copy link
Owner

I've published a new version that fixes tokens expiration issue. Please test it out for me, because I haven't made test for this package. Let me know if everything works.

@duinzaghi
Copy link

Oh, great ! Thank you so much. It worked

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

3 participants