C’est très bien !
Un petit détail cependant : sur Safari/Mac, l’authentification Web est une fonctionnalité réservée aux développeurs, désactivée par défaut. Une fois activée, elle fonctionne bien. Mais il faudrait probablement afficher un message ou un avertissement lorsque l’authentification Web n’est pas activée. Actuellement, sur Safari par défaut, rien dans l’interface n’indique que le processus d’enregistrement ne fonctionnera pas (la console affiche une erreur) :
Nous pouvons faire de la détection de fonctionnalité en utilisant navigator.credentials, je suppose.
Certaines personnes ont du mal à justifier la dépense supplémentaire, même si c’est agréable 
Argh, j’ai vérifié cette fonctionnalité sur navigator.credentials.get mais pas sur create, désolé pour ça, ce sera une correction rapide !
Le commit le plus récent semble fonctionner. Je viens d’utiliser l’authentification à deux facteurs (avec empreinte digitale !) sur mon téléphone Android.
Salut Penar, j’ai une correction prête pour cela maintenant. Peux-tu vérifier cela sur Safari ? [FIX] Check webauthn support when registering security keys by mjrbrennan · Pull Request #8146 · discourse/discourse · GitHub. Je ne suis pas sûr que la méthode de détection de fonctionnalité que j’utilise fonctionnera avec Safari si la fonctionnalité est désactivée.
Sam m’a devancé pour la fusion, mais cela semble bon sur Safari, merci Martin !
C’est vrai, mais dans ce cas, il s’agit d’une partie normale du travail chez Discourse, donc nous prendrions en charge les frais. Désolé si ce n’était pas clair, mais j’espère que c’est plus clair maintenant ?
Félicitations pour le support de WebAuthn ! C’est intéressant de voir que vous avez développé votre propre solution plutôt que d’utiliser le gem webauthn. Si vous avez des retours à nous faire part, je serais ravi de les entendre
J’ai remarqué que votre implémentation ne prend en charge que l’algorithme -7 (ES256), mais les authentificateurs de plateforme Windows Hello (basés sur le matériel TPM 2.0) nécessitent -257 (RS256), comme l’indique la documentation de Microsoft. Le TPM 2.0 est requis depuis le 28 juillet 2016 pour les nouveaux modèles d’ordinateurs de bureau Windows 10, ce qui représente une quantité non négligeable de matériel.
Une suggestion pour la maquette du “Flux de connexion” : WebAuthn dispose d’un logo officiel qui pourrait être utilisé à la place d’une image générique d’empreinte digitale. Outre l’empreinte digitale, la reconnaissance faciale, le motif de balayage ou le code PIN sont également des options courantes de vérification utilisateur.
@Martin_Brennan des avis sur 
Merci pour votre retour, Rafe. Nous devrons donc ajouter l’algorithme supplémentaire ici. Et merci également pour le lien vers le logo officiel ! Mon idée, lorsque j’ai limité l’implémentation à un seul algorithme, était d’ajouter le nombre minimum d’algorithmes pris en charge pour faire fonctionner la version 1, car je n’étais pas sûr des nuances entre tous les algorithmes, et ES256 était utilisé dans tous les exemples que j’ai pu trouver.
Quant à la raison pour laquelle j’ai choisi de ne pas utiliser le gem à l’époque, j’y ai beaucoup réfléchi et je savais que cette décision serait remise en question à un moment ou à un autre. J’ai certes lu une grande partie du code du gem pour mieux comprendre son implémentation. Les principales raisons étaient :
Cela dit, cela aurait pu être une erreur de ma part. Si nous passons aux versions V2, V3, etc. du support de WebAuthn avec des attestations, plus d’algorithmes pris en charge, et ainsi de suite, cela pourrait devenir trop fastidieux pour nous de devenir des « experts WebAuthn », pour ainsi dire. À ce stade, je pense que nous pourrions réévaluer l’utilisation du gem.
Le support de l’algorithme supplémentaire RS256 a désormais été fusionné FEATURE: Support RS256 algorithm for webauthn by martin-brennan · Pull Request #8385 · discourse/discourse · GitHub
Il semble que iOS 13.3 et macOS Catalina 10.15.2 prennent désormais également en charge l’authentification FIDO2 
Cette mise à jour inclut également des corrections de bugs et d’autres améliorations. Cette mise à jour :
- Ajoute la prise en charge des clés de sécurité conformes FIDO2 NFC, USB et Lightning dans Safari
C’est dommage que les iPhone aient besoin d’un périphérique tiers pour cela. J’espère qu’iOS++ l’intégrera nativement avec l’authentification de l’appareil et la puce sécurisée, comme Windows, macOS et Android.
Je suis tellement ravi de pouvoir me connecter à Discourse en le regardant !!
Je suis en fait un peu surpris que le support de l’authentificateur intégré ne soit pas encore là… mais c’est une nouvelle excitante quand même - je continuerai à retenir mon souffle.
J’ai remarqué une possible incohérence de l’interface utilisateur concernant l’indication de l’activation de la double authentification (2FA) pour certains utilisateurs sur une instance hébergée de Discourse :
La liste de tous les utilisateurs « Staff » ne montre pas que mon compte a la 2FA activée :
La page de résumé du compte suggère que la 2FA est activée, étant donné le texte du bouton « Gérer l’authentification à deux facteurs ».
L’authentification à deux facteurs indique qu’une clé de sécurité est activée et que le second facteur pourrait être désactivé.
D’autres utilisateurs sur la même instance qui choisissent d’utiliser un authentificateur basé sur des jetons (sans clé de sécurité) affichent l’icône de cadenas dans la liste « tous les utilisateurs ».
Veuillez me faire savoir s’il s’agit d’un bug de l’interface utilisateur ou si l’ajout d’une simple clé de sécurité ne suffit pas pour activer la 2FA sur cette plateforme.
Je vais examiner cela, mais mon hypothèse est qu’il s’agit simplement d’un bug d’interface utilisateur dans la liste des utilisateurs, si la section 2FA réelle de l’interface affiche tout correctement. Merci pour votre signalement !
C’était le problème. La correction est en cours de construction ici et je la fusionnerai dès que possible : FIX: Admin user list not showing 2FA icon for only security keys enabled by martin-brennan · Pull Request #8839 · discourse/discourse · GitHub
Édition : Ceci est maintenant corrigé.
Bienvenue dans le nouveau monde
Apple a rejoint l’Alliance FIDO (alias Fast Identity Online), une organisation qui compte déjà des géants tels que Google, Intel, Microsoft et Samsung.
Cela fonctionne avec TouchID / FaceID sur iOS 14.
