AND-6313 [Biometrics] Fixed keystore master key invalidation #359
Conversation
iiiburnyiii
requested review from
Mama1emon,
kozarezvlad,
Yoggam1,
Sateetas and
iMaks99
as code owners
| get() = requireNotNull(canAuthenticateInternal.value) { | ||
| "`canAuthenticate` has not been initialized" | ||
| get() { | ||
| check(value = biometricsStatus.value != BiometricsStatus.UNINITIALIZED) { |
There was a problem hiding this comment.
неочевидный чек, в условии видишь проверку на != UNINITIALIZED, а мессадж по результату, что должно быть инвертировано условие
| return | ||
| } | ||
| override fun onPause(owner: LifecycleOwner) { | ||
| biometricsStatus.value = BiometricsStatus.UNINITIALIZED |
There was a problem hiding this comment.
это не выстрелит выше в чеках, когда например залочен экран и прошло время для запроса биометрии? либо приложение свернуто и также прошли 90сек
There was a problem hiding this comment.
Не выстрелит, но место все таки скользкое.
Это изменение состояния будет иметь эффект, только если активити было запущено и её остановили, это не даст запустить авторизацию пока активити не откроют.
А если промпт был открыт, а активити остановили, то он сам закроется и выкинет ошибку.
| if (canAuthenticate) { | ||
| withContext(Dispatchers.Main) { | ||
| authenticateInternal() | ||
| if (params.timeout != null) { |
There was a problem hiding this comment.
мб вынести в переменную, чтобы !! не вызвать, смотрится почище
| BiometricsStatus.UNINITIALIZED -> { | ||
| Log.biometric { "Awaiting for the biometrics status to be initialized" } | ||
|
|
||
| biometricsStatus.first { it != BiometricsStatus.UNINITIALIZED } |
There was a problem hiding this comment.
не совсем понял что делает эта строчка
… key configurations
…key configurations
iiiburnyiii
force-pushed
the
bugfix/AND-6313_biometry_fix
branch
from
9048c92 to
648a5e7
Compare
tangem-sdk-android/src/main/java/com/tangem/sdk/authentication/AndroidAuthenticationManager.kt
Show resolved
Hide resolved
tangem-sdk-android/src/main/java/com/tangem/sdk/authentication/AndroidAuthenticationManager.kt
Show resolved
Hide resolved
tangem-sdk-android/src/main/java/com/tangem/sdk/authentication/AndroidAuthenticationManager.kt
Outdated
Show resolved
Hide resolved
tangem-sdk-core/src/main/java/com/tangem/common/authentication/storage/AuthenticatedStorage.kt
Outdated
Show resolved
Hide resolved
tangem-sdk-core/src/main/java/com/tangem/common/authentication/keystore/DummyKeystoreManager.kt
Show resolved
Hide resolved
tangem-sdk-android/src/main/java/com/tangem/sdk/authentication/AndroidKeystoreManager.kt
Outdated
Show resolved
Hide resolved
No description provided.