Add support for MFA #136
Add support for MFA #136
Conversation
|
@cam-rod very nice, I had this on my TO-DO list for a long time while finding some time... Will have a look and merge it soon, thanks again! Now I need to/can adapt the home-assistant integration too ;-) |
|
FWIW I just tested this, and it worked with my account too. |
|
I wonder, maybe we can add a wait for the emailed code, pull it, then use that to automate the entire process? |
|
Sounds like an good idea. It would involve setting up email authentication (ex. generating an app password for Gmail) and grabbing the right email. I don't have time to work on that, but if you want to fork off my branch feel free to do so. |
|
Retrieving the email and getting the code was easy honestly. I'm working on a withings to garmin sync. Basically merging 2 repos into one (yours and another lol). It's gonna be pretty ugly honestly, but just want it to work haha. |
|
Sorry for offtopic, but, @Cloud9Developer, do you really need to merge the repos together? Couldn't you have a third project, which uses the two libraries as dependencies? I made a small, quick script that syncs weight data from Fitbit to Garmin Connect. It's a single script, plus |
|
What I'm doing is pushing everything to a separate repo to make one project, then later on will reduce it down to more specific code. For now, I'm just wanting the functionality to work lol |
|
This can be closed without merging. The current version of |
Made a patch for this a while ago, so I got around to formalizing it. Checks if the server requested an MFA code, and if so calls a function to request it from the user. Tested successfully on my account and closes #129 .
Only major drawback is that it requires interactive input, so I used a timed prompt to make sure automated scripts don't get stuck forever.