Verwenden von chown $ USER: $ USER im Bash-Skript

In einem kleinen Bash-Skript, das ich ausführe, versuche ich, ein neues Verzeichnis zu erstellen, das erstellt wird. Ich habe hinzugefügt:

sudo chown $USER:$USER /var/www/$sitename
sudo chmod 775 /var/www/$sitename

nach der Zeile, in der ich mkdir ( sudo mkdir /var/www/$sitename).

Aus irgendeinem Grund wird das Chown nicht ausgeführt. Ich kann es manuell ausführen, aber wenn es in die Datei geschrieben wird, funktioniert es nicht. Ich habe festgestellt, dass "chown" nicht in der gleichen Farbe wie "mkdir" und "chmod" hervorgehoben ist, aber ich kann mein Problem nicht herausfinden.

Warum funktioniert Chown hier nicht?

Ist es ein Problem mit $USER:$USER?

BEARBEITEN Hier ist das vollständige Skript. Wie würde ich die Datei an den Nicht-Root-Benutzer weiterleiten, der das Skript ausgeführt hat?

#!/bin/sh
#!/bin/bash
# New Site

cd /etc/apache2/sites-available/
echo "New site name (test.my):"
read sitename
echo "<VirtualHost *:80>

        ServerAdmin admin@$sitename

    ServerName $sitename

        ServerAlias $sitename

    DocumentRoot /var/www/$sitename

        <Directory />
                Options FollowSymLinks
                AllowOverride All
        </Directory>
        <Directory /var/www/$sitename>
                Options Indexes FollowSymLinks MultiViews
                AllowOverride All
                Order allow,deny
                allow from all
        </Directory>

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined


</VirtualHost>" > $sitename.conf
sudo mkdir /var/www/$sitename
sudo chown $USER:$USER /var/www/$sitename
echo USER is $USER
sudo chmod 775 /var/www/$sitename
sudo a2ensite $sitename.conf
sudo apachectl restart
echo "New site created"

Wenn aus irgendeinem Grund $USERnicht festgelegt ist, können Sie den idBefehl verwenden, um die Identität des tatsächlichen Benutzers abzurufen. Wenn Sie die $USERVariable zum ersten Mal verwenden , können Sie mithilfe der Shell-Erweiterung einen Standardwert angeben. Ändern Sie die chownZeile in Ihrem Skript in:

sudo chown ${USER:=$(/usr/bin/id -run)}:$USER /var/www/$sitename

Wenn dies USERleer ist oder nicht gesetzt ist, setzt bash die USERVariable auf die Ausgabe von/usr/bin/id -run

Wenn ich mein Skript sudodamit aufrufe, wird es $USERauf root gesetzt.

$ sudo ./myscript.sh

Ich habe das versucht, chown ${USER:=$(/usr/bin/id -run)}:$USER /var/www/$sitenameaber es würde immer noch root zurückgeben.

Wenn ich whomit verwendet awkhabe, konnte ich den aktuellen Benutzer ermitteln, mit dem das Skript aufgerufen wurde sudo.

currentuser=$(who | awk '{print $1}')}
chown -R $currentuser:$currentuser /var/www/$sitename`

Warum lesen Sie nicht eine Benutzername-Variable, um das Problem zu vereinfachen und da Sie die Variable Site-Name erhalten?

Damit stellen Sie sicher, dass die Skriptausführung nicht von den Umgebungsvariablen abhängt, die für die Ausführung des Skripts verfügbar gemacht werden.

Ich denke, es gibt nur einen winzigen Fehler. sudo öffnet eine neue Shell zum Ausführen des Befehls und nach sudo ist der Benutzer root. Vielleicht sollten Sie so etwas verwenden:

MYUSER = $ USER

sudo chown $ MYUSER: $ MYUSER

da ich denke MYUSER ist nicht systemspezifisch überschrieben und soll funktionieren.