Merge branch 'soru/mutex-lock-store' into 'main'

fix: SecureStorage sometimes reading wrong / bad values

See merge request famedly/fluffychat!369
This commit is contained in:
Krille Fear 2021-02-03 14:54:35 +00:00
commit f4594eae26

View File

@ -40,9 +40,30 @@ Future<Database> getDatabase(Client client) async {
Database _db; Database _db;
bool _generateDatabaseLock = false; bool _generateDatabaseLock = false;
// see https://github.com/mogol/flutter_secure_storage/issues/161#issuecomment-704578453
class AsyncMutex {
Completer<void> _completer;
Future<void> lock() async {
while (_completer != null) {
await _completer.future;
}
_completer = Completer<void>();
}
void unlock() {
assert(_completer != null);
final completer = _completer;
_completer = null;
completer.complete();
}
}
class Store { class Store {
LocalStorage storage; LocalStorage storage;
final FlutterSecureStorage secureStorage; final FlutterSecureStorage secureStorage;
static final _mutex = AsyncMutex();
Store() Store()
: secureStorage = PlatformInfos.isMobile ? FlutterSecureStorage() : null; : secureStorage = PlatformInfos.isMobile ? FlutterSecureStorage() : null;
@ -69,6 +90,7 @@ class Store {
} }
} }
try { try {
await _mutex.lock();
return await secureStorage.read(key: key); return await secureStorage.read(key: key);
} catch (_) { } catch (_) {
return null; return null;
@ -89,6 +111,7 @@ class Store {
await _setupLocalStorage(); await _setupLocalStorage();
return await storage.setItem(key, value); return await storage.setItem(key, value);
} }
await _mutex.lock();
return await secureStorage.write(key: key, value: value); return await secureStorage.write(key: key, value: value);
} }
@ -101,6 +124,7 @@ class Store {
await _setupLocalStorage(); await _setupLocalStorage();
return await storage.deleteItem(key); return await storage.deleteItem(key);
} }
await _mutex.lock();
return await secureStorage.delete(key: key); return await secureStorage.delete(key: key);
} }
} }