Little fixs

debian/postinst

@@ -1,9 +1,16 @@
 #!/bin/sh
 set -e
 
+# Hide root warning for pip
+export PIP_ROOT_USER_ACTION=ignore
+export PIP_BREAK_SYSTEM_PACKAGES=1
+
 case "$1" in
     install|configure)
-        echo "No work to do..."
+        echo "Install Python requirements"
+
+        /usr/bin/python3 -m pip install pwinput
+
         exit 0
         ;;
     upgrade|abort-upgrade)

usr/bin/linuxmuster-fileserver

@@ -12,9 +12,12 @@ import subprocess
 import socket
 import os.path
 import os
+import sys
 
+from pwinput import pwinput
 from subprocess import PIPE
 
+
 def __execute(command) -> subprocess.CompletedProcess:
     """
     Execute a command and return a subprocess.CompletedProcess object
@@ -88,8 +91,7 @@ def setup(domain, username, password, school) -> str:
     if os.path.exists("/etc/krb5.conf"):
         print("✅")
     else:
-        print("❌")
-        exit(1)
+        sys.exit("❌")
 
     print("Copy template for new nsswitch.conf... ", end="")
     with open("/var/lib/linuxmuster-fileserver/nsswitch.conf.example", "r") as f:
@@ -99,8 +101,7 @@ def setup(domain, username, password, school) -> str:
     if os.path.exists("/etc/nsswitch.conf"):
         print("✅")
     else:
-        print("❌")
-        exit(1)
+        sys.exit("❌")
 
     print("Copy template for new smb.conf... ", end="")
     with open("/var/lib/linuxmuster-fileserver/smb.conf.example", "r") as f:
@@ -113,16 +114,14 @@ def setup(domain, username, password, school) -> str:
     if os.path.exists("/etc/samba/smb.conf"):
         print("✅")
     else:
-        print("❌")
-        exit(1)
+        sys.exit("❌")
 
     print("Try to join domain with given data... ", end="")
-    __execute(["/usr/bin/net", "ads", "join", "-U", username + "%" + password])
+    output = __execute(["/usr/bin/net", "ads", "join", "-U", username + "%" + password])
     if "Join is OK" in __execute(["/usr/bin/net", "ads", "testjoin"]).stdout.decode():
         print("✅")
     else:
-        print("❌")
-        exit(1)
+        sys.exit(f"❌ \n\033[91m{output.stdout.decode()}\033[0m")
 
     print("Create share for school on this fileserver... ", end="")
     path = "/srv/samba/schools/" + school
@@ -139,8 +138,7 @@ def setup(domain, username, password, school) -> str:
     if school in __execute(["/usr/bin/net", "conf", "list"]).stdout.decode():
         print("✅")
     else:
-        print("❌")
-        exit(1)
+        sys.exit("❌")
 
     print("Restart service smbd.... ", end="")
     __execute(["/usr/bin/systemctl", "restart", "smbd"])
@@ -189,7 +187,7 @@ def main():
 
     if args.command == "setup":
         if not args.password:
-            password = input("Please enter password for %s@%s: " % (args.username, args.domain.upper()))
+            password = pwinput("Please enter password for %s@%s: " % (args.username, args.domain.upper()))
         else:
             password = args.password
         print(setup(args.domain.upper(), args.username, password, args.school.lower()))
@@ -201,4 +199,4 @@ def main():
 
 
 if __name__ == "__main__":
-    main()
+    main()