TurboVNC / turbovnc

Main TurboVNC repository
https://TurboVNC.org
GNU General Public License v2.0
746 stars 136 forks source link

java.lang.IllegalArgumentException: Value too long #410

Closed Patrick-Poitras closed 2 months ago

Patrick-Poitras commented 2 months ago

I'm getting this error when trying to connect to a new server.

java.lang.IllegalArgumentException: Value too long: 192.168.45.16:30906,192.168.45.16:31840,...

With ... being the entirety of the values I've used to connect before, separated by commas. Seems like Java is balking at how many connections I've had before when trying to parse the history.

Details: Windows 10, tested on 3.0.3

Patrick-Poitras commented 2 months ago

Was on 3.0.3, upgraded to 3.1.1 and still have the same error.

Patrick-Poitras commented 2 months ago

image

Patrick-Poitras commented 2 months ago

Workaround found: I went into the settings and cleared the connection history. This fixed the problem, at the expense of clearing my history.

dcommander commented 2 months ago

That's really weird. So the viewer appeared to be trying to use a comma-separated list of previous connections as the server name? I can't imagine why it would have done that. Any further information you can provide that may help me reproduce the issue would be appreciated.

dcommander commented 2 months ago

Also, assuming you got a stack trace when the failure occurred, can you post the stack trace?

Patrick-Poitras commented 2 months ago

I got nothing more. The error message was a popup window from the viewer that chained the addresses I had previously connected to, separated by commas.

I am no longer able to reproduce due to clearing the history, but that also fixed the problem, so we've got a workaround. My guess is that the dropdown menu is serialized into a comma separated list at some point and then sent through some API that chokes on the amount of data.