Skip to content

Commit 8f7173b

Browse files
authored
Merge pull request #188 from Infomaniak/delete-legacy-account-if-migration-canceled
fix: Delete legacy account if we remove the account after a canceled migration
2 parents 2a42361 + e36c577 commit 8f7173b

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

multiplatform-lib/src/commonMain/kotlin/internal/managers/AuthenticatorManager.kt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,10 @@ import com.infomaniak.auth.lib.internal.extensions.firstOrElse
2525
import com.infomaniak.auth.lib.internal.models.ClientExtensionResults
2626
import com.infomaniak.auth.lib.internal.models.VerifyAuthenticationData
2727
import com.infomaniak.auth.lib.internal.models.VerifyResponse
28+
import com.infomaniak.auth.lib.internal.otp.deleteLegacyAccount
29+
import com.infomaniak.auth.lib.internal.otp.deleteLegacyDB
30+
import com.infomaniak.auth.lib.internal.otp.getLegacyAccounts
31+
import com.infomaniak.auth.lib.internal.otp.needMigration
2832
import com.infomaniak.auth.lib.internal.repositories.AccountsRepository
2933
import com.infomaniak.auth.lib.internal.requests.WebAuthnRequests
3034
import com.infomaniak.auth.lib.internal.utils.SignUtils
@@ -135,6 +139,11 @@ internal class AuthenticatorManager(
135139
}
136140

137141
accountsRepository.deleteAccount(userId)
142+
143+
if (needMigration()) {
144+
deleteLegacyAccount(userId.toString())
145+
if (getLegacyAccounts().isEmpty()) deleteLegacyDB()
146+
}
138147
}
139148

140149
suspend fun deleteKeysFor(userId: Long) {

0 commit comments

Comments
 (0)