pfaffman
(Jay Pfaffman)
11. August 2020 um 16:54
1
Ich kann die Zwei-Faktor-Authentifizierung (2FA) auf meinem Konto hier bei Meta nicht deaktivieren. Wenn ich auf „Deaktivieren
Können wir das @tshenry reproduzieren?
2 „Gefällt mir“
@pfaffman Wie stehen die Chancen, dass dies auf eine bestimmte Art von 2FA beschränkt ist?
Ich habe es gerade versucht, dies nachzustellen, und meine TOTP-2FA wurde sofort deaktiviert.
tshenry
(Taylor)
11. August 2020 um 17:29
5
Ja, das ist ein Problem mit Sicherheitsschlüsseln. Das Klicken auf die Schaltfläche Deaktivieren auf der Hauptseite für die Verwaltung des zweiten Faktors in Ihrem Benutzerprofil deaktiviert nur Token-basierte Authentifizierungsverfahren, obwohl alle Methoden des zweiten Faktors deaktiviert werden sollten.
Hinweis: Sie können die Schlüssel deaktivieren, indem Sie auf das kleine Bleistift-Symbol neben dem Schlüsselnamen klicken und dann auf den Mülleimer im Modal-Fenster klicken:
Ich habe diesen Fehler seit einiger Zeit auf meiner Liste, also ist es mein Fehler, dass ich hier nicht konsequent geblieben bin. Ich denke, ein Grund dafür war, dass ich dies gesehen, aber nicht genau genug hingeschaut habe:
master ← riking:patch-156276
merged 07:19PM - 07 Jul 20 UTC
Ich ging davon aus, dass dies das Problem wahrscheinlich behebt und wollte das überprüfen, bin aber nie dazu gekommen. Die oben genannte Korrektur betraf die Admin-Sicht. Ich denke, die Korrektur aus der Benutzersicht sollte sehr ähnlich sein. In jedem Fall werde ich sicherstellen, dass diese Woche eine Korrektur in die Pipeline aufgenommen wird.
7 „Gefällt mir“
pfaffman
(Jay Pfaffman)
11. August 2020 um 17:30
6
100 %
Ich habe sie einzeln gelöscht und es hat einfach funktioniert. Ich weiß nicht, warum mir das vorher nicht eingefallen ist. Entschuldigung dafür.
Also, wenn es einen Fehler gibt, dann liegt er an der Option ALLE AUSSCHALTEN.
4 „Gefällt mir“
tshenry
(Taylor)
31. August 2020 um 19:55
8
Das sollte jetzt behoben sein
master ← second-factor-fixes
merged 04:56PM - 31 Aug 20 UTC
This will fix the bug reported [here on Meta](https://meta.discourse.org/t/cant-… disable-2fa/160500)
The second factor management page has previously included a "Disable" button. Selecting this button brings up a confirmation:
<img width="619" alt="old" src="https://user-images.githubusercontent.com/22733864/90696567-5a087500-e231-11ea-8156-1ce333502d9f.png">
Instead of disabling all registered TOTP Authenticators **and** Security Keys, confirming only disables all TOTP Authenticators.
---
This PR will change the "Disable" button on the second factor management page to say "Disable All," and make a few styling/copy improvements to the confirmation:
<img width="618" alt="Screen Shot 2020-08-13 at 6 45 51 PM" src="https://user-images.githubusercontent.com/22733864/90697350-0d259e00-e233-11ea-84cf-48b44d0389dd.png">
<img width="618" alt="Screen Shot 2020-08-19 at 3 31 25 PM" src="https://user-images.githubusercontent.com/22733864/90697359-13b41580-e233-11ea-98cb-95262ad444e2.png">
It also fixes the behavior so that **all** second factor methods are removed when using the "Disable All" button.
**Notes**
- The test was tricky and takes heavy inspirations from other tests. I imagine there may be a more elegant or complete implementation, but I'm pretty sure I have the most important functionality tested. If I need to make any changes, let me know.
- Both second factor methods can be disabled individually instead of using the "Disable All" button using the pencil icon buttons showing in the above screenshot. What I came to realize is that the implementation for individual disabling is quite different than the disable all method. The "Disable All" button deletes the relevant `user_second_factors` and `user_security_keys` records from the database, whereas disabling the second factors individually sets `enabled: false` in the relevant record and keeps it around. This is not new behavior and nothing is technically broken, I just wanted to bring it up in case we might want to have consistency. If anything needs to change, I imagine it can be addressed in a follow-up commit.
4 „Gefällt mir“
tshenry
(Taylor)
Geschlossen,
2. September 2020 um 01:00
9
Dieses Thema wurde automatisch nach 29 Stunden geschlossen. Neue Antworten sind nicht mehr möglich.