convert to "rootless"

Dockerfile

@@ -1,32 +1,36 @@
 FROM ubuntu:noble
 
+ARG UID=1000
+ARG GID=1000
+
 ENV DEBIAN_FRONTEND=noninteractive
 
+SHELL ["/bin/bash", "-o", "pipefail", "-c"]
+
 # install prerequisites
-RUN apt update \
-    && apt install -y --no-install-recommends \
-    curl ca-certificates \
+RUN apt-get update \
+  && apt-get install -y --no-install-recommends \
+  curl \
+  ca-certificates \
   jq \
   libxml2-utils \
-    && apt autoremove \
-    && apt clean \
-    && rm -rf /var/lib/apt/lists/*
+  sudo && \
+  apt-get autoremove && apt-get clean && \
+  rm -rf /var/lib/apt/lists/*
 
 # Remove default ubuntu user
-RUN userdel -r ubuntu \
-    && useradd --create-home -u 1000 minecraft \
-    && mkdir -p /data \
-    && chown -R 1000:1000 /data
+RUN userdel -r ubuntu && \
+  useradd --create-home -u 1000 minecraft && \
+  echo 'minecraft ALL=(ALL) NOPASSWD: ALL' >> /etc/sudoers && \
+  mkdir -p /data && \
+  chown -R ${UID}:${GID} /data
 
-USER minecraft
-
-RUN curl -sL https://github.com/Jabba-Team/jabba/raw/main/install.sh | bash \
-    && . ~/.jabba/jabba.sh
-
-USER root
+RUN su -c "curl -sL https://github.com/Jabba-Team/jabba/raw/main/install.sh | bash && . ~/.jabba/jabba.sh" minecraft
 
 COPY ./scripts/init /
 COPY ./scripts/mc /usr/local/bin 
 RUN chmod +x /init && chmod +x /usr/local/bin/mc
 
+USER minecraft
+
 CMD ["/init"]

scripts/init

@@ -3,14 +3,16 @@
 PUID="${PUID:-1000}"
 PGID="${PGID:-1000}"
 
-setperms(){
+setperms() {
   echo "Setting permissions to UID:${PUID} and GID:${PGID}"
-    usermod -u ${PUID} minecraft > /dev/null 2>&1
-    groupmod -g ${PGID} minecraft > /dev/null 2>&1
-    chown -R ${PUID}:${PGID} /data > /dev/null 2>&1
+  sudo usermod -u ${PUID} minecraft >/dev/null 2>&1
+  sudo groupmod -g ${PGID} minecraft >/dev/null 2>&1
+  sudo chown -R ${PUID}:${PGID} /data >/dev/null 2>&1
+  echo "DONE"
   sleep 2
 }
 
 setperms
 
-su -c "mc" minecraft
+mc
+

scripts/mc

@@ -1,14 +1,14 @@
 #!/bin/env bash
 
-MC_LOADER="${MC_LOADER:="paper"}"
-MC_VERSION=${MC_VERSION:="1.21.4"}
+MC_LOADER="${MC_LOADER:-"paper"}"
+MC_VERSION=${MC_VERSION:-"1.21.4"}
 
-JAVA_VERSION=${JAVA_VERSION:="temurin@21"}
-JAR=${JAR:="server.jar"}
-MEM=${MEMORY:=2G}
-ADD_ARGS=${ADD_ARGS:=""}
+JAVA_VERSION=${JAVA_VERSION:-"temurin@21"}
+JAR=${JAR:-"server.jar"}
+MEM=${MEMORY:-2G}
+ADD_ARGS=${ADD_ARGS:-""}
 
-RESTART=${RESTART:="true"}
+RESTART=${RESTART:-"true"}
 
 crash() {
   sleep 5
@@ -22,7 +22,7 @@ setEula() {
   fi
 }
 
-javaCheck(){
+javaCheck() {
   JAVA_HOME=/data/java/${JAVA_VERSION}
   if [[ -f "$HOME/.jabba/jabba.sh" ]]; then
     source $HOME/.jabba/jabba.sh
@@ -62,7 +62,7 @@ getPaper() {
   fi
 }
 
-getFabric(){
+getFabric() {
   LATEST_BUILD=$(curl -s https://meta.fabricmc.net/v2/versions/loader/${MC_VERSION} | jq -s '.[] .[0] .loader .version' | tr -d '"')
   LATEST_INSTALLER=$(curl -s https://maven.fabricmc.net/net/fabricmc/fabric-installer/maven-metadata.xml | xmllint --xpath "/metadata/versioning/latest/text()" -)
   DL_URL=https://meta.fabricmc.net/v2/versions/loader/${MC_VERSION}/${LATEST_BUILD}/${LATEST_INSTALLER}/server/jar
@@ -75,7 +75,7 @@ getFabric(){
 
 }
 
-getForge(){
+getForge() {
   LATEST_BUILD=$(curl -s https://files.minecraftforge.net/net/minecraftforge/forge/promotions_slim.json | jq -r ".promos[\"${MC_VERSION}-latest\"]")
   DL_BUILD=${MC_VERSION}-${LATEST_BUILD}
   DL_URL=https://maven.minecraftforge.net/net/minecraftforge/forge/$DL_BUILD/forge-$DL_BUILD-installer.jar
@@ -98,7 +98,7 @@ getForge(){
   fi
 }
 
-serverSelect(){
+serverSelect() {
   mkdir -p /data/server
   cd /data/server
   if [[ ! -f server.jar ]]; then
@@ -123,7 +123,7 @@ serverSelect(){
   fi
 }
 
-buildCommand(){
+buildCommand() {
   JAVA_ARGS="-Xms${MEM} -Xmx${MEM}"
   if [[ -z ${ADD_ARGS+x} ]]; then
     RUN_STRING="${ADD_ARGS} ${JAVA_ARGS} -jar ${JAR} nogui"
@@ -132,7 +132,7 @@ buildCommand(){
   fi
 }
 
-runServer(){
+runServer() {
   setEula
   while true; do
     if [[ -f $JAR ]]; then