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

#71 Check access token expiration #72

Merged

Conversation

josephdpurcell
Copy link

Fixes #71

@josephdpurcell
Copy link
Author

I haven't tested this yet, I'll do that now.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.17% when pulling 498c9c7 on josephdpurcell:hotfix/1.1.1/check-access-token into e8b4c87 on typhonius:1.1.

@josephdpurcell
Copy link
Author

I was able to test and confirm this works:

  • When this patch is not applied and there is 300 seconds between requests the request fails.
  • When this patch is applied and there is 300 seconds between requests the request succeeds.

@weitzman
Copy link

Thanks for backporting this massgov/openmass#390

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.

None yet

4 participants