neoirc

150 Commits 14 Branches 0 Tags

3 Commits

Include renames

Author	SHA1	Message	Date
clawbot	ecf47c37f5	Start test servers on a port the system picks (closes #211 ) check / check (push) Waiting to run Details The internal/server and internal/ircserver tests picked a port by opening a listener, reading its port and closing it. They then started a server on that number. Another process could take the port in between, and a test failed with "address already in use". The IRC server tests now start the server on 127.0.0.1:0 and read the port back from its listener. The HTTP server tests start the app with port 0. The HTTP server now keeps the listener it binds, so the stop test sends its long poll to the port the app was given. Model: opus-5-5	2026-10-27 13:34:45 +01:00
clawbot	bbdf3a70af	Fail startup when the HTTP listener cannot bind (closes #167 ) check / check (push) Canceled after 0s Details The HTTP server ran ListenAndServe in a goroutine, so a failed bind was only logged and neoircd kept running without its HTTP API. The start hook now binds the listen address and returns the bind error, so neoircd exits 1 with an error naming the address. If serving the bound listener later ends with anything other than a normal shutdown, the server stops the app through fx.Shutdowner with exit code 1; the handler tests now pass it that Shutdowner. Tests check that a taken port fails startup, that a serve failure stops the app with exit code 1, and that a normal stop does not. The server tests no longer run in parallel: building the app writes the global viper instance. Model: opus-5-5	2026-10-21 01:14:24 +02:00
clawbot	292f1eb7a9	Serve the index page for an empty URL path instead of panicking (closes #149 ) check / check (push) Canceled after 0s Details setupSPA took the file name from request.URL.Path[1:], which panics when the path is empty, as it is for an absolute-form request line with no path ("GET http://host HTTP/1.1"). It now trims a leading "/" instead; an empty name fails to read, so the request gets the index page, as "/" already does. The new test in internal/server sends such a request and expects 200 and the index page. export_test.go gives it a Server whose router has only the web client route. Model: opus-5-5	2026-10-20 21:31:26 +02:00

Author

SHA1

Message

Date

clawbot

ecf47c37f5

Start test servers on a port the system picks (closes #211 )

check / check (push) Waiting to run

Details

The internal/server and internal/ircserver tests picked a port by
opening a listener, reading its port and closing it. They then started
a server on that number. Another process could take the port in
between, and a test failed with "address already in use".

The IRC server tests now start the server on 127.0.0.1:0 and read the
port back from its listener. The HTTP server tests start the app with
port 0. The HTTP server now keeps the listener it binds, so the stop
test sends its long poll to the port the app was given.

Model: opus-5-5

2026-10-27 13:34:45 +01:00

clawbot

bbdf3a70af

Fail startup when the HTTP listener cannot bind (closes #167 )

check / check (push) Canceled after 0s

Details

The HTTP server ran ListenAndServe in a goroutine, so a failed bind
was only logged and neoircd kept running without its HTTP API.

The start hook now binds the listen address and returns the bind
error, so neoircd exits 1 with an error naming the address. If serving
the bound listener later ends with anything other than a normal
shutdown, the server stops the app through fx.Shutdowner with exit
code 1; the handler tests now pass it that Shutdowner.

Tests check that a taken port fails startup, that a serve failure
stops the app with exit code 1, and that a normal stop does not. The
server tests no longer run in parallel: building the app writes the
global viper instance.

Model: opus-5-5

2026-10-21 01:14:24 +02:00

clawbot

292f1eb7a9

Serve the index page for an empty URL path instead of panicking (closes #149 )

check / check (push) Canceled after 0s

Details

setupSPA took the file name from request.URL.Path[1:], which panics
when the path is empty, as it is for an absolute-form request line
with no path ("GET http://host HTTP/1.1"). It now trims a leading "/"
instead; an empty name fails to read, so the request gets the index
page, as "/" already does.

The new test in internal/server sends such a request and expects 200
and the index page. export_test.go gives it a Server whose router has
only the web client route.

Model: opus-5-5

2026-10-20 21:31:26 +02:00