[Open] Connection Lost

[RoiEX]

@ssoloff I'd appreciate if you could setup a branch for this.
If you could do this within the next 7 hours I'd try submitting a PR this evening

[RoiEX]

Also if we're going to create a hack anyways, I'll just going to increase the log level to error, so that @VanBattlesick doesn't need to set something up, even if it might open too much windows

[ssoloff]

@RoiEX I created the branch spike/topic-1104-connection-lost-debugging in my fork for your use. I sent you a collaborator invite to my fork so you don't have to wait for me to merge PRs. Please be careful while it's in effect. (GitHub collaboration seems to be an all-or-nothing affair--I couldn't find a way to limit collaboration to specific branches.)

Pushing to the above branch will generate a new GitHub release here: https://github.com/ssoloff/triplea-game-triplea/releases. Once you post a link to a testable release in this thread, I'll clean up any previous releases that were generated.

[RoiEX]

@VanBattlesick So I modified the code in a way that should force it to display an error message even for "harmless errors" i.e. normal disconnects.
Please install the modified version you can find here, and try to reproduce your steps.
This time you should get an error dialog with a "Show Details" button you can click.
This will open an error console window, which could perhaps be out-of focus and behind the main window, so check your opened windows if nothing seems to appear for you.
If you finally found the console, please post the content here so we can proceed to help you with your issue.

Thanks @ssoloff for providing the infrastructure

[VanBattlesick]

[RoiEX]

@VanBattlesick
Ok, that definitely helps, but not as much as I thought.
I googled this error message and it seems to be a known problem that appears to occur to a lot of Minecraft players.
I didn't find a definite answer on how to fix this though, so I can't guarantee they will work for you.

Option 1

If you have java installed and can access the java configuration via the control panel which looks like this, except in your language:
If you switch to the "Advanced" Tab and scroll down to check this checkbox this is supposed to help some people

I don't really kow why this is supposed to work, but I just wanted to note it down.

Option 2

This one is a little bit more complicated and not that straightforward, but is more promising IMO.
Basically redoing your local Firewall rules to allow java to communicate with the internet:
Try searching for this Firewall menu in your local control panel: (Unfortunately I can't really tell you the english terms for the individual setting windows, I don't have an english windows installation)

And check if theres an entry for java in the outgoing or the incoming rules section.
If there's an entry try removing it.
If you then start TripleA again it should ask you if you want to allow java in certain network types.
Try selecting all checkboxes (public and private) and see if that helps.

@ssoloff We should really modify the NioReader and NioWriter classes to actually log those IOExceptions with a higher level, this one shouldn't be treated as a "normal disconnect"

[VanBattlesick]

@RoiEX Playing tonight and it went good for a little while and then started lagging. It froze and jumped ahead again once my opponent typed in chat screen. Then I got disconnected.

Also lost connection to server

[RoiEX]

Which "Option" did you try?

[VanBattlesick]

I tried option one. I will try the other one next.

[ssoloff]

@RoiEX I'm cleaning up my fork. Do you still need the spike/topic-1104-connection-lost-debugging branch? If you don't need it now but think you'll need it again in the future, it's pretty simple to re-create.

[RoiEX]

Not anymore. We have all the debug information we can get for now, so we should be fine

[ssoloff]

@RoiEX said in Connection Lost:

I finally found the post where I proposed the original solution.
I modified the properties file to be

handlers=games.strategy.debug.ConsoleHandler

games.strategy.net.nio.NioWriter.level=FINEST
games.strategy.net.nio.NioReader.level=FINEST

but unfortunately I got a

Can't load log handler "games.strategy.debug.ConsoleHandler"
java.lang.InstantiationException: games.strategy.debug.ConsoleHandler

Because the ConsoleHandler class no longer has a default constructor.
So we need a code change either way unfortunately, adding a default constructor might be the fastest way nevertheless though.

I was playing around with the logging system recently. I discovered that specifying the games.strategy.debug.ConsoleHandler in the configuration file appears to be unnecessary since it is registered in code during application startup. That means you can remove the handlers property to avoid the InstantiationException, but the logger level configuration lines continue to be processed as expected.

In summary, even though our ConsoleHandler doesn't provide a default constructor, the approach you proposed above should continue to be a viable solution if we ever need to provide a custom logger configuration (at least with respect to log levels).

[VanBattlesick]

Hi,
Sorry I have not updated on what happened while playing but I got so fed up that I just said "Forget it!". I find myself still wanting to play at times however and now wonder if their has been any progress with a fix for my problem. What else can I do to make this work?
VanB.........

[RoiEX]

@VanBattlesick You wanted to try option 2

[James Fisher]

This post is deleted!

[liqik]

I tried it again with a different map and it still showed the same.

I only realised recently that https://speedtestfast.com/verizon-speed-test/ is a speed test site that shows you accurate data on the streaming speeds you can get on Netflix, as it uses the same CDN servers used by Netflix video streams.

[RoiEX]

Unfortunately there's not really a lot we can do about it

However if that helps, @LaFayette is currently putting a lot of effort into rewriting parts of the networking code, so maybe that'll fix this problem as a nice side-effect in the future.
No guarantees though.