Usando chown $ USER: $ USER all'interno dello script bash

In un piccolo script bash che sto eseguendo, sto tentando di creare una nuova directory creata. Ho aggiunto:

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

dopo la linea in cui mkdir ( sudo mkdir /var/www/$sitename).

Per qualche motivo il chown non sta eseguendo. Posso eseguirlo manualmente ma quando scritto nel file non funziona. Ho notato che "chown" non è evidenziato nello stesso colore di "mkdir" e "chmod" ma non riesco a capire il mio problema.

Perché il chown non funziona qui?

È un problema con $USER:$USER?

EDIT Ecco lo script completo. Come chown il file a qualsiasi utente non root eseguisse lo script?

#!/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"

Se, per qualche motivo, $USERnon è impostato, è possibile utilizzare il idcomando per ottenere l'identità dell'utente reale. Quindi la prima volta che usi la $USERvariabile, puoi usare l'espansione della shell per fornire un valore predefinito. Cambia la chownriga nel tuo script in:

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

Se USERè vuoto o non impostato quando viene eseguito, bash imposterà la USERvariabile sull'output di/usr/bin/id -run

Quando chiamo il mio script con sudoesso sarebbe impostato $USERsu root.

$ sudo ./myscript.sh

Ho provato il chown ${USER:=$(/usr/bin/id -run)}:$USER /var/www/$sitenamema avrebbe comunque restituito root.

Ho scoperto che se usato whocon awkero in grado di ottenere l'utente corrente che ha chiamato lo script con sudo.

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

Per semplificare il problema e poiché stai ottenendo la variabile sitename, perché non leggi una variabile username?

Con ciò ti assicureresti che l'esecuzione dello script non dipenda dalle variabili ambientali rese disponibili nel modo in cui lo script viene eseguito.

C'è solo un piccolo difetto, penso. sudo apre una nuova shell per l'esecuzione del comando e dopo sudo l'utente è root. Quindi forse dovresti usare qualcosa del genere:

MYUSER = $ USER

sudo chown $ MYUSER: $ MYUSER

poiché penso che MYUSER non sia sovrascritto sistemicamente e funzionerà.