Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
51e714c to
1c6c787
Compare
36a65d6 to
daf9967
Compare
| private val clientId: String | ||
| get() = auth0.clientId | ||
|
|
||
| private var transactionState: EmbeddedAuthState? = null |
There was a problem hiding this comment.
we are capturing transactionState here. Can we not add authSession in the function signature itself for client to send it. We do send auth session in EmbeddedAuthException right?
There was a problem hiding this comment.
No . This is intentional. Please re check the design doc. SDK is expected to handle the auth_session within it and not expose it to the calling clients
Summary
EmbeddedAuthClientwith the full multi-step embedded/e/authorizeflow:authorize,identifyEmail,identifyPhone,challengeEmail,verifyOtpEmbeddedAuthExceptioncarrying a rotatedauth_sessionandnextActions;verifyOtpis the terminal step that yieldsCredentialsauthorize()now acceptsscope(default"openid profile email offline_access") and optionalaudienceso the token exchange returns anid_tokenauthorizeUrlto alazyproperty to avoid recomputing the URL on every step