search

DOMjudge / domjudge-packaging

DOMjudge packaging for (Linux) distributions and live image
31 stars 37 forks source link

Check for missing '/' in DOMSERVER_BASEURL #133

Closed vmcj closed 9 months ago

vmcj commented 1 year ago

In the old containers we don't check for the '/', we should verify if we do in the latest containers

[Jul 29 16:56:04.624] judgedaemon[44]: Judge started on judgedaemon-0-0 [DOMjudge/7.3.3] [Jul 29 16:56:04.629] judgedaemon[44]: executing chroot script: 'chroot-startstop.sh check' [Jul 29 16:56:04.630] judgedaemon[44]: Registering judgehost on endpoint default: http://localhost:22345api/v4 [Jul 29 16:56:04.632] judgedaemon[44]: warning: Error while executing curl POST to url http://localhost:22345api/v4/judgehosts: [Jul 29 16:56:04.632] judgedaemon[44]: Registering judgehost on endpoint default failed. [Jul 29 16:56:04.632] judgedaemon[44]: error: Error while executing curl GET to url http://localhost:22345api/v4/config?:

docker run -it --privileged -v /sys/fs/cgroup:/sys/fs/cgroup:ro --name judgehost-0 --hostname judgedaemon-0 -e DOMSERVER_BASEURL=http://localhost:22345/ -e JUDGEDAEMON_PASSWORD=randomis4 -e DAEMON_ID=0 domjudge/judgehost:7.3.3

vmcj commented 9 months ago

This seems to work for every patch release in 8.x.latest