kumi/Android-Password-Store
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: temporarily disable pre-launch biometric auth (#2803)

Browse source 
* fix: temporarily disable pre-launch biometric auth

Completely broken right now and prevents users from opening the app.

Updates #2802

* fix: disable UnusedResources lint
This commit is contained in:
Harsh Shandilya 2023-12-10 22:55:25 +05:30 committed by GitHub
parent d00626d5d3
commit a04d6fa9f4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 14 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/src/main/java/app/passwordstore/ui/main/LaunchActivity.kt
View file

@ -25,14 +25,14 @@ class LaunchActivity : AppCompatActivity() {
override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
val prefs = sharedPrefs
if (prefs.getBoolean(PreferenceKeys.BIOMETRIC_AUTH, false)) {
if (prefs.getBoolean(PreferenceKeys.BIOMETRIC_AUTH_2, false)) {
BiometricAuthenticator.authenticate(this) { result ->
when (result) {
is Result.Success -> {
startTargetActivity(false)
}
is Result.HardwareUnavailableOrDisabled -> {
prefs.edit { remove(PreferenceKeys.BIOMETRIC_AUTH) }
prefs.edit { remove(PreferenceKeys.BIOMETRIC_AUTH_2) }
startTargetActivity(false)
}
is Result.Failure,

17
app/src/main/java/app/passwordstore/ui/settings/GeneralSettings.kt
View file

@ -65,14 +65,15 @@ class GeneralSettings(private val activity: FragmentActivity) : SettingsProvider
defaultValue = false
}
val canAuthenticate = BiometricAuthenticator.canAuthenticate(activity)
switch(PreferenceKeys.BIOMETRIC_AUTH) {
// val canAuthenticate = BiometricAuthenticator.canAuthenticate(activity)
switch(PreferenceKeys.BIOMETRIC_AUTH_2) {
titleRes = R.string.pref_biometric_auth_title
defaultValue = false
enabled = canAuthenticate
summaryRes =
if (canAuthenticate) R.string.pref_biometric_auth_summary
else R.string.pref_biometric_auth_summary_error
enabled = false
// summaryRes =
// if (canAuthenticate) R.string.pref_biometric_auth_summary
// else R.string.pref_biometric_auth_summary_error
summary = "Temporarily disabled due to a bug, see issue 2802"
onClick {
enabled = false
val isChecked = checked
@ -81,7 +82,7 @@ class GeneralSettings(private val activity: FragmentActivity) : SettingsProvider
when (result) {
is Result.Success -> {
// Apply the changes
putBoolean(PreferenceKeys.BIOMETRIC_AUTH, checked)
putBoolean(PreferenceKeys.BIOMETRIC_AUTH_2, checked)
enabled = true
}
is Result.Retry -> {}
@ -89,7 +90,7 @@ class GeneralSettings(private val activity: FragmentActivity) : SettingsProvider
// If any error occurs, revert back to the previous
// state. This
// catch-all clause includes the cancellation case.
putBoolean(PreferenceKeys.BIOMETRIC_AUTH, !checked)
putBoolean(PreferenceKeys.BIOMETRIC_AUTH_2, !checked)
checked = !isChecked
enabled = true
}

1
app/src/main/java/app/passwordstore/util/settings/PreferenceKeys.kt
View file

@ -10,6 +10,7 @@ object PreferenceKeys {
const val APP_THEME = "app_theme"
const val AUTOFILL_ENABLE = "autofill_enable"
const val BIOMETRIC_AUTH = "biometric_auth"
const val BIOMETRIC_AUTH_2 = "biometric_auth_delete_soon_please"
@Deprecated(
message = "Use CLEAR_CLIPBOARD_HISTORY instead",
replaceWith = ReplaceWith("PreferenceKeys.CLEAR_CLIPBOARD_HISTORY"),

2
build-logic/src/main/kotlin/app/passwordstore/gradle/LintConfig.kt
View file

@ -40,6 +40,8 @@ object LintConfig {
disable += "FragmentFieldInjection"
// Too pedantic
disable += "ArgInFormattedQuantityStringRes"
// Biometric strings are temporarily unused due to issue 2802
disable += "UnusedResources"
}
baseline = project.file("lint-baseline.xml")
}