Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sentry ANR in SelfVerificationViewModel init #8482

Closed
BillCarsonFr opened this issue May 30, 2023 · 3 comments · Fixed by #8485
Closed

Sentry ANR in SelfVerificationViewModel init #8482

BillCarsonFr opened this issue May 30, 2023 · 3 comments · Fixed by #8485
Assignees
Labels
Z-ANR Application Not Responding

Comments

@BillCarsonFr
Copy link
Member

BillCarsonFr commented May 30, 2023

runBlocking in getCryptoDeviceInfo causing ANRs

io.sentry.android.core.ApplicationNotResponding: ANR for at least 5000 ms.
    at jdk.internal.misc.Unsafe.park(Unsafe.java)
    at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:234)
    at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:59)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:77)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(Builders.kt:7)
    at org.matrix.android.sdk.internal.crypto.RustCryptoService.getCryptoDeviceInfo(RustCryptoService.kt:4)
    at im.vector.app.features.crypto.verification.self.SelfVerificationViewModel.<init>(SelfVerificationViewModel.kt:118)
    at im.vector.app.DaggerVectorApplication_HiltComponents_SingletonC$MavericksViewModelCImpl$SwitchingProvider$95.create(DaggerVectorApplication_HiltComponents_SingletonC.java:2)
    at im.vector.app.DaggerVectorApplication_HiltComponents_SingletonC$MavericksViewModelCImpl$SwitchingProvider$95.create(DaggerVectorApplication_HiltComponents_SingletonC.java:1)
    at im.vector.app.core.di.HiltMavericksViewModelFactory.create(HiltMavericksViewModelFactory.kt:57)
    at im.vector.app.features.crypto.verification.self.SelfVerificationViewModel$Companion.create(SelfVerificationViewModel.kt:1)
    at im.vector.app.features.crypto.verification.self.SelfVerificationViewModel$Companion.create(SelfVerificationViewModel.kt:2)
    ```
@BillCarsonFr BillCarsonFr added the Z-ANR Application Not Responding label May 30, 2023
@BillCarsonFr BillCarsonFr self-assigned this May 30, 2023
@BillCarsonFr
Copy link
Member Author

Also in

io.sentry.android.core.ApplicationNotResponding: ANR for at least 5000 ms.
    at jdk.internal.misc.Unsafe.park(Unsafe.java)
    at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:234)
    at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:59)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:77)
    at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(Builders.kt:7)
    at org.matrix.android.sdk.internal.crypto.RustCryptoService.getCryptoDeviceInfo(RustCryptoService.kt:4)
    at im.vector.app.features.home.UnknownDeviceDetectorSharedViewModel.<init>(UnknownDeviceDetectorSharedViewModel.kt:62)
    at im.vector.app.DaggerVectorApplication_HiltComponents_SingletonC$MavericksViewModelCImpl$SwitchingProvider$82.create(DaggerVectorApplication_HiltComponents_SingletonC.java:2)
    at im.vector.app.DaggerVectorApplication_HiltComponents_SingletonC$MavericksViewModelCImpl$SwitchingProvider$82.create(DaggerVectorApplication_HiltComponents_SingletonC.java:1)
    at im.vector.app.core.di.HiltMavericksViewModelFactory.create(HiltMavericksViewModelFactory.kt:57)
    at im.vector.app.features.home.UnknownDeviceDetectorSharedViewModel$Companion.create(UnknownDeviceDetectorSharedViewModel.kt:1)
    at im.vector.app.features.home.UnknownDeviceDetectorSharedViewModel$Companion.create(UnknownDeviceDetectorSharedViewModel.kt:2)

@Tlay-1991
Copy link

Tlay-1991 commented May 30, 2023 via email

@Tlay-1991
Copy link

Tlay-1991 commented May 30, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-ANR Application Not Responding
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants