search

Crypton / statusapp

A status application for small groups to keep each other in the loop
Mozilla Public License 2.0
16 stars 4 forks source link

corner case: Touch ID remains for multiple users #186

Closed reneelynn closed 8 years ago

reneelynn commented 8 years ago

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 8 years ago

Thanks!

daviddahl commented 8 years ago

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

reneelynn commented 8 years ago

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

daviddahl commented 8 years ago

I think this is fixed in the new build

reneelynn commented 8 years ago

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 8 years ago

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 8 years ago

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 8 years ago

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 8 years ago

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.