mbround18 / valheim-docker

Valheim Docker powered by Odin. The Valheim dedicated gameserver manager which is designed with resiliency in mind by providing automatic updates, world backup support, and a user friendly cli interface.

https://hub.docker.com/r/mbround18/valheim

BSD 3-Clause "New" or "Revised" License

762 stars 81 forks source link

fix `scheduled_restart` not running due to invalid user #892

Closed nbusseneau closed 1 month ago

nbusseneau commented 2 months ago

Description

Contributions

USER is not a known environment variable when run via Cron. We switch to using whoami instead similar to what is done in auto_update and auto_backup.

Checklist

[ ] I added one or multiple labels which best describes this PR.
[x] I have tested the changes locally.
[ ] This PR has a reviewer on it.
[x] I have validated my changes in a docker container and on Ubuntu. (Only needed for Odin or Docker Changes)

mbround18 commented 1 month ago

Were you able to test locally and this fixes the issue?

nbusseneau commented 1 month ago

Yep, as ticked in the form ;)