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

[Bug]: ANR Card.logImpression #266

Open
ashraf-patel-hs opened this issue Mar 4, 2024 · 4 comments
Open

[Bug]: ANR Card.logImpression #266

ashraf-patel-hs opened this issue Mar 4, 2024 · 4 comments
Labels

Comments

@ashraf-patel-hs
Copy link

Braze Android SDK Version

23.2.1

Steps To Reproduce

Card.logImpression is causing the ANR when performed on the main thread.

Expected Behavior

NO ANRs or documentation on which thread shall we use to perform such operation

Actual Incorrect Behavior

It's causing an ANR

main (waiting):tid=1 systid=3971
at jdk.internal.misc.Unsafe.park(Native method)
at java.util.concurrent.locks.LockSupport.park(LockSupport.java:211)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquire(AbstractQueuedSynchronizer.java:715)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquire(AbstractQueuedSynchronizer.java:938)
at java.util.concurrent.locks.ReentrantLock$Sync.lock(ReentrantLock.java:153)
at java.util.concurrent.locks.ReentrantLock.lock(ReentrantLock.java:322)
at bo.app.p.a(SourceFile:10)
at com.appboy.models.cards.Card.logImpression(SourceFile:5)

Verbose Logs

No response

Additional Information

Are we supposed to execure Card.logImpression on the UI thread?

@bryanlogan
Copy link
Contributor

It should run just fine there. Can you provide verbose logs as well, along with any other stacks that contain Braze in the stack at the time of the crash?

If you don't want to provide those via GitHub, then go through Braze support so we can handle it there.

@ashraf-patel-hs
Copy link
Author

@bryanlogan thanks for the reply. That's all we get related to braze, starts from braze's Card.logImpression is called from our codebase, inside a fragment which causes the ANR.

Could you please give me a hint on what other logs you need?

@bryanlogan
Copy link
Contributor

The output of "adb" contains other Braze information. You can add "BrazeLogger.logLevel = Log.VERBOSE" in your Application class to turn on as much logging as possible.

Does this recreate in any sample app?

@abkanov
Copy link

abkanov commented Mar 5, 2024

@bryanlogan It is not reproducible. Logs shared in the original message are the only thing we have from Crashlytics/Play Console

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants