Desktop: a server without a port gives "invalid socket address"

littlefox commented

2026-10-17 17:10:13 +00:00

Owner

User report: typing kom.lysator.liu.se in the server field and logging in fails with "invalid socket address". The desktop app (and kom-relay) pass the text straight to TcpStream::connect, which needs host:port. A bare host should get Protocol A's port 4894, and a bare IPv6 address its brackets.

User report: typing `kom.lysator.liu.se` in the server field and logging in fails with "invalid socket address". The desktop app (and kom-relay) pass the text straight to TcpStream::connect, which needs host:port. A bare host should get Protocol A's port 4894, and a bare IPv6 address its brackets.

littlefox added the

area/desktop

kind/bug

labels

2026-10-17 17:10:13 +00:00

littlefox referenced this issue from a pull request that will close it,

2026-10-17 17:11:42 +00:00

Accept a server without a port (default 4894) #48

littlefox referenced this issue from a commit

2026-10-17 17:11:43 +00:00

Accept a server without a port: default to Protocol A's 4894

littlefox commented

2026-10-17 17:13:20 +00:00

Author

Owner

Also in #48, per the user: the desktop app's default server is now kom.lysator.liu.se (D-029). NEOKOM_SERVER still overrides it; the desktop smoke test sets it to its devserver.

littlefox referenced this issue from a commit

2026-10-17 17:13:20 +00:00

Desktop: default to kom.lysator.liu.se (D-029)

littlefox closed this issue

2026-10-17 17:26:48 +00:00

littlefox commented

2026-10-17 17:26:48 +00:00

Author

Owner

Merged in #48 (CI run #78 green). Verified: unit tests for the address forms; new integration test bare_host_uses_default_port against the devserver by IP and by name; once, logged out, against kom.lysator.liu.se (plain after NotTls, GetTime ok). Desktop default server is kom.lysator.liu.se (D-029).

Rows
Columns

Desktop: a server without a port gives "invalid socket address" #47