Skip to content
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

Secure storage should set IOSAccessibility accessibility #152

Closed
HannesVDB opened this issue Feb 18, 2021 · 2 comments · Fixed by #160
Closed

Secure storage should set IOSAccessibility accessibility #152

HannesVDB opened this issue Feb 18, 2021 · 2 comments · Fixed by #160
Assignees
Labels
enhancement New feature or request P0 Highest prio

Comments

@HannesVDB
Copy link
Contributor

HannesVDB commented Feb 18, 2021

IOSAccessibility accessibility is ignored for the secure storage

class SecureStorage extends SecureStoring {
  final FlutterSecureStorage _storage;
  final iOSOptions = IOSOptions(accessibility: IOSAccessibility.first_unlock);
  SecureStorage(this._storage);
  @override
  Future<void> deleteAll() {
    return _storage.deleteAll(iOptions: iOSOptions);
  }
  @override
  Future<void> write({@required String key, @required String value}) {
    return _storage.write(key: key, value: value, iOptions: iOSOptions);
  }
  @override
  Future<String> read({@required String key}) {
    return _storage.read(key: key, iOptions: iOSOptions);
  }
  @override
  Future<void> delete({@required String key}) {
    return _storage.delete(key: key, iOptions: iOSOptions);
  }
}
@vanlooverenkoen vanlooverenkoen added enhancement New feature or request P0 Highest prio labels Feb 18, 2021
@vanlooverenkoen
Copy link
Contributor

We had an issue where our app was working fine when in the foreground & in the background. But as soon as your phone was locked. you couldn't access the keychain anymore on iOS.

@vanlooverenkoen
Copy link
Contributor

This issue was found while debugging icapps/flutter-background-location-tracker#25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P0 Highest prio
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants