-
Notifications
You must be signed in to change notification settings - Fork 269
iOS10 Support? #48
Comments
Apparently going into Compatibility Settings and enabling Keychain Sharing 'fixes' the issue. Haven't been doing iOS specific development for a while now so hoping that bringing this up might shed some light.. EDIT: From talking to some iOS Dev friends apparently there haven't really even been much in the way of changes when it comes to the keychain for iOS10. It looks like the EDIT2: So it looks like this plugin is using a year old version of this plugin.. maybe that has something to do with the lack of iOS10 support.. Going to see what happens if I pull in more up to date files from that plugin EDIT3: That's a no go on pulling in the latest from that plugin. Would love to get some eyes on this since the iOS10 release is coming up soon @ggozad |
Hey @briananderson1222! |
Also note soffes/SAMKeychain#149 |
@ggozad Is there a reason the SAMKeychain code is so out of date? Also, is there a reason that 'SAM' is renamed to 'SS' or was this what the library used a year ago when this code was last updated? |
@briananderson1222 indeed SSKeychain apparently became SAMKeychain. There's no particular reason, SSKeychain was more or less a sane simple API around keychain which is why it was used. |
@ggozad Okay thanks for the background information. I'll continue looking into this as time allows and open a PR if I can get it working. Let me know if there is any other background information that may be useful |
I've made the upgrade to the latest version if you would like me to put up a PR for that.. unfortunately doesn't seem to fix the underlying Keychain issue on iOS10 |
Doomsday has come and gone. All seems fine here, did you encounter any trouble? |
@briananderson1222 closing this as everything seems fine on iOS 10. If you do encounter any problems let me know and will reopen. |
Hi @ggozad, @briananderson1222 I still encountering a similar issue on iphone 7 simulator (iOS 10), with Xcode version 8.0 (8A218a) and lastest plugin version from git master branch. The ss.set() method returns the following error :
Are you experiencing the same issue ? |
Hey @fmhun /cc @briananderson1222 We deployed the latest release successfully on our own app and have had no problems reported so far. Would appreciate verification of tests passing on different physical devices. |
My experience has been the same as @ggozad. If you do want it to work on a simulator I have had luck with enabling the 'Keychain Accessbility' Capability as mentioned in this issue: https://openradar.appspot.com/27844971. I would monitor this issue as it appears to be an issue that stems from the the simulator itself. |
Will leave this open even if it does not affect apps in production. If something changes please report here to keep everybody in the loop. |
Has a resolution been reached on this issue? |
Well everybody reported the problem only appears on the emulator and seems to be a known bug. Is this different for you? |
Well, kinda. I think we should leave this open for discussion in case a solution is found. |
I get this error also on a device. But not directly. After a fresh install and live building on the device at some point this issue appears and from then on it doesn't work anymore. Any news on this? |
Hey @wh33ler , sorry for the late reply, christmas and all :) |
I actually haven't been receiving errors with this under 10.2 and 10.2.1 |
@jvjvjv yep this is indeed the case, had forgotten to come back to this. |
Curious if anyone has looked into iOS 10 support for this plugin.
I tested this out for the first time today and I am seeing this error when calling
cordovaSecureStorage.set
I will continue looking into this to see if I can find more information and will pass along any findings
EDIT: This seems to affect all setting and fetching
The text was updated successfully, but these errors were encountered: