-
-
Notifications
You must be signed in to change notification settings - Fork 32
Change to use OkHttp instead of TubeSock #13
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
Comments
This may actually solve another problem I'm having, related to getting it to use the configured proxy server. Apparently TubeSock does not support the configured HTTP proxy, which makes it difficult to use with tools like Charles proxy. OkHttp appears to use the system proxy automatically, so that would be helpful. @rogerhu is this something that we can test in our applications using the latest snapshots, or would we have to manually create the OkHttp websocket client in order to test it? |
Looks like we can create a class in the com.parse package to hoist access to the |
I'm going to change now. I really don't think we should be using TubeSocket especially since it is using the old Apache HttpClient which is no longer being supported by Android. |
TubeSockWebSocketClient library is being used. We should just standardize on OkHttp v3.6.0 (and possibly share instances between Parse/Parse Live Query?) One advantage is that we can use ParseLogHttpInterceptor to troubleshoot network requests.
The text was updated successfully, but these errors were encountered: