Open CoderSpinoza opened 6 years ago
@nicolasgarnier
Always thanks for your prompt review :) I added a new commit that contains several fixes you mentioned.
@nicolasgarnier
Any update on this PR? :)
Hey @CoderSpinoza sorry about that :) I'll get to it soon, I'm just done with a quite busy bunch of weeks.
@nicolasgarnier Haha, guess you are busy these days! Just get to it whenever you are free :)
Sorry for the delay ^^ I'll try to be faster next time :P
Any updates? :)
Any updates? This is very important.
Waiting for updates too.
Custom auth is not practical without the ability to merge accounts. +1
10 demonstrates developers' concerns with correctly linking custom oauth providers with users already signed in with privileged providers. @nicolasgarnier wrote a wonderful comment on how this can be achieved using Auth#setCustomUserClaims(uid, customClaims).
Although there is a limitation in this approach, compared to FirebaseUser#linkWithCredential() method, I assume this could be a fair guide to third-party developers for structuring complete auth system with multiple oauth providers including custom ones.
Could you review this pull request? @nicolasgarnier @samtstern
p.s. Token verification functionality was added to Kakao API so I also included it. :)