-
Notifications
You must be signed in to change notification settings - Fork 147
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
error: client_assertion token has an expiration too far into the future #382
Comments
one hacky workaround is to modify the variable: okta.jwt.JWT.ONE_HOUR -= 5 # allow 5 seconds of clock skew |
Hi, this issue is open for more than a year. Is there a plan to fix this? We hit this error today when moving from API Tokens to PrivateKey authentication. As it is currently implemented, we don't feel comfortable using PrivateKey authentication in production. As suggested above, a solution would be to reduce the expiry_time by a few seconds to allow for clock skew:
Or even better, using 45-min instead of 1h. Is there a reason why the expiry time is set for so long in the future? |
i'm guessing not. see also #375. there are dozens of open issues going back years.
i wouldn't feel comfortable using something that doesn't get bug fixes or is replaced every few years -- there was an old beta SDK that never went GA, then the current one, now they're doing it again -- that's 3 times in 4 years!? i just use i have example code. if you're interested, let me know. |
60 minutes is too long and causes error okta#382
i just submitted this PR #434 |
Thanks for the PR! Hopefully it gets reviewed soon. Also, I appreciate the link on the SDK refresh. I wasn't aware of the plan for a new Python SDK. Our team will wait a bit longer before committing to writing our own. |
Okta node sdk sets expiration for 5 minutes:
https://github.com/okta/okta-sdk-nodejs/blob/895d48c9bf4e2e1d0d2fadcf0a84cc550e301851/src/jwt.js#L72-L84
whereas Python uses 60 minutes:
okta-sdk-python/okta/jwt.py
Lines 114 to 122 in 700c5f1
i frequently get
errors on Python, but never on node. i set my clock using time.windows.com, but it's usually 1 sec fast
The text was updated successfully, but these errors were encountered: