Merge pull request #2218 from cdr/whoami-c324

Remove unnecessary whoami

Author: nhooyr

README.md

@@ -43,11 +43,12 @@ The install script will print out how to run and start using code-server.
 Docs on the install script, manual installation and docker image are at [./doc/install.md](./doc/install.md).
 
 ### Alpha Program 🐣
+
 We're working on a cloud platform to make deploying and managing code-server easier. If you don't want to worry about
 
-* TLS
-* Authentication
-* Port Forwarding
+- TLS
+- Authentication
+- Port Forwarding
 
 consider [joining our alpha program](https://codercom.typeform.com/to/U4IKyv0W).
 

ci/release-image/Dockerfile

@@ -43,5 +43,6 @@ EXPOSE 8080
 # the uid will remain the same. note: only relevant if -u isn't passed to
 # docker-run.
 USER 1000
+ENV USER=coder
 WORKDIR /home/coder
 ENTRYPOINT ["/usr/bin/entrypoint.sh", "--bind-addr", "0.0.0.0:8080", "."]

ci/release-image/entrypoint.sh

@@ -1,10 +1,6 @@
 #!/bin/sh
 set -eu
 
-# This isn't set by default.
-USER="$(whoami)"
-export USER
-
 if [ "${DOCKER_USER-}" ] && [ "$DOCKER_USER" != "$USER" ]; then
   echo "$DOCKER_USER ALL=(ALL) NOPASSWD:ALL" | sudo tee -a /etc/sudoers.d/nopasswd > /dev/null
   # Unfortunately we cannot change $HOME as we cannot move any bind mounts