-
Notifications
You must be signed in to change notification settings - Fork 60
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
Crash on Datadog v2.0.0 on DdRumContentProvider: java.lang.ClassNotFoundException #1584
Comments
hi @RamiTrabelsi - we'll look into this asap. Do you have any additional details that might help in debugging this crash (for example, do you know whether these are these upgraded or fresh installations? does this issue seem to occur only on specific device types? does it affect all or only some Android 10?) |
Hello, Thanks for the reactivity. We don't have the info about if the devices having the crash, are upgraded or fresh installations. We know it's 100% Android 10 and the crash is happening in the background (it seems happening on app startup). Since the crash was growing, we decided to revert to the old version on 1.x.x. |
Hello @RamiTrabelsi. Can you please open your |
@RamiTrabelsi Thanks for the quick return. Do you see this crash only on Android 10? If you run the app on say Android 11, does it crash with SDK v2? Can you also share AGP version you are using? I see that this class is not in a primary DEX file, can you try to follow the docs here to push this file to the primary DEX file and see if it helps? |
Thanks for the quick reply. On our latest release version, we have today 9 crash events impacting 8 users. 8 of them are on Android 10, and 1 appeared today on Android 9. Our actual AGP version is 7.4.2. Okay, I'll try to add to our
Thanks for your help! |
Hello @RamiTrabelsi! Do you have any update on this issue? Did the proposed solution help you? |
Hey mates @RamiTrabelsi is off today. I'll try to test it today. Otherwise, we'll bring news next week. Thanks! |
Hello @0xnm, so it seems to work as I could push that class to primary DEX. But since we were blocked by this issue to push this change to production, we couldn't yet 100% confirm that it's resolved as the new version is not yet pushed to production (we reverted) so for now we can't see if previously affected devices by the crash, don't have it anymore. Maybe we can close this thread, and if we see that the problem is still there (rare chances) we can re-open the issue. Thank you very much for the help 🙏 |
Hello @0xnm, it seems that the crash is back again. I made sure that the class Stacktrace:
Thanks for your help ! |
Hello @RamiTrabelsi! This is a quite strange issue, indeed. I managed to find a similar issue for Can you give us a bit more information based on the discussions in the threads I linked:
|
Hello @RamiTrabelsi! I'm closing this ticket due to inactivity. Please don't hesitate to re-open it if needed. |
Hi, I would like the issue to be reopened as we are facing the exact same crash, only happening on some Android 10 devices in our production app. I couldn't reproduce the issue on my Android 10 test device, so any proposed fix will have to be sent to production to be validated. To answer the questions you asked above:
We are on AGP 8.3.2
It is not We didn't try the Proguard rule solution proposed above, this is one of the fix we're planning to try even if it doesn't seem to have worked for @RamiTrabelsi. Is there anything we can try, or information we can provide to help fix this issue? |
Hi @NFesquet , Are you publishing your application via an APK or App Bundle ? If you rename the Also, can you try making sure that all the Datadog libraries you're using are using the same version in your app? |
Thanks for reopening the issue.
Both, depending on the targeted country. In the country we see the most crashes, the app is distributed via apk. But there are occurrences in countries where app is distributed via aab.
Tried it on apk,
Confirmed, all lib versions (logs, http, rum and trace) are centralised in a single value, 2.10.0 currently. |
Yes so it's indeed the class not being kept in the primary DEX, let me know how the proguard rule works. |
Hey ! Just wanted to let you know that the crash is unfortunately still present even after adding the proguard rule 😞 (@NFesquet also upgraded to the 2.11.0 version for all Datadog libraries) |
Hello @WessimBetclic! Can you please confirm that after adding a Proguard rule |
Hi @0xnm, For a reason I cannot explain, the Proguard rule does not put
android {
buildTypes {
getByName("release") {
multiDexKeepProguard = file("multidex-config.pro")
...
}
}
}
I also tried both at the same time. Every solution will put file in either |
Hi @0xnm, I opened an issue in Google tracker and according to them:
Updating the Proguard rule to keep class didn't work, so I'm a little short of ideas right now. |
@NFesquet Honestly it looks like as unfortunate issue of the particular Android version itself, I doubt we can do anything on our side. Can you try to use the newer SDK versions? Maybe AGP will re-arrange classes and issue will be gone. |
Hello,
Since the migration to Datadog v2.0.0 on Android, we're facing a new crash happening on devices on Android 10.
Stack Trace:
Do you know how can we fix this?
Thank you
The text was updated successfully, but these errors were encountered: