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

use microtime(true) instead of time() for the JWT validation #523

Open
bshaffer opened this issue Jun 28, 2023 · 0 comments
Open

use microtime(true) instead of time() for the JWT validation #523

bshaffer opened this issue Jun 28, 2023 · 0 comments
Labels
type: feature request v7.0 changes targeted for the next major version

Comments

@bshaffer
Copy link
Collaborator

See #488 and #492

use microtime instead of time for JWT validation.

We MAY want to do this in a minor version because it's possible that gettimeofday (which is required by microtime) doesn't exist on all systems. It may be possible to verify that the function does exist, and fall back on date if not.

@bshaffer bshaffer added type: feature request v7.0 changes targeted for the next major version labels Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature request v7.0 changes targeted for the next major version
Projects
None yet
Development

No branches or pull requests

1 participant