chiark / gitweb /
Improve adduser calls:
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Mon, 9 Dec 2013 16:43:53 +0000 (16:43 +0000)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Mon, 9 Dec 2013 16:43:53 +0000 (16:43 +0000)
   - no longer guard with calls to id, as adduser --quiet is idempotent
   - pass --shell /bin/false and --no-create-home

debian/changelog
debian/userv-dyndns.postinst
debian/userv-git-daemon.postinst

index bcd63e7..93ae89f 100644 (file)
@@ -8,6 +8,9 @@ userv-utils (0.6.0~~iwj3) unstable; urgency=low
   * Add debhelper token to postinst scripts.
   * Change default user for git-daemon to userv-git and
     default user for dyndns to userv-dyndns.
+  * Improve adduser calls:
+     - no longer guard with calls to id, as adduser --quiet is idempotent
+     - pass --shell /bin/false and --no-create-home
 
  --
 
index da9acd7..a6779f9 100755 (executable)
@@ -23,7 +23,8 @@ set -e
 #DEBHELPER#
 
 user="$(sed -n '/^[a-z]/{p;q;}' /etc/userv/dyndns-service-users)"
-if id $user >/dev/null 2>&1; then exit 0; fi
 
-adduser --system --group --gecos 'userv dyndns' \
-       --home /etc/userv
+if [ "$user" ]; then
+       adduser --system --quiet --group --gecos 'userv dyndns' \
+               --home /etc/userv --no-create-home --shell /bin/false $user
+fi
index 5ed3b4e..5ccaa88 100755 (executable)
@@ -30,8 +30,6 @@ if test -f $defaults; then . $defaults; fi
 #DEBHELPER#
 
 if [ "$GITDUSER" ]; then
-       if id $GITDUSER >/dev/null 2>&1; then exit 0; fi
-
-       adduser --system --group --gecos 'userv git daemon' \
-               --home /etc/userv $GITDUSER
+       adduser --system --quiet --group --gecos 'userv git daemon' \
+               --home /etc/userv --no-create-home --shell /bin/false $GITDUSER
 fi