unkn0w
commented
3 years ago Twój commit zabiera bit wykonywalności ze wszystkich innych skryptów, modyfikując zbędnie 16 dodatkowych plików.
Nie wiem także dlaczego powłoka ustalana jest przez dodatkowe wywołanie chsh zamiast przez odpowiedni parametr w useradd.
Tworzony jest także zbyteczny plik .sudo_as_admin_successful, który do niczego nie jest nigdzie używany. Twoja zmiana wyrzuca także pytanie użytkownika o hasło jak i o username.
Modyfikacji w takiej formie nie możemy niestety przyjąć.
Skrypt nie działał prawidłowo. Posiadł błąd składni, część kodu zawartego w funkcjach była powtórzona niżej, hasła wygenerowanie i wpisane nie działały, ponieważ useradd -p przyjmuje zaszyfrowane hasło nowego konta a w skrypcie podane było bezpośrednio to, co wygenerowanie/wpisane.
Dodatkowo zmieniłem powłokę usera z /bin/sh/ na /bin/bash/ dzięki czemu będzie działać wygodniej a przed znakiem zachęty pojawi się user@host:/katalog.
Dodałem symlink do noobs w katalogu domowym nowego użytkownika.