Crypton/statusapp

corner case: Touch ID remains for multiple users

Closed this issue · 9 comments

reneelynn commented
  1. Enable touchID for a user
  2. Force quit app and reopen
  3. Create a new user
  4. force quit app and reopen

TouchID will still be in effect for the new user.

iOS6
Kloak build: 0.6.0pre 20151013

daviddahl commented

Thanks!

daviddahl commented

Wait, was TouchId in effect for the original user after creating a new user?

reneelynn commented

once you create a new user it doesn't save the credentials for the first user, so no?

daviddahl commented

I think this is fixed in the new build

reneelynn commented

Build 0.7.0 20151017 is where you expect it to be fixed? It isn't. Please let me know if there's a newer build to check against.

daviddahl commented

I think after re-reading the issue STR, this is not a bug. There might be some new functionality wanted, like remembering all of the handles a user uses to login - and allowing touchID for all of them, but the behavior here is expected, as we only remember the last user( to login) credentials.

reneelynn commented

so you don't consider it a bug that user 2, who never personally set the preference for touchid, automatically has touchid enabled?

daviddahl commented

Yes. For now, its is automatic. When you create a new account TouchID is automatically set on. This, I think wil be preferred by most users. In fact I think we will have more folks wondering why it is not the default that not.

daviddahl commented

Naturally, we need to think through how to really handle this after an appropriate amount of user feedback comes in:) I could be totally wrong.