Skip to content
New issue

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

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

Already on GitHub? # to your account

OutOfMemoryError in interceptor #218

Closed
jakoss opened this issue Jan 30, 2020 · 5 comments · Fixed by #267
Closed

OutOfMemoryError in interceptor #218

jakoss opened this issue Jan 30, 2020 · 5 comments · Fixed by #267
Labels
bug Something isn't working

Comments

@jakoss
Copy link

jakoss commented Jan 30, 2020

📢 Describe the bug

This came to me from firebase crashlytics, i don't know the reproduction steps

Fatal Exception: java.lang.OutOfMemoryError: Failed to allocate a 13536178 byte allocation with 5074056 free bytes and 4MB until OOM
       at okio.Buffer.readByteArray(Buffer.java:1438)
       at okio.Buffer.readString(Buffer.java:301)
       at okio.Buffer.readString(Buffer.java:290)
       at com.chuckerteam.chucker.api.ChuckerInterceptor.processResponseBody(ChuckerInterceptor.java:156)
       at com.chuckerteam.chucker.api.ChuckerInterceptor.processResponse(ChuckerInterceptor.java:130)
       at com.chuckerteam.chucker.api.ChuckerInterceptor.intercept(ChuckerInterceptor.java:61)
       at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:112)
       at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:87)
// rest of interceptors

💣 To Reproduce

I don't know how to reproduce that

🔧 Expected behavior

Chucker should handle any payload size gracefully

📱 Smartphone

  • Device: Galaxy A5 (2016)
  • OS: 7.1.1
  • Lib version: 3.1.1
@vbuberen vbuberen added the bug Something isn't working label Jan 30, 2020
@vbuberen
Copy link
Collaborator

Hi @NekroMancer
Any updates on this issue? Were you able to see it in Crashlytics again?

@jakoss
Copy link
Author

jakoss commented Feb 18, 2020

Hi,

No, not a single occurence in 2 weeks

@vbuberen vbuberen added the incomplete This issue needs more data in order to be triaged label Feb 18, 2020
@vbuberen
Copy link
Collaborator

Got it, thanks. I am not closing it, just wanted a status update.

You have Chucker somewhere in production app?
Or it is just Crashlytics for internal QA devices?

@vbuberen vbuberen added the help wanted Extra attention is needed label Feb 18, 2020
@jakoss
Copy link
Author

jakoss commented Feb 18, 2020

I have chucker on staging version for internal and external (clients) QA. It does not go into production itself

@vbuberen
Copy link
Collaborator

vbuberen commented Mar 6, 2020

I believe that this is the same problem as in #263

@ghost ghost added the Pending PR The resolution for the issue is in PR label Mar 6, 2020
@ghost ghost removed Pending PR The resolution for the issue is in PR bug Something isn't working help wanted Extra attention is needed incomplete This issue needs more data in order to be triaged labels Mar 25, 2020
@cortinico cortinico added the bug Something isn't working label Mar 25, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants