-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial implementation of reauthentication flow #124
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Pending real-world test
Tested in real world. Functionally works, but gives this warning:
|
Implement migration flow Enhance reauth flow
…-myskoda into implement-reauth
Gah, stupid linter. |
@WebSpider are you still working on this one? |
Hope to make some time for this over the holiday, but if you have something ready, dont let me stop you |
This is too out of sync. I will rework this and submit a new PR. |
Reauthentication is triggered when initial login fails.
Needs real-world testing still