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]: com.onesignal.OneSignalRestClient.callResponseHandlerOnFailure #620

Open
1 task done
shefich opened this issue Jul 6, 2023 · 1 comment
Open
1 task done
Labels

Comments

@shefich
Copy link

shefich commented Jul 6, 2023

What happened?

As we can see these threads: OneSignal/OneSignal-Android-SDK#443 and segmentio/analytics-android#490
Passed several years and the issues are still there.
Think you need to have some check for creating a thread while the app closes or there is no memory.

Devices in my Crashlytics: Galaxy A03 Core
Andoid OSs: 11 and 12

Steps to reproduce?

1. Can't be reproduced. Probably shut down the app in time OneSignal creating a thread...

What did you expect to happen?

I expected no such issues. As they're ruining app health on the Google Play.

Unity version

2021.2.12 (latest)

OneSignal Unity SDK version

3.0.10

Platform

Android

Relevant log output

Fatal Exception: java.lang.OutOfMemoryError: pthread_create (1040KB stack) failed: Try again
       at java.lang.Thread.nativeCreate(Thread.java)
       at java.lang.Thread.start(Thread.java:887)
       at com.onesignal.OneSignalRestClient.callResponseHandlerOnFailure(OneSignalRestClient.java:296)
       at com.onesignal.OneSignalRestClient.startHTTPConnection(OneSignalRestClient.java:260)
       at com.onesignal.OneSignalRestClient.access$100(OneSignalRestClient.java:46)
       at com.onesignal.OneSignalRestClient$4.run(OneSignalRestClient.java:114)
       at java.lang.Thread.run(Thread.java:923)

Code of Conduct

  • I agree to follow this project's Code of Conduct
@emawby emawby added the Bug label Jul 10, 2023
@jkasten2
Copy link
Member

jkasten2 commented Mar 6, 2024

@shefich Thanks for reporting, what percentage of users / devices and total crashes have you seen for this issue?

The code has been completely refactored in the v5 release so this shouldn't be an issue any more. Could you try updating to the latest SDK and let us know if it fixes the issue for you? See our v5 migration guide to get started.

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