Skip to content
This repository was archived by the owner on Apr 20, 2026. It is now read-only.

Close all ignored response bodies - #70

Merged
henriquetruta merged 1 commit into
spotify:masterfrom
thna123459:close-all-ignored-responses
Aug 12, 2021
Merged

henriquetruta merged 1 commit into
spotify:masterfrom
thna123459:close-all-ignored-responses

Conversation

@thna123459

Copy link
Copy Markdown
Contributor

This fixes several warnings of this form (#65):
A connection to https://github.com/ was leaked. Did you forget to close a response body?

According to the okhttp documentation:
"Response bodies must be closed and may be consumed only once"
https://square.github.io/okhttp/4.x/okhttp/okhttp3/-response/body/

This fixes several warnings of this form (spotify#65):
A connection to https://github.com/ was leaked. Did you forget to close a response body?

According to the okhttp documentation:
"Response bodies must be closed and may be consumed only once"
https://square.github.io/okhttp/4.x/okhttp/okhttp3/-response/body/
@thna123459

Copy link
Copy Markdown
Contributor Author

I have only tested these changes with statuses; if you prefer this implementation, I will close #69

@henriquetruta
henriquetruta merged commit 6e16ed9 into spotify:master Aug 12, 2021
@thna123459
thna123459 deleted the close-all-ignored-responses branch August 12, 2021 07:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants