Conversation
|
hi @bitsnaps , when you get the chance, could you help test and review this? thank you! |
|
Thanks @reynaldichernando ! While this may reduce the friction of maintaining multiple auth related code, this could limit some use cases in some conditions: automated env (CI/CD), terminal-only access, even termux from mobile... my best bet is give the user choice (e.g. Login via browser or via Credentials) |
|
hi @bitsnaps , thanks for the feedback, you are right about the concerns above i think what i can do to cater the above is, to first try the as for automated env, i think with the direction puter is going is using an |
|
actually after testing the |
|
updates:
kindly help to review @bitsnaps 🙏 i did test them and all of it works, except for 2FA because none of my account is 2FA enabled, plus i'm having some sort of issue in prod trying to enable it |
|
LGTM, I thought of the same thing except of |
getAuthTokenfunction which is meant to be used by nodejs cli apps