Fix issue for getting otp using keycloak auth#2937
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2937 +/- ##
==========================================
+ Coverage 77.79% 78.48% +0.68%
==========================================
Files 471 673 +202
Lines 34647 55277 +20630
Branches 728 728
==========================================
+ Hits 26955 43386 +16431
- Misses 7614 11813 +4199
Partials 78 78
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
EmilyRagan
left a comment
There was a problem hiding this comment.
Fixes the problem I was having with Enterprise 👍🏻
|
How was I not running into this? |
It didn't affect core. But if you didn't run into this in enterprise, then it was probably a race condition in how it was being used. If something requested a normal session token before requesting an OTP, then the keycloak class' |
Fix issue for getting otp using keycloak auth



Quick fix to handle differences between core and enterprise (keycloak) auth classes (creating token in constructor vs. in token getter)
related to #2911