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.

Sign up for GitHub

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

Jump to bottom

Chucker is crashing opening responses with big bodies #55

Closed
cortinico opened this issue Jun 13, 2019 · 0 comments · Fixed by #72
Closed

Chucker is crashing opening responses with big bodies #55

cortinico opened this issue Jun 13, 2019 · 0 comments · Fixed by #72
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@cortinico
Copy link
Member

Describe the bug

I do have an app that is calling an endpoint that returns a JSON Body of nearly 700kb.
Chucker is blocking the UI thread trying to parse the body and is causing an ANR/Crash if trying to inspect the response body.

To Reproduce

  1. Use Chucker to call an endpoint with a big response
  2. Click on the Chucker Push notification
  3. Open the HTTP Request from the list
  4. Click on the "Response" tab

Expected behavior
The tab should open immediately

Actual behavior
The Chucker UI is completely frozen and eventually crashes

Smartphone

  • Emulator - Nexus 5X
  • OS: API27
  • Lib version: 3.0.0-alpha1
@cortinico cortinico added bug Something isn't working help wanted Extra attention is needed labels Jun 13, 2019
@redwarp redwarp mentioned this issue Aug 2, 2019
Merged
redwarp pushed a commit to redwarp/chucker that referenced this issue Aug 7, 2019
Fetch payload in AsyncTask
4cd4c31 
To avoid the formating of json or xml stalling the UI, the payload will be loaded and formatted in an async task instead.
Fixes ChuckerTeam#55
cortinico added a commit that referenced this issue Aug 7, 2019
@cortinico
Fetch payload in AsyncTask (#72)
0192512 
* Fetch payload in AsyncTask

To avoid the formating of json or xml stalling the UI, the payload will be loaded and formatted in an async task instead.
Fixes #55

* Format with ktlint
* Fix post rebase, move bitmap deserialization to the UiLoaderTask
* Fix detekt issue
* More formatting fixes. (missing \n)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fetch payload in AsyncTask redwarp/chucker
1 participant
@cortinico