chce_usera.sh - Githubissues

stefopl commented 3 years ago

syntax error fixed, redundant code fragments removed, user shell changed, noobs symlink added in new user home directory

stefopl commented 3 years ago

Czy są jakieś uwagi do wprowadzonych przeze mnie zmian?

Jeszcze dokładniej opiszę co zrobiłem:

funkcja _check_if_user_blank() nie miała } przez co cały skrypt nie działał
fragmenty skryptu, które wykonywane są za pomocą funkcji były powtórzone poza funkcjami
wprowadzone/wygenerowane hasła 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
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
w katalogu nowego użytkownika tworzony jest teraz symlink do noobs uznałem, że może to być przydatne

loper commented 3 years ago

Hej, nie mam uwag - testowałem i działa, ale Unknow pisał o kilku błędach, więc wolę jemu zostawić kwestię akceptacji.

eerozeteen commented 3 years ago

Po wysłaniu mojego PR #83 , zauważyłem że są już prace nad plikiem i poprawka jest zawarta.

Osobiście testowałem zmiany @stefopl i mogę potwierdzić działanie kodu.

unkn0w / noobs