Add synchronized option #2
Conversation
|
Did these changes come from the repo we forked? |
| { | ||
| NSDictionary *query = @{ | ||
| (__bridge NSString *)kSecClass: (__bridge id)(kSecClassGenericPassword), | ||
| (__bridge NSString *)kSecAttrService: service, | ||
| (__bridge NSString *)kSecReturnAttributes: (__bridge id)kCFBooleanTrue, | ||
| (__bridge NSString *)kSecAttrSynchronizable: (__bridge id)kCFBooleanTrue |
There was a problem hiding this comment.
Does this implicitly make everything synchronised?
There was a problem hiding this comment.
It seems like that but I might need a test to make sure it is
No, it wasn't, I refered to react-native-keychain issue => oblador#156 (comment) and https://developer.apple.com/documentation/security/ksecattrsynchronizable |
|
code looks good to me 👍 but, if A & B are synced, and A logs out, then what will happen to B? will it be usable or we need to clear other account-related(non-secure) data from B too? |
I will test this case once the build has done |
No description provided.