github/awesome-cordova-plugins
github/awesome-cordova-plugins
2
0
Fork 1
mirror of https://github.com/danielsogl/awesome-cordova-plugins.git synced 2025-02-21 00:23:00 +08:00
Code Issues Projects Releases Wiki Activity

fix(keychain): mark useTouchID as required (#4182)

Browse Source 
`useTouchID` is not an optional parameter. When it is not explicitly set, the whole app crashes with this error:
> Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[NSNull boolValue]: unrecognized selector sent to instance 0x111f02f08'

This bug is known since November 2018 and likely won't be fixed: https://github.com/ionic-team/cordova-plugin-ios-keychain/issues/34
This commit is contained in:
Pascal Clanget 2022-05-21 20:22:58 +02:00 committed by GitHub
parent f5b7b8469b
commit 6b2c1fd32e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/@awesome-cordova-plugins/plugins/keychain/index.ts
View File

@ -17,7 +17,7 @@ import { Cordova, AwesomeCordovaNativePlugin, Plugin } from '@awesome-cordova-pl
*
* ...
*
* this.keychain.set(key, value).then(() => {
* this.keychain.set(key, value, false).then(() => {
* this.keychain.get(key)
* .then(value => console.log('Got value', value))
* .catch(err => console.error('Error getting', err));
@ -57,7 +57,7 @@ export class Keychain extends AwesomeCordovaNativePlugin {
@Cordova({
callbackOrder: 'reverse',
})
set(key: string, value: string | number | boolean, useTouchID?: boolean): Promise<any> {
set(key: string, value: string | number | boolean, useTouchID: boolean): Promise<any> {
return;
}