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

server crash on client disconnect #317

Open
bazola opened this issue Jul 2, 2015 · 4 comments
Open

server crash on client disconnect #317

bazola opened this issue Jul 2, 2015 · 4 comments
Milestone

Comments

@bazola
Copy link
Contributor

bazola commented Jul 2, 2015

It seems like sometimes disconnecting from the server causes it to error in such a way that it can no longer send messages to clients

@Zomis
Copy link
Member

Zomis commented Jul 2, 2015

Whoops... any stack trace on this? Is it with the HTML5 LibGDX client or does it happen with other clients as well? Could it be #191 that hasn't been fully fixed?

@Zomis
Copy link
Member

Zomis commented Jul 2, 2015

From the logfile I received I find two problems:

  • Stack Overflow Error
  • End Of File Exception

Both of these should be addressed, separately.

Stack Overflow Error

The Stack Overflow error is caused by this:

(...)
at com.cardshifter.core.game.ServerGame.send(ServerGame.java:95)
at com.cardshifter.server.model.Server.lambda$onDisconnected$3(Server.java:171)
(...)
at com.cardshifter.server.model.Server.onDisconnected(Server.java:171)
at com.cardshifter.api.ClientIO.disconnected(ClientIO.java:57)
at com.cardshifter.server.clients.ClientWebSocket.onSendToClient(ClientWebSocket.java:42)
at com.cardshifter.api.ClientIO.sendToClient(ClientIO.java:27)
at com.cardshifter.core.game.ServerGame.lambda$send$22(ServerGame.java:95)
(...)

These things seem to be repeated.

EOFException

End Of File Exception is likely caused by incompatibility between LibGDX code and the parent project. I'd recommend doing gradlew clean install on the real Cardshifter project, and then clean the LibGDX project to make it update the dependency for cardshifter-api.

@Zomis Zomis added this to the 0.6.5 milestone Jul 23, 2015
@jacwah
Copy link
Contributor

jacwah commented Aug 23, 2015

Is the stack overflow solved by #352? If not, how can I reproduce it?

@Zomis
Copy link
Member

Zomis commented Aug 23, 2015

Stack Overflow error has been fixed, yes. As for EOFException, I think that is not a global bug, only a local one (because of incompatible versions between the gdx project and main Cardshifter repo).

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

No branches or pull requests

3 participants