ProtonMail
/
protoncore_android
mirror of https://github.com/ProtonMail/protoncore_android
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

chore(network): Add a warning log if clientId is missing in `HumanVerificationNeededHandler`.

This commit is contained in:
Mateusz Armatys 2024-04-19 16:10:47 +02:00 committed by MargeBot
parent bbcc15eb6c
commit b2667fb9f4
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
network/domain/src/main/kotlin/me/proton/core/network/domain/handlers/HumanVerificationNeededHandler.kt
View File

@ -24,12 +24,14 @@ import me.proton.core.network.domain.ApiBackend
import me.proton.core.network.domain.ApiErrorHandler
import me.proton.core.network.domain.ApiManager
import me.proton.core.network.domain.ApiResult
import me.proton.core.network.domain.LogTag
import me.proton.core.network.domain.ResponseCodes
import me.proton.core.network.domain.client.ClientId
import me.proton.core.network.domain.client.ClientIdProvider
import me.proton.core.network.domain.humanverification.HumanVerificationAvailableMethods
import me.proton.core.network.domain.humanverification.HumanVerificationListener
import me.proton.core.network.domain.session.SessionId
import me.proton.core.util.kotlin.CoreLogger
import org.jetbrains.annotations.TestOnly
import java.util.concurrent.TimeUnit
@ -50,7 +52,11 @@ class HumanVerificationNeededHandler<Api>(
call: ApiManager.Call<Api, T>
): ApiResult<T> {
// Do we have a clientId ?
val clientId = clientIdProvider.getClientId(sessionId) ?: return error
val clientId = clientIdProvider.getClientId(sessionId)
if (clientId == null) {
CoreLogger.w(LogTag.DEFAULT, "HumanVerificationNeededHandler: clientId not found.")
return error
}
// Recoverable with human verification ?
if (error !is ApiResult.Error.Http || error.proton?.code != ResponseCodes.HUMAN_VERIFICATION_REQUIRED) {