Das deutsche Python-Forum

@feldmaus: Sieht so aus als wenn man nur bei `pexpect.spawn()` die Argumente als Liste übergeben kann aber nicht bei `pexpect.run()`. Das ist doof.

Das Snippet

Code: Alles auswählen

if antwort2 == ('s' or 'S') :

macht nicht ganz bestimmt nicht das was du willst:

Code: Alles auswählen

In [1]: 's' or 'S'
Out[1]: 's'

In [2]: acceptable = 's' or 'S'

In [3]: answer = 's'

In [4]: answer == acceptable
Out[4]: True

In [5]: answer = 'S'

In [6]: answer == acceptable
Out[6]: False

Alternativen:

Code: Alles auswählen

In [7]: acceptable = ('s', 'S')

In [8]: answer in acceptable
Out[8]: True

In [9]: answer = 's'

In [10]: answer in acceptable
Out[10]: True

In [11]: acceptable = 'sS'

In [12]: answer in acceptable
Out[12]: True

In [13]: answer = 'S'

In [14]: answer in acceptable
Out[14]: True

(Und natuerlich das schnoede `answer == 's' or answer == 'S'`)

BlackJack hat geschrieben:@feldmaus: Sieht so aus als wenn man nur bei `pexpect.spawn()` die Argumente als Liste übergeben kann aber nicht bei `pexpect.run()`. Das ist doof.

Also in der Doku steht ein Beispiel:

Code: Alles auswählen

child = pexpect.spawn ('ls', ['-latr', '/tmp'])

Also muss der eigentliche Befehl vor der Liste mit den Argumenten stehen, oder geht das wirklich auch nur mit einer Liste?

@feldmaus: Das ist `spawn()` -- Du hattes es ja aber mit `run()` versucht.

Ich weiß, ich bin sofort zu spawn gesprungen. Hier gibt es zumindest Beispiele mit Listen. Übersichtlichkeit sollte schon sein.

Irgendwie kennt rsync keine Option --exclude, sondern nur --exclude= mit dem Gleichheitszeichen am Ende, nur dann muss das Argument direkt nach dem Gleichheitszeichen beginnen, ansonsten erkennt er es als Quelle. Hat Jemand eine Idee?

Grüße Markus

@feldmaus: Was für ein ``rsync`` verwendest Du denn da? Das geht auch ohne das ``=``.

Also ich nutze rsync Version

Code: Alles auswählen

rsync  version 3.0.7  protocol version 30
Copyright (C) 1996-2009 by Andrew Tridgell, Wayne Davison, and others.
Web site: http://rsync.samba.org/
Capabilities:
    64-bit files, 64-bit inums, 32-bit timestamps, 64-bit long ints,
    socketpairs, hardlinks, symlinks, IPv6, batchfiles, inplace,
    append, ACLs, xattrs, iconv, symtimes

Also in der manpage steht ein Beispiel ohne "=", aber wenn ich es ohne "=" ausführe bekomme ich die Meldung

Code: Alles auswählen

rsync: --exclude : unknown option

Und mit dem "=" versucht er mein Exclude-Argument als Source zu verwenden

Code: Alles auswählen

building file list ... 

rsync: link_stat "/home/markus/bin/__ThereIsNothingToExcludeInThisSourceButINeedAnEmptyExclude__ " failed: No such file or directory (2)

rsync: link_stat "/home/markus/bin/ " failed: No such file or directory (2)

0 files to consider

Bei diesem Beispiel hat meine Quelle keine Auszuschliessenden, trotzdem verwende ich die Option "--exclude[=]", da ich mein Programm sonst auch durch eine Fallunterscheidung unnötig verkomplizieren würde. Wie auch immer, die Quellen mit Auszuschliesenden funktionieren so zumindest nicht.

Was ich nicht verstehe ist, warum er mein /home/markus/bin/ Verzeichnis nicht findet. Und von markus@server:/home/markus/bin/ (meinem Ziel-Verzeichnis) wurde hier nichts erwähnt.

@feldmaus: Du solltest Dich wirklich ganz dringend endlich mal damit beschäftigen wie die Übergabe von Argumenten funktioniert. Auf C-Ebene. Und was zum Beispiel eine Shell alles macht um von einer Zeile in der Shell auf eine Argumentliste auf C-Ebene zu bekommen. Genau da machst Du nämlich schon wieder Fehler.

