-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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
Segfault in secureResponseCachingAndRedirects on Android #437
Comments
I am not sure it is related but since I migrated an bitmap downloader from HttpClient to OkHttp (v1.3.0) I have random crashes :
|
Ok I think it is related, I have some url redirect. As my code already follow redirects, this line fixed the crashes :
Thanks |
@eboudrant curious. Protocol redirects are those from HTTP to HTTPS. |
@eboudrant By the way, does this also fix your problem? #184 |
Ok, make sense, some urls I use do a protocol redirect http://farm6.staticflickr.com/5243/5231017493_6ae9145a3d_c.jpg I don't know about #184 I did not participate |
I'm testing this again and it looks like getting the NPN protocols is returning bogus data. I got 165 chars each time. The first time this:
The 2nd time this:
The test case that produces this reliably runs two tests in sequence:
The problem happens on |
Tracking upstream. https://code.google.com/p/android/issues/detail?id=66562 |
Workaround: avoid reusing SSLContext. For both client and server. This is unlikely to cause problems in practice. I'm marking as closed since there's no action we can take. |
I saw this in 2 of 3 runs:
The text was updated successfully, but these errors were encountered: