Почему не создается пользователь системы?

Question

[Twelfth Doctor]

Здравствуйте! Подскажите, почему может не создаваться пользователь системы?
Вот функция

protected function create(array $data)
    {
//создаем пользователя в базе данных
        $user= User::create([
            'name' => $data['name'],
            'email' => $data['email'],
            'password' => bcrypt($data['password']),
            'role'=>'customer',
        ]);

//получаем его ID
        $id = $user->id;
        $pass = $data['password'];
//создаем пользователя в Linux
        shell_exec("useradd -d /home/u$id/ u$id");
        shell_exec("echo u$id:$pass | chpasswd");
//возвратить экземпляр класса User
      return $user;
    }

Проблема в том, что пользователь системы Linux почему-то не создается. Для пользователя, от которого работает веб-приложение в /etc/sudoers написал:
user ALL=(ALL) NOPASSWD:ALL
(к сожалению не знаю, как дать доступ только к отдельным командам).

Comments

[D3lphi]

А зачем вы "получаете объект только что созданного пользователя", если метод create() сам по себе возвращает этот самый объект? Вы двойную работу выполняете.

[Twelfth Doctor]

D3lphi: Спасибо за подсказку, исправил код.

[Алексей Черемисин]

Я не увидел в коде вызов sudo! Хотя вы как раз его то и настраивали

Answer 1

[Назар Мокринский]

shell_exec возвращает вывод запущенной команды как результат работы функции.
Для начала сделайте var_dump(shell_exec()) и посмотрите в чём дело, на что ругается команда (если ругается).
А вообще я бы рекомендовал exec() и проверять код статуса результата работы команды, ибо вторую команду не нужно же выполнять если первая завалилась с ошибкой.

Comments

[Twelfth Doctor]

var_dump(shell_exec()) пустой

[Назар Мокринский]

verdex: Тогда, скорее всего, пользователь создался. Код статуса какой у выполненной команды?

[Twelfth Doctor]

А как его получить?

[Назар Мокринский]

verdex: Используйте третий параметр команды exec(). Ну и sudo не забудьте, ибо в /etc/sudoers вы пользователя прописали, а команду вызываете без sudo.

[Twelfth Doctor]

Назар Мокринский: Сделал такую проверку:

$pass = $data['password'];
        $output1 = '';
        $output2 = '';
        $code1 = 0;
       $code2 = 0;
      $cruser=  exec("sudo useradd -d /home/u$id/ u$id" , $output1,$code1);
      if ($code1>0) {
file_put_contents("/myapp/logs/users.log" , "Create successful" , FILE_APPEND);
}
      else {
file_put_contents("/myapp/logs/users.log" , "Create error occured",FILE_APPEND);
}
        $chpass =  exec("sudo echo u$id:$pass | chpasswd" , $output2,$code2);
    if ($code2>0) {
file_put_contents("/myapp/logs/users.log" , "Chpass successful" , FILE_APPEND);
}
      else {
file_put_contents("/myapp/logs/users.log" , "Chpass error occured" , FILE_APPEND);
}

Содержимое файла /myapp/logs/user.log:

Create error occured
Chpass successful

[Назар Мокринский]

verdex: Распечатайте какой именно код получили. man usedadd в конце содержит список кодов и что они значат.

[Twelfth Doctor]

Назар Мокринский: Получил код 0, что означает, что все хорошо. Странно

$output1 = '';
        $output2 = '';
        $code1 = 0;
       $code2 = 0;
      $cruser=  exec("sudo useradd -d /home/u$id/ u$id" , $output1,$code1);
      if ($code1>0) {
          file_put_contents("/myapp/logs/users.log" , "Create successful" , FILE_APPEND);
          
      }
      else {
          file_put_contents("/myapp/logs/users.log" , "Create error occured. Code $code1",FILE_APPEND);
          
      }
        $chpass =  exec("sudo echo u$id:$pass | chpasswd" , $output2,$code2);
    if ($code2>0) {
        file_put_contents("/myapp/logs/users.log" , "Chpass successful" , FILE_APPEND);
        
    }
      else {
          file_put_contents("/myapp/logs/users.log" , "Chpass error occured. Code $code2" , FILE_APPEND);
      }

Лог:

Code 0
Chpass successful

Из доки:

EXIT VALUES
       The useradd command exits with the following values:
       0
           success

[Назар Мокринский]

verdex: У вас же была ошибка в создании пользователя, $code1 не должен быть 0.

[Twelfth Doctor]

Назар Мокринский: Я неправильно условие в коде поставил
Так правильно будет

if ($code1>0) {
                 file_put_contents("/myapp/logs/users.log" , "Create error occured. Code $code1",FILE_APPEND);
          
      }
      else {
             file_put_contents("/myapp/logs/users.log" , "Create successful" , FILE_APPEND);
      }

[Назар Мокринский]

verdex: Да, так будет верно

[Twelfth Doctor]

Теперь в user.log все правильно:

Create successful
Chpass error occured. Code 1

[Twelfth Doctor]

Только я не вижу домашнюю папку нового пользователя по адресу /home/u%id%/

[Twelfth Doctor]

А в /etc/passwd они есть

[Назар Мокринский]

verdex: Всё верно, папка и не должна создаваться. Домашнюю папку при создании пользователя создает adduser

[Назар Мокринский]

verdex: Печально у verdex: По поводу пароля, попробуйте перевод на новую строку после пароля добавить \n

[Twelfth Doctor]

И chpasswd возвращает 1, у пользователя нет пароля. Коды ошибок в man не нашел

[Twelfth Doctor]

Назар Мокринский: Не помогло

[Назар Мокринский]

verdex: Ха-ха:) Вам не sudo echo, а sudo chpasswd нужно (и не нужен перевод строки).

[Twelfth Doctor]

Теперь работает, но при попытке войти под пользователем (su u19, к примеру) возникает ошибка
su: Authentication failure

[Назар Мокринский]

verdex: Давайте не задавать кучу вопросов в одном. Поищите решение, если не найдете то задавайте новый вопрос, можете меня там упомянуть потом. Так будет удобнее всем, в том числе тем, кто будет искать вопросы и ответы после вам.