Authenticator-Extension / Authenticator-Extension/Authenticator
Scanned or imported QR code doesn't respect algorithm (and probably period)
- Vorherrschende Sprache
- TypeScript
- Sterne
- 4.7k
- Forks
- 1.2k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
### Describe the issue
Scanning or importing a QR code image with a URI like
otpauth://totp/USERNAME?secret=THESECRET&issuer=THEISSUER&algorithm=SHA512&digits=8&period=60
results in non-working TOTPs.
When scanning the QR code that is shown in the plugin's UI (when hovering over the entry and clicking the QR code icon) with an other external QR code scanner, the parameters `alogrithm` and `period` are missing.
Because the TOTPs are wrong, I guess that these parameters are not only missing in the generated plugin's QR code but aren't taken into account, when translating the imported/scanned QR code.
When entering all the data manually, the TOTP is correct and the parameters are shown correctly in the generated plugin's QR code.
Same issue in Edge (Version 144.0.3719.104) and probably also in Chrome.
Unfortunately, the manual setup is a workaround, but not a real good behaviour for most "normal" users, so I would appreciate an appropriate bug fix.
Thanks in advance.
Apart from that... a great plugin.
### Browser
Firefox
### Browser Version
143.0.1
### Extension Version
8.0.2
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Verfolge die Verarbeitung beim QR-Import und -Export für die bereitgestellte otpauth://totp-URI und vergleiche sie mit dem manuellen Einrichtungspfad. Überprüfe, ob Algorithmus, Ziffernanzahl und Zeitraum den Import überstehen und in den generierten QR-Codes erscheinen, und teste anschließend die resultierenden TOTP-Werte mit SHA512 und einem Zeitraum von 60 Sekunden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- authentication
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100