-
Notifications
You must be signed in to change notification settings - Fork 92
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
Add keychain synchronisation and authentication context #221
Conversation
6f33289
to
c3630bb
Compare
Please check the build failures on iOS. It's unfortunate that |
c3630bb
to
ac1909a
Compare
ac1909a
to
dd45957
Compare
@kornelski, I fixed the MR & tests (I had forgotten to include a cfg) but I had not realised you had pushed your own changes and overwrote your push :( -- Would you mind re-applying? Thoughts welcome on |
dd45957
to
6a650bc
Compare
Thanks @kornelski, I can see the As for the |
I think the cleanest solution would be to hide the new field in |
6a650bc
to
88ed949
Compare
Alright, it's done. Let me know your thoughts. |
88ed949
to
8864ece
Compare
I'm an idiot 🤦 - it's fixed now. |
8864ece
to
77392cf
Compare
Added the ability to pass an LAContext to the keychain key operations as well as the kSecAttrSynchronizable flag during the creation of keys.