-
Notifications
You must be signed in to change notification settings - Fork 442
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
fix: improve secret generation for apple idp #7843
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7843 +/- ##
===========================================
+ Coverage 50.10% 62.98% +12.88%
===========================================
Files 1244 1333 +89
Lines 117337 110058 -7279
===========================================
+ Hits 58788 69325 +10537
+ Misses 55594 36783 -18811
- Partials 2955 3950 +995
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
* fix: improve secret generation for apple idp * remove accidental commit * change exp time * change exp time * change exp time * change exp time (cherry picked from commit 6ab06aa)
🎉 This PR is included in version 2.51.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
A customer reported that the login with Apple not always works, but often an invalid_client is returned.
We saw duplicate invocation of the callback in the logs, but are missing detailed information.
This PR adds additional logs. Since the authentication is relying on JWT, we also change the
iat
by 2 seconds into the past to mitigate potential clock skew problems.Additionally a previous unintended change in the config parsing is removed.
Definition of Ready