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.

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

Fixed #71 Set Socket Timeout for input stream from socket. #69

Merged
merged 1 commit into from
Mar 13, 2016

Conversation

hideki
Copy link

@hideki hideki commented Mar 13, 2016

TJWS set SO_TIMEOUT 0 in Serve.java. It causes hanging state in listener.

TJWS set SO_TIMEOUT 0 in Serve.java. It causes hanging state in listener.
pasin added a commit that referenced this pull request Mar 13, 2016
Set Socket Timeout for input stream from socket.
@pasin pasin merged commit 824645a into master Mar 13, 2016
@pasin pasin deleted the feature/fix_setSOTimeout_0 branch March 13, 2016 02:57
@pasin pasin removed the in progress label Mar 13, 2016
@hideki hideki changed the title Set Socket Timeout for input stream from socket. Fixed #71 Set Socket Timeout for input stream from socket. Mar 13, 2016
@hideki
Copy link
Author

hideki commented Sep 22, 2017

With current architecture, it does not expect that Listener (server) needs to wait for the request from the client (javascript). The server-side timeout could happen if a connection failure occurs during the client is sending a request.

@NitzDKoder
Copy link

@hideki

"The server-side timeout could happen if a connection failure occurs during the client is sending a request."
as we are 127.0.0.1 loopback address kind of server running..how the lte network is causing an issue here?

@hideki
Copy link
Author

hideki commented Sep 22, 2017

@NitzDKoder,

as we are 127.0.0.1 loopback address kind of server running..how the lte network is causing an issue here?

Honestly, I don't know. Even I am not sure if LTE causes this problem. It seems this is Android OS level issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants