Broken link causes a leak

Question

duduBaiao opened this issue 8 years ago · 8 comments

Hi,

It seems that if the url is broken the connection leaks.

When that happens I see a message like this on the console:

W/OkHttp: A connection to http://feeds-tmp.soundcloud.com/ was leaked. Did you forget to close a response body?

Answer 1 · 2016-09-26T01:50:40.000Z

Which version did you test on? I have tested this url, but I didn't meet this error.

Answer 2 · 2016-09-26T14:23:32.000Z

Thanks for answering @coolerfall.

I'm using the last master commit.

You can test on any invalid/broken url. The url showed above is not complete. Please try this one:

You'll see something like this on the log:

W/OkHttp (25435): A connection to http://podcast.canaltech.com.br/ was leaked. Did you forget to close a response body?

Answer 3 · 2016-09-27T01:54:22.000Z

I still didn't get this message, just get 404 error however.

Answer 4 · 2016-09-27T14:39:56.000Z

Yeah. That's right. This happens to me too.

But my app tries to download the same file several times, after a short delay. Can you please test that situation?

Answer 5 · 2016-09-27T15:29:39.000Z

1.Try to download serveral times, with different DownloadRequest or same DownloadRequest?
2.After a short delay, what does this mean?

Answer 6 · 2016-09-27T17:11:04.000Z

1 - With different instances of "DownloadRequest".
2 - 15 seconds

Answer 7 · 2016-09-28T06:59:57.000Z

Ok, I got this message. I'v changed some code, and I didn't get this message anymore. U can checkout on develop branch.

Answer 8 · 2016-09-28T18:47:49.000Z

I'll test it in the next days. ;)