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

[BUG] Datetime should be a float instead of string #702

Closed
yassinrais opened this issue Mar 14, 2021 · 4 comments
Closed

[BUG] Datetime should be a float instead of string #702

yassinrais opened this issue Mar 14, 2021 · 4 comments

Comments

@yassinrais
Copy link
Contributor

Generated by the method ->toString()

eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJodHRwOi8vMTI3LjAuMC4xOjgwMDAiLCJhdWQiOiJodHRwOi8vMTI3LjAuMC4xOjgwMDAiLCJpYXQiOiIxNjE1NzUwODQzLjk2Mzg4MCIsImp0aSI6ImYyMjYwYjZjNTM5MTk5YjFmZGM1YmFjODFlNWIxNDNhIiwiZXhwIjoiMTYxNzA0MzI0My45NjM4ODAiLCJ1aWQiOiI1MDA5OTFjMS1lMWNhLTQ5ODUtYjBiMy1mNWNiYmRhNmU5MGIifQ._FgRzQFb4wtXk7zDoQofZH30KyhYx2jKOtyWh-cQtjw

[IMAGE] decoded by official JWT website

{
  "iss": "http://127.0.0.1:8000",
  "aud": "http://127.0.0.1:8000",
  "iat": "1615750843.963880",
  "jti": "f2260b6c539199b1fdc5bac81e5b143a",
  "exp": "1617043243.963880",
  "uid": "500991c1-e1ca-4985-b0b3-f5cbbda6e90b"
}

We should expect

{
  "iss": "http://127.0.0.1:8000",
  "aud": "http://127.0.0.1:8000",
  "iat": 1615750843.963880,
  "jti": "f2260b6c539199b1fdc5bac81e5b143a",
  "exp": 1617043243.963880,
  "uid": "500991c1-e1ca-4985-b0b3-f5cbbda6e90b"
}

Instead of the previus one

@yassinrais yassinrais changed the title [BUG] Dates should be a float instead of strings [BUG] Datetime should be a float instead of strings Mar 14, 2021
@yassinrais yassinrais changed the title [BUG] Datetime should be a float instead of strings [BUG] Datetime should be a float instead of string Mar 14, 2021
@yassinrais
Copy link
Contributor Author

duplicate of #618

@zerkms
Copy link

zerkms commented Mar 19, 2021

This should be reopened: the bug is there and the referred PR is closed.

@yassinrais
Copy link
Contributor Author

we are working in a possible solution #706

@lcobucci
Copy link
Owner

Hello folks, thanks to @yassinrais we managed to fix this compatibility issue whilst still preserving the microseconds precision. We already released new versions to solve the problem.

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

No branches or pull requests

3 participants