mirror of
https://github.com/aliasvault/aliasvault.git
synced 2026-05-19 13:57:18 -04:00
Cleanup (#520)
This commit is contained in:
@@ -60,8 +60,6 @@ class AutofillService : AutofillService() {
|
||||
}
|
||||
|
||||
try {
|
||||
Log.d(TAG, "onFillRequest called")
|
||||
|
||||
// Check if request was cancelled
|
||||
if (cancellationSignal.isCanceled) {
|
||||
return
|
||||
@@ -123,15 +121,11 @@ class AutofillService : AutofillService() {
|
||||
}
|
||||
|
||||
private fun launchActivityForAutofill(fieldFinder: FieldFinder, callback: (FillResponse?) -> Unit) {
|
||||
Log.d(TAG, "Launching activity for autofill authentication")
|
||||
|
||||
// Get the app/website information from assist structure.
|
||||
val appInfo = fieldFinder.getAppInfo()
|
||||
Log.d(TAG, "Autofill request from: $appInfo")
|
||||
|
||||
// Ignore requests from our own unlock page as this would cause a loop
|
||||
if (appInfo == "net.aliasvault.app") {
|
||||
Log.d(TAG, "Skipping autofill request from AliasVault app itself")
|
||||
callback(null)
|
||||
return
|
||||
}
|
||||
@@ -144,7 +138,6 @@ class AutofillService : AutofillService() {
|
||||
if (store.tryGetAllCredentials(object : CredentialOperationCallback {
|
||||
override fun onSuccess(result: List<Credential>) {
|
||||
try {
|
||||
Log.d(TAG, "Retrieved ${result.size} credentials")
|
||||
if (result.isEmpty()) {
|
||||
// No credentials available
|
||||
Log.d(TAG, "No credentials available")
|
||||
@@ -217,7 +210,6 @@ class AutofillService : AutofillService() {
|
||||
})
|
||||
) {
|
||||
// Successfully used cached key - method returns true
|
||||
Log.d(TAG, "Successfully retrieved credentials with unlocked vault")
|
||||
return
|
||||
}
|
||||
}
|
||||
|
||||
@@ -190,7 +190,6 @@ class AliasVaultCredentialProviderService : CredentialProviderService() {
|
||||
): BeginCreateCredentialResponse? {
|
||||
// Get or initialize VaultStore instance (needed for subsequent passkey registration)
|
||||
VaultStore.getExistingInstance() ?: run {
|
||||
Log.d(TAG, "VaultStore not initialized, initializing now...")
|
||||
// Need a FragmentActivity reference for biometrics, but this is a service
|
||||
// We'll create a minimal VaultStore instance here
|
||||
val keystoreProvider = AndroidKeystoreProvider(applicationContext) { null }
|
||||
|
||||
@@ -114,8 +114,6 @@ class CredentialIdentityStore private constructor(context: Context) {
|
||||
prefs.edit()
|
||||
.putString(KEY_PASSKEY_IDENTITIES, jsonArray.toString())
|
||||
.apply()
|
||||
|
||||
Log.d(TAG, "Saved ${passkeyIdentities.size} passkey identities")
|
||||
} catch (e: Exception) {
|
||||
Log.e(TAG, "Error saving credential identities", e)
|
||||
}
|
||||
@@ -163,7 +161,6 @@ class CredentialIdentityStore private constructor(context: Context) {
|
||||
prefs.edit()
|
||||
.remove(KEY_PASSKEY_IDENTITIES)
|
||||
.apply()
|
||||
Log.d(TAG, "Removed all credential identities")
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -42,7 +42,6 @@ class PasskeyRegistrationActivity : FragmentActivity() {
|
||||
try {
|
||||
// Get or initialize VaultStore instance
|
||||
vaultStore = VaultStore.getExistingInstance() ?: run {
|
||||
Log.d(TAG, "VaultStore not initialized, initializing now...")
|
||||
val keystoreProvider = AndroidKeystoreProvider(applicationContext) { this }
|
||||
val storageProvider = AndroidStorageProvider(applicationContext)
|
||||
VaultStore.getInstance(keystoreProvider, storageProvider)
|
||||
@@ -129,7 +128,6 @@ class PasskeyRegistrationActivity : FragmentActivity() {
|
||||
}
|
||||
},
|
||||
)
|
||||
Log.d(TAG, "Waiting for biometric authentication...")
|
||||
} catch (e: Exception) {
|
||||
Log.e(TAG, "Error in onCreate", e)
|
||||
finish()
|
||||
@@ -143,7 +141,6 @@ class PasskeyRegistrationActivity : FragmentActivity() {
|
||||
try {
|
||||
// Check for existing passkeys
|
||||
val db = vaultStore.database
|
||||
Log.d(TAG, "Db is not null?: ${db != null}")
|
||||
|
||||
if (db != null) {
|
||||
viewModel.existingPasskeys = vaultStore.getPasskeysWithCredentialInfo(
|
||||
|
||||
@@ -611,8 +611,6 @@ class NativeVaultManager(reactContext: ReactApplicationContext) :
|
||||
@ReactMethod
|
||||
override fun copyToClipboardWithExpiration(text: String, expirationSeconds: Double, promise: Promise?) {
|
||||
try {
|
||||
Log.d(TAG, "Copying to clipboard with expiration of $expirationSeconds seconds")
|
||||
|
||||
val clipboardManager = reactApplicationContext.getSystemService(
|
||||
android.content.Context.CLIPBOARD_SERVICE,
|
||||
) as android.content.ClipboardManager
|
||||
@@ -689,7 +687,6 @@ class NativeVaultManager(reactContext: ReactApplicationContext) :
|
||||
val alarmManager = reactApplicationContext.getSystemService(android.content.Context.ALARM_SERVICE) as android.app.AlarmManager
|
||||
|
||||
if (!alarmManager.canScheduleExactAlarms()) {
|
||||
Log.d(TAG, "Requesting exact alarm permission via system settings")
|
||||
val intent = Intent().apply {
|
||||
action = Settings.ACTION_REQUEST_SCHEDULE_EXACT_ALARM
|
||||
data = Uri.parse("package:${reactApplicationContext.packageName}")
|
||||
@@ -724,7 +721,6 @@ class NativeVaultManager(reactContext: ReactApplicationContext) :
|
||||
} else {
|
||||
true // Pre-Android 6.0 doesn't have battery optimization
|
||||
}
|
||||
Log.d(TAG, "Is ignoring battery optimizations: $isIgnoring")
|
||||
promise?.resolve(isIgnoring)
|
||||
} catch (e: Exception) {
|
||||
Log.e(TAG, "Error checking battery optimization status", e)
|
||||
|
||||
@@ -162,7 +162,6 @@ class AndroidKeystoreProvider(
|
||||
val keyFile = File(context.filesDir, ENCRYPTED_KEY_FILE)
|
||||
keyFile.writeText(encryptedKeyB64)
|
||||
|
||||
Log.d(TAG, "Encryption key stored successfully")
|
||||
callback.onSuccess("Key stored successfully")
|
||||
} catch (e: Exception) {
|
||||
Log.e(TAG, "Error storing encryption key", e)
|
||||
|
||||
@@ -119,10 +119,7 @@ class WebApiService(private val context: Context) {
|
||||
if (requiresAuth && !requestHeaders.containsKey("Authorization")) {
|
||||
getAccessToken()?.let { accessToken ->
|
||||
requestHeaders["Authorization"] = "Bearer $accessToken"
|
||||
Log.d(TAG, "Added Authorization header from stored token")
|
||||
}
|
||||
} else if (requiresAuth && requestHeaders.containsKey("Authorization")) {
|
||||
Log.d(TAG, "Using provided Authorization header instead of stored token")
|
||||
}
|
||||
|
||||
// Add client version header
|
||||
@@ -318,8 +315,6 @@ class WebApiService(private val context: Context) {
|
||||
*/
|
||||
suspend fun extractFavicon(url: String): ByteArray? = withContext(Dispatchers.IO) {
|
||||
try {
|
||||
Log.d(TAG, "Extracting favicon for $url")
|
||||
|
||||
val response = executeRequest(
|
||||
method = "GET",
|
||||
endpoint = "Favicon/Extract?url=$url",
|
||||
|
||||
Reference in New Issue
Block a user