close Warning: Can't synchronize with repository "keplertools" (/srv/svn/keplertools does not appear to be a Subversion repository.). Look in the Trac log for more information.

Opened 2 years ago

Last modified 21 months ago

#5323 accepted Aufgabe

OTP-Generator testen

Reported by: Rüdiger Kupper Owned by: Rüdiger Kupper
Priority: bei Gelegenheit Milestone: Image »Klandestine Kerstin«
Component: Software: Ubuntu Severity: beeinträchtigt den Unterricht einiger Lehrer
Keywords: Cc: bau

Description (last modified by Rüdiger Kupper)

Beim Anzeigen der Backup-Codes steht in der ersten Zeile ein falscher String. Da läuft etwas beim greppen der Codes falsch.

Change History (12)

comment:1 by Rüdiger Kupper, 2 years ago

Erzeugung des OTP funktioniert, auch das Fenster-Handling.

comment:2 by Rüdiger Kupper, 2 years ago

Cc: bau added
Description: modified (diff)
Priority: bald erledigen → bei Gelegenheit

comment:3 by Rüdiger Kupper, 2 years ago

Milestone: Image »Klandestine Kerstin« → Schuljahr 2023/24

comment:4 by Rüdiger Kupper, 2 years ago

Resolution: → erledigt
Status: new → closed

Scheint weiterhin zu laufen.

in reply to:  4 comment:5 by Rüdiger Kupper, 2 years ago

Milestone: Schuljahr 2023/24 → Image »Klandestine Kerstin«
Resolution: erledigt
Severity: beeinträchtigt keinen Unterricht → beeinträchtigt den Unterricht einiger Lehrer
Status: closed → reopened

Replying to kup:

Scheint weiterhin zu laufen.

... aber leider nicht korrekt unter Wayland: Das Fenster wird nicht automatisch in den Vordergrund geholt und der Code wird auch nicht automatisch generiert. Sonst läuft es prinzipiell.

Last edited 2 years ago by Rüdiger Kupper (previous) (diff)

comment:6 by Rüdiger Kupper, 2 years ago

Owner: changed from Rüdiger Kupper to bau
Status: reopened → assigned

Genauer: Es sind zwei getrennte Probleme:

  • Beim Start erscheint die Meldung "Passwort generiert und in Zwischenablage kopiert", es wird aber tatsächlich nichts in die Zwischenablage kopiert. Erst wenn man auf den Knopf drückt klappt es.
@bau
Das ist wohl ein Fall für Dich, bitte mal im Skript "/usr/local/sbin/OTP.py" auf einem der Betarechner testen und korrierieren.
  • Dass das Fenster nicht in den Vordergrund geholt wird liegt daran, dass "devilspie" unter Wayland nicht funktioniert. Das müsste dann auch die Chrome-Apps betreffen (schul.cloud, trac, sogo in der Seitenleiste). Die sind aber veilleicht auch mal verzichtbar.

comment:7 by Rüdiger Kupper, 2 years ago

Die Chrome-Apps starten wie sie sollen, doch das alte Problem ist zurück, dass der rote Punkt in der Startleiste nicht bei der App erscheint, sondern bei Chrome. (Regression, hatte ich über devilspie gelöst, das erfordert aber X11).

Siehe:

comment:8 by bau, 22 months ago

Das OTP-Script /usr/local/sbin/OTP.py konnte das Logo nicht mehr finden, dieses war im Tauschlaufwerk unter
home/share/Kollegium/Logo_und_Briefvorlagen/LOGO_KEPLER_klein 40x26_Kaupp_geglaettet.png war gerade eben dort aber nicht mehr aufzufinden.

Das Script ist eigentlich OK, es erzeugt die Oberfläche und ruft dann ButtonClick() auf. Die Meldung "Passwort generiert und in Zwischenablage kopiert" wird in dieser Funktion generiert. Anscheinend ist das mal wieder eine Race-Condition, da beim regulären Click auf den Button das OTP ja in die Zwischenablage kopiert wird.

comment:9 by bau, 21 months ago

OTP.py konnte dem Clipboard das erzeugte OTP zum Startzeitpunkt nicht übergeben, laut ​https://doc.qt.io/qtforpython-5/PySide2/QtGui/QClipboard.html ist es verwunderlich, dass das jemals funktioniert hat:

Lastly, the X11 clipboard is event driven, i.e. the clipboard will not function properly if the event loop is not running. Similarly, it is recommended that the contents of the clipboard are stored or retrieved in direct response to user-input events, e.g. mouse button or key presses and releases. You should not store or retrieve the clipboard contents in response to timer or non-user-input events.

Ich habe das jetzt timergesteuert umgesetzt:

    w.timer=QTimer()
    w.timer.timeout.connect(ButtonClick)
    w.timer.start(1000)

    #der Timer ersetzt ButtonClick() 

comment:10 by bau, 21 months ago

Owner: changed from bau to Rüdiger Kupper

Script muss jetzt auf alle Rechner verteilt werden -> kup
(liegt auf dem paedml unter (/usr/local/sbin/OTP.py)

das Script /usr/local/sbin/OTP.pyc kann entfernt werden

in reply to:  10 comment:11 by Rüdiger Kupper, 21 months ago

Replying to bau:

Script muss jetzt auf alle Rechner verteilt werden -> kup
(liegt auf dem paedml unter (/usr/local/sbin/OTP.py)

Okay, und auch ins Image.

das Script /usr/local/sbin/OTP.pyc kann entfernt werden

Das ist der compilierte Bytecode, wenn wir den liegen lassen, geht der Aufruf schneller.

comment:12 by Rüdiger Kupper, 21 months ago

Status: assigned → accepted
Note: See TracTickets for help on using tickets.