python subprocess: „su - -c” als Variable

[Peter N. (mikronix)]

Hallo liebe Leser!
Hab mir mal wieder ein wenig python Programmierung vorgenommen...!?
Als Beispiel mal ein Verzeichnis als root erstellen, was so auch 
funktioniert:

Root = "su - -c"; Pfad = "/tmp/.abc/"; PfadTmp = "def"
subprocess.run(f"{Root} 'mkdir -p {Pfad}{PfadTmp}'", shell=True) # ok
subprocess.run(f"{Root} 'mkdir -p {Pfad}{PfadTmp}'", shell="/bin/bash") # ok
subprocess.run(["su", "-", "-c", 'mkdir -p ' + Pfad + PfadTmp]) # ok, ohne shell=irgendwas

shell=True soll man ja vermeiden, bliebe also nur der letzte Befehl.
Das ich auch als User nach /tmp schreiben darf ist schon klar, darum 
gehts nicht da ich so einen Befehl in Variationen öfter brauche.
Kann ich "su", "-", "-c" irgendwie funktionsfähig in eine Variable 
packen und falls ja, dann wie?
Mit meinen Versuchen mit Root=... bin ich leider kläglich gescheitert!
Der geneigte Leser darf mein Betriebssystem gern als debian erkennen...

[Axel G. (axelg)]

Peter N. schrieb:

> Als Beispiel mal ein Verzeichnis als root erstellen, was so auch
> funktioniert: [...]

Warum nicht "os.mkdir('/tmp/datda')"?

[Peter N. (mikronix)]

Und das funktioniert auch mit:
os.mkdir('/abc')
Wohlgemerkt: ich starte das Skript nicht als root!

[Daniel A. (daniel-a)]

Das Konzept an sich ist etwas fragwürdig, wenn ein Script Root braucht, 
startet man es halt als root, statt darin mit su herumzumachen, aber was 
solls...

Du könntest dir eine Funktion machen, in der du das Escaping machst, 
usw:

#!python3
def shell_escape(cmd):
    return ' '.join("'"+x.replace("'","'\\''")+"'" for x in cmd)
def as_root(cmd):
    return ['su', '-', '-c', shell_escape(cmd)]

Oder du könntest "sudo -- " stat "su - -c" nehmen, da musst du nur die 
Listen verketten, und musst das nicht mehr extra escapen.

shell=True sollte man ja vermeiden, damit man nicht je nach Shell falsch 
escapt, aber su -c führt das zeug auch in einer Shell aus, damit ist man 
wieder am Anfang.

#!python3
def as_root(cmd):
    return ['sudo', '--', *cmd]

Am besten wäre es aber immer, gar nicht erst root zu benötigen.

[Andreas M. (amesser)]

Peter N. schrieb:
> Kann ich "su", "-", "-c" irgendwie funktionsfähig in eine Variable
> packen und falls ja, dann wie?

Klar, einfach dran denken, das die eckigen Klammern eine Liste bedeuten. 
(Und auch run eine liste erwartet)

root = ["su", "-", "-c"] 
# oder besser zu lessen:
root = "su - -c".split()

Pfad = "/tmp/.abc/"
PfadTmp = "def"

# und dann liste neinfach "addieren"
subprocess.run(root + ["mkdir -p " + Pfad+  PfadTmp])

[Peter N. (mikronix)]

Aha, so geht das also!
Vielen Dank Andreas, funktioniert perfekt:

Root = ["su", "-", "-c"]; Pfad = "/tmp/.abc/"; PfadTmp = "def"
subprocess.run(Root + ['mkdir -p ' + Pfad + PfadTmp])

Für die paar root Befehle die ich brauche möchte ich nicht das komplette 
Skript als root laufen lassen, da könnten verschiedene Änderungen schon 
verheerende Folgen haben...
gelöst, größten Dank an alle Beteiligten!

[Stephan S. (uxdx)]

Wenn das wirklich in /tmp angelegt werden soll, brauchst Du doch keinen 
root

[Sheeva P. (sheevaplug)]

Daniel A. schrieb:
>

> #!python3
> def shell_escape(cmd):
>     return ' '.join("'"+x.replace("'","'\\''")+"'" for x in cmd)
>

OMG, Dein Ernst? Obwohl es shlex gibt?