Depuis la création de Discourse, les connexions sociales sur ordinateur utilisaient toujours une fenêtre « popup » pour se connecter.
L’intention était d’être « moins » distrayant en vous maintenant sur le site même pendant l’acte de « connexion ».
Je pense que cela n’a plus de sens de conserver cela pour plusieurs raisons.
Cela est déjà incohérent avec le fonctionnement du « SSO Discourse », cela peut être instable sur les navigateurs qui bloquent les popups (mon Firefox se plaint et nécessite un clic pour l’autoriser), le code pour publier des messages est complexe et difficile à maintenir, et nous exigeons déjà un clic supplémentaire dans la popup.
Que pensez-vous d’éliminer le code « ouvrir une popup pour la connexion sociale » et de n’avoir qu’une seule méthode cohérente de connexion, qu’il s’agisse de SSO, de connexion sociale ou autre ?
Je suis d’accord, nous devrions nous débarrasser des popups et rendre tout cohérent. Cela nous permettrait également de supprimer beaucoup de code
Les problèmes que vous avez décrits dans Firefox sont corrigables, mais comme vous l’avez dit, le code des popups est complexe et sujet à des bugs comme celui-ci.
Je peux travailler là-dessus plus tard cette semaine si vous le souhaitez ?
Une fois qu’elle sera intégrée dans la version stable, nous devrons supprimer les options de configuration redondantes « plein écran » des plugins (par exemple, OAuth2, OIDC).
Cela modifie des parties assez critiques de notre code d’authentification, donc la branche est maintenant déployée sur Meta pour test.
Si vous avez un moment à consacrer, veuillez essayer de vous connecter en utilisant plusieurs fournisseurs différents, et de déconnecter/reconnecter des comptes dans vos préférences. Si vous rencontrez des problèmes, faites-le-moi savoir et je m’en occuperai
C’est aussi une excellente occasion d’essayer notre nouvelle authentification Discord, configurée ici sur Meta
Connexion testée avec Facebook, Google, Twitter et Discord, toutes les tentatives ont réussi. J’ai également testé la déconnexion puis la reconnexion des mêmes comptes depuis les préférences utilisateur. Les deux processus se sont déroulés sans problème.
Je ne peux pas tester GitHub car j’ai l’authentification à deux facteurs (2FA) activée sur le compte Discourse lié à mon compte GitHub.
J’ai effectué des tests supplémentaires (les tests ci-dessus ont tous été réalisés dans Chrome sous Windows). J’ai confirmé que la connexion fonctionne également avec IE, Edge et Firefox sous Windows. J’ai découvert deux bugs mineurs, l’un lié à la déconnexion et l’autre à la connexion du compte dans IE, mais il a été confirmé que les deux existaient avant cette modification.
C’était un problème de mise en cache qui ne concernait que IE11. Il devrait être résolu par ce commit. Nous déployons cette correction sur Meta maintenant
