-
Notifications
You must be signed in to change notification settings - Fork 369
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
Issued At Claim Errors #320
Comments
Thanks for reporting this @sixfootsixdesigns. Could you show your configuration for the leeway? |
|
I can confirm this issue as well, using Here is a related issue in Perhaps they are related. |
This logic was introduced in version |
I discovered that the error message given doesn't use the correct iat timestamp. This might be why it seems that the 180 second leeway option isn't working. This is the issue I filed related to that problem: #322 |
Hi everyone - after discussing this internally, we've taken the decision to remove the iat value check altogether, which should make this a non-issue. We will still be checking for presence and type, but will not be comparing the issued-at time. I will be raising a PR in the very near future to correct this. |
@stevehobbsdev Thank you. I never understood why the browser should check iat. That's for the resource server to do. |
Description
We are consistently getting issued at claim errors when our users attempt to login. In this example, the current time is dec 12th 2019 12:54:29 GMT-0600 and the issued at time is dec 12th 2019 12:54:48 GMT-0600. I have set the leeway option to 3 minutes to attempt to solve the issue but it is clearly not working as expected. We are using the loginWithRedirect method on the library to login the users.
Reproduction
using the loginWithRedirect method on the library
Environment
Let me know if you need any more information.
The text was updated successfully, but these errors were encountered: