search

Romern / syncMyMoodle

Synchronization client for RWTH Moodle
GNU General Public License v3.0
70 stars 18 forks source link

Add totp generator ontop of MFA support #120

Closed D-VR closed 1 week ago

D-VR commented 1 week ago

This is in relation to a feature request in #118

closes #118

D-VR commented 1 week ago

This is missing secretstorage support! Don't have it setup for myself..

D-VR commented 1 week ago

I made it into a separate PR as #119 as this is an additional feature unrelated to just supporting MFA.

D-VR commented 1 week ago

I have rebased the feature

Romern commented 1 week ago

Würde noch secretstorage hinzufügen, und hier im Zuge secretstorage durch keyring ( https://pypi.org/project/keyring/ ) ersetzen (das benutzt unter linux secretstorage unter der haube), dann kann man das auch unter windows und macos benutzen.

D-VR commented 1 week ago

Ist das soweit gut, oder soll ich nochmal testen?

Romern commented 1 week ago

Ist das soweit gut, oder soll ich nochmal testen?

Wäre nice wenn du das noch testen könntest, hatte grad noch gemerkt das es evtl nicht so smart ist zu requiren das man das TOTP secret storen muss wenn man keyring aktiviert hat. Ansonsten kannst du das einfach mergen.