No longer uses UID=1000 GID=1000 by default

[Frederick888]

Describe the problem

In 97fe6b1, some ENV arguments were moved into different scripts, however it seems default UID and GID were gone after this change.

As a result, now it starts with UID=0 and empty GID, and also changes ownership of data volume at

docker-minecraft-server/scripts/start

Lines 35 to 38 in 8ef3fe1

	if [[ $(stat -c "%u" /data) != "$UID" ]]; then
	log "Changing ownership of /data to $UID ..."
	chown -R ${runAsUser}:${runAsGroup} /data
	fi

Container definition

No response

Container logs

minecraft-mc-1  | [init] Changing ownership of /data to 0 ...
minecraft-mc-1  | [init] Running as uid=0 gid=1000 with /data as 'drwxrwxr-x 15 0 1000 4096 Mar 12 13:48 /data'
minecraft-mc-1  | [init] Resolved version given 1.18.2 into 1.18.2 and major version 1.18
minecraft-mc-1  | [init] Resolving type given FABRIC
minecraft-mc-1  | [init] Checking Fabric Launcher version information.
minecraft-mc-1  | [init] Checking Fabric Loader version information.
minecraft-mc-1  | [init] server.properties already created, skipping
minecraft-mc-1  | [init] Updating ops
minecraft-mc-1  | [init] FrederickZh already present in ops.json, skipping
minecraft-mc-1  | [init] Checking for JSON files.
minecraft-mc-1  | [init] Setting initial memory to 2G and max to 8G
minecraft-mc-1  | [init] Starting the Minecraft server...

[itzg]

I'm sorry for introducing this bug and thanks for reporting this. PR testing is going now and will merge+publish immediately after that.

[itzg]

Latest images now include the fix for this.

[Frederick888]

No need to apologise and thank you for maintaining this project :)