search

denniedegroot / com.ring

Ring for Homey
https://www.athom.com
Other
15 stars 15 forks source link

Added MFA support #48

Closed kaohlive closed 4 years ago

kaohlive commented 4 years ago

This is a version that supports the MFA requirement. I added the authorization to the new device wizard. First the user enters their credentials and when they receive their MFA code they can enter it in the now visible box. Then pressing authorize this MFA code is posted to get the refresh token and bearer token. The refresh token is then stored and a refresh pattern has been added to use that token to get new bearer tokens. So no more user credentials are being stored and the app should remain authorized for a long period (not very well tested at this moment).

denniedegroot commented 4 years ago

Thanks for the pull request! I will look into it asap.

denniedegroot commented 4 years ago

I merged it and did some small changes with a rebase. Could you check if the master branch is still working correctly?

kaohlive commented 4 years ago

Looks good from my quick test.