Da steht nämlich nicht, dass ``rsync`` die Option '--exclude' nicht kennt, sondern dass es die Option '--exclude ' nicht kennt. Da ist ein Leerzeichen was da nicht hin gehört:

Code: Alles auswählen

bj@s8n:~$ rsync '--exclude'
rsync: --exclude: missing argument
rsync error: syntax or usage error (code 1) at main.c(1443) [client=3.0.7]
bj@s8n:~$ rsync '--exclude '
rsync: --exclude : unknown option
rsync error: syntax or usage error (code 1) at main.c(1443) [client=3.0.7]

Es funktioniert

BlackJack hat geschrieben:@feldmaus: Du solltest Dich wirklich ganz dringend endlich mal damit beschäftigen wie die Übergabe von Argumenten funktioniert. Auf C-Ebene. Und was zum Beispiel eine Shell alles macht um von einer Zeile in der Shell auf eine Argumentliste auf C-Ebene zu bekommen. Genau da machst Du nämlich schon wieder Fehler.

Mich mögen die Leerzeichen halt besonders gerne

Es funktioniert

Es kommen aber noch einige andere Funktionen mit rein z.b. falls ein falsches Passwort eingegeben wurde.

Und hier der Code mit ungekürzten Leerzeichen

Code: Alles auswählen

                    PASSWORT=raw_input('Bitte Passwort fuer '+BENUTZER+' eingeben:')
                    benutzer_passwoerter[BENUTZER]=PASSWORT
                    child=rsync()
                    child.logfile = sys.stdout
                    child.expect('password:', timeout=3)
                    child.logfile = sys.stdout
                    child.sendline(PASSWORT)
                    child.logfile = sys.stdout
                    while not child.expect(pexpect.EOF):
                        PASSWORT=raw_input('Falsches Passwort!\nBitte Passwort fuer '+BENUTZER+' eingeben:')
                        benutzer_passwoerter[BENUTZER]=PASSWORT
                        child.expect('password:', timeout=3)
                        child.sendline(PASSWORT)

Nach erfolgreichem Beenden von dem rsync Befehl springt er trotzdem noch in die while-Schleife. Der Datenstrom müsste doch zu Ende sein, dann dürfte er doch auch nicht mehr in meine Schleife springen, oder interpretiere ich den Befehl child.expect(pexpect.EOF) falsch?

Habe nun einige wichtige Sachen geändert auch hinsichtlich der Lesbarkeit. Wenn der eine oder andere noch Style-Ideen hat, dann her damit. Als nächste Änderung kommt der Fehler TIMEOUT, den ich noch nicht berücksichtigt habe.

Code: Alles auswählen

    for eintrag in katalog:
        for BENUTZER in katalog[eintrag]['Benutzer']:
            antwort2=raw_input('Sichern/Zurueckspielen/Nichts'+katalog[eintrag]['Frage'][0]+BENUTZER+katalog[eintrag]['Frage'][1])
            PASSWORT=benutzer_passwoerter.get(BENUTZER, '')
            if antwort2 in 'sSzZ' and antwort2 not in '':
                child=rsync()
                child.logfile_read = sys.stdout
                if child.expect([' password:', pexpect.EOF],  timeout=3) in [0]:
                    child.sendline(PASSWORT)
                while child.expect([' password:', pexpect.EOF],  timeout=3) in [0]:
                    PASSWORT=sys.stdin.readline()
                    benutzer_passwoerter[BENUTZER]=PASSWORT
                    child.sendline(PASSWORT))

Ich werde wohl noch die Schleife und die letzte Fall-Unterscheidung zusammenfassen. Solange(WHILE) das Child am Leben ist frage den Status ab(IF) und reagiere dementsprechend.

@feldmaus: `eintrag` wird einzig und allein dafür verwendet um aus `katalog` den dazugehörigen Wert zu holen. Da Du aber sowieso über alle Werte aus `katalog` iterierst, ist dieser Umweg über den Schlüssel unnötig. Du kannst auch gleich über `katalog.itervalues()` iterieren.

Komplett gross geschriebene Namen sind per Konvention (PEP8) für Konstanten vorgesehen. `BENUTZER` und `PASSWORT` sind aber keine Konstanten.

Durchnummerieren von Namen ist ein "code smell". Meistens dafür, dass man eigentlich eine Liste verwenden möchte, aber hier eher dafür das zu viel in einer Funktion steckt.

Das so etwas wie ``katalog[eintrag]['Frage'][0]`` und superlange Zeilen die zudem keine Leerzeichen um Operatoren enthalten, schwer lesbar und schwer verständlich sind, hatten wir doch schon mal, oder!?

``in`` ist schon ein netter Operator, aber man muss nicht versuchen *alle* Bedingungen damit auszudrücken. Von den vier Anwendungen in dem Quelltext kann man drei offensichtlicher mit einfachen Vergleichen ausdrücken.

Warum wird für die Passworteingabe `sys.stdin.readline()` anstelle von `raw_input()` verwendet? Sicherer wäre die Eingabe über das `getpass`-Modul. Dann kann niemand die eingegebenen Passwörter am Bildschirm mitlesen.

Am Ende ist eine Klammer zu viel.

BlackJack hat geschrieben:@feldmaus: `eintrag` wird einzig und allein dafür verwendet um aus `katalog` den dazugehörigen Wert zu holen. Da Du aber sowieso über alle Werte aus `katalog` iterierst, ist dieser Umweg über den Schlüssel unnötig. Du kannst auch gleich über `katalog.itervalues()` iterieren.

Deine Idee mit dem Iterator-Objekt finde ich interessant, doch jedesmal wenn ich drüber nachdenke wie man damit meinen Code vereinfachen/lesbarer machen kann habe ich eher das Gefühl es wird nur schlimmer. Ein Iterator-Objekt hat zu wenig Methoden. Mit next() kann ich nicht alles erschlagen was ich brauche. Und zudem kommt noch, das ich die Ausnahme von dem Iterator behandeln muss(wenn das Ende des iterator erreicht ist), was Zusatz-Code verursacht. Kann man beim Iterator das derzeitige Element ausgeben? Ansonsten müsste ich viel mit Zuweisungen arbeiten und die verlängern meinen Code.

Falls das mit dem Iterator klappt, könnte man damit auch meine langen Zeilen erschlagen.

BlackJack hat geschrieben:Warum wird für die Passworteingabe `sys.stdin.readline()` anstelle von `raw_input()` verwendet? Sicherer wäre die Eingabe über das `getpass`-Modul. Dann kann niemand die eingegebenen Passwörter am Bildschirm mitlesen.

Weil ich nur Daten eingeben will und keine zusätzliche Ausgabe benötige, dass macht das Child schon für mich. Ich hatte eine Fehlermeldung bei raw_input(). Die genaue Fehlermeldung weiß ich nicht mehr, wahrscheinlich hat sich bei den Ausgaben irgendwas gebissen. Ich weiß es nicht mehr :K . Das mit dem Modul getpass finde ich gut und wollte ich eigentlich später ansprechen, aber egal

Hier mein vorläufiger Code:

Code: Alles auswählen

    for eintrag in katalog:
        for benutzer in katalog[eintrag]['Benutzer']:
            wasWillstDu=raw_input('Sichern/Zurueckspielen/Nichts'+katalog[eintrag]['Frage'][0]+benutzer+katalog[eintrag]['Frage'][1])
            passwort=benutzer_passwoerter.get(benutzer, '')
            if wasWillstDu in 'sSzZ' and wasWillstDu not in '':
                child=rsync()
                child.logfile_read = sys.stdout
                while child.isalive():
                    statusChild=child.expect([' password:', 'denied', pexpect.TIMEOUT, pexpect.EOF],  timeout=-1)
                    if statusChild == 0:
                        child.sendline(passwort)
                    elif statusChild == 1:
                        passwort=sys.stdin.readline()
                        benutzer_passwoerter[benutzer]=passwort
                        child.sendline(passwort)

Grüße Markus

@feldmaus: Mit dem Iterator solltest Du selbst überhaupt nichts machen ausser darüber iterieren -- mit einer ``for``-Schleife und nicht "zu Fuss".

BlackJack hat geschrieben:@feldmaus: Mit dem Iterator solltest Du selbst überhaupt nichts machen ausser darüber iterieren -- mit einer ``for``-Schleife und nicht "zu Fuss".

Ich werde mal ein wenig konkreter, aber nicht zu viel. Hier der Code

Code: Alles auswählen

    iterEintraege=katalog.itervalues()
    for benutzer in iterEintraege.next()['Benutzer']:
        wasWillstDu=raw_input('Sichern/Zurueckspielen/Nichts'+katalog[eintrag]['Frage'][0]+benutzer+katalog[eintrag]['Frage'][1])
        passwort=benutzer_passwoerter.get(benutzer, '')
        if wasWillstDu in 'sSzZ' and wasWillstDu not in '':
            child=rsync()
            child.logfile_read = sys.stdout
            while child.isalive():
                statusChild=child.expect([' password:', 'denied', pexpect.TIMEOUT, pexpect.EOF],  timeout=-1)
                if statusChild == 0:
                    child.sendline(passwort)
                elif statusChild == 1:
                    passwort=sys.stdin.readline()
                    benutzer_passwoerter[benutzer]=passwort
                    child.sendline(passwort)

Ich habe jetzt zwar eine Schleife gespart, aber der Code funktioniert nicht, weil ich in der for-Anweisung keine Zuweisung von dem next() Objekt machen kann. Somit steht mir dieses Objekt auch weiter unten nicht zur Verfügung. Hast Du eine Idee wie ich eine Zuweisung da rein bekomme, also so ähnlich wie

Code: Alles auswählen

for benutzer in eintrag=iterEintraege.next()['Benutzer']:

Und irgendwie müsste ich noch eine Exception einbauen, für den Fall das der Iterator zu Ende ist.

@feldmaus: Natürlich funktioniert das so nicht. Nach dem ``in`` in einer ``for``-Schleife muss ein iterierbares Objekt stehen. Zum Beispiel ein Iterator. Über die Elemente dieses Iterators geht die ``for``-Schleife dann. Du musst dafür weder die `next()`-Methode aufrufen noch die `StopIteration` behandeln. Das macht alles die ``for``-Schleife intern. Die einzelnen Elemente werden nacheinander an den Namen nach dem ``for`` gebunden. Das sind Grundlagen; so funktioniert *jede* ``for``-Schleife in Python.

@BlackJack
Sag das doch gleich

Code: Alles auswählen

    for eintrag in katalog.itervalues():
        for benutzer in eintrag['Benutzer']:
            wasWillstDu=raw_input('(S)ichern/(Z)urueckspielen/(N)ichts/(A)bbrechen'+eintrag['Frage'][0]+benutzer+eintrag['Frage'][1]+' ? (s/z/n/a) ')
            passwort=benutzer_passwoerter.get(benutzer, '')
            if wasWillstDu in 'sSzZ' and wasWillstDu not in '':
                child=rsync()
                child.logfile_read = sys.stdout
                while child.isalive():
                    statusChild=child.expect([' password:', 'denied', pexpect.TIMEOUT, pexpect.EOF],  timeout=-1)
                    if statusChild == 0:
                        child.sendline(passwort)
                    elif statusChild == 1:
                        passwort=getpass.getpass('')
#                        passwort=sys.stdin.readline()
                        benutzer_passwoerter[benutzer]=passwort
                        child.sendline(passwort)
            elif wasWillstDu in 'aAqQ' and wasWillstDu not in '':
                sys.exit(0)

EDIT: Habe das getpass Modul nochmal nachgetragen.

Ich kenne die Struktur von ``katalog`` nicht, aber vermutlich suchst du so etwas wie:

Code: Alles auswählen

for eintrag in katalog.values():
    benutzer = eintrag['Benutzer']
    [...]

EDIT: Anscheinend hast du schon gefunden, was du wolltest.

Das getpass Modul macht nicht das was es soll. Das eingegebene Passwort wird jedes mal an die Variable <passwort> angehängt. Also z.b. gebe ich zu erst 943jfn ein dann sj38sks und dann dk32k5, dann würde beim dritten Durchgang die Variable passwort wie folgt lauten 943jfnsj38sksdk32k5

Hat da Jemand einen Tipp für mich?

EDIT: Ich wollte das Programm gerne im Terminal aber grafisch in Richtung ncurses aufpeppen und vielleicht noch eine XML Datenbank dazu. Gibt es ein Modul womit ich solche eine Terminal/Konsolen

-Grafik bewerkstelligen kann?

EDIT2: Hier ein Link zu curses unter Python http://de.wikibooks.org/wiki/Python_unter_Linux:_Curses

feldmaus hat geschrieben:Das getpass Modul macht nicht das was es soll. Das eingegebene Passwort wird jedes mal an die Variable <passwort> angehängt. Also z.b. gebe ich zu erst 943jfn ein dann sj38sks und dann dk32k5, dann würde beim dritten Durchgang die Variable passwort wie folgt lauten 943jfnsj38sksdk32k5

Habs ausprobiert, kann ich nicht nachvollziehen. Bei mir funktionierts wie erwartet.

Das deutsche Python-Forum

rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe

Re: rsync ssh und Passwort Eingabe