### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [Septembre 9, 2019, 1:49 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/1 "2019-09-09T01:49:02Z")
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 ?
@codinghorror / @david / @eviltrout ?
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Septembre 9, 2019, 8:48 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/2 "2019-09-09T08:48:45Z")
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 ?
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [Septembre 9, 2019, 8:49 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/3 "2019-09-09T08:49:18Z")
Je serais ravi 😁
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Septembre 12, 2019, 2:30 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/6 "2019-09-12T14:30:23Z")
J’ai créé une PR pour cela
>
>
> Using popups is becoming increasingly rare. Full page redirects are already used… on mobile, and for some providers. This commit removes all logic related to popup authentication, leaving only the full page redirect method.
>
> For more info, see https://meta.discourse.org/t/do-we-need-popups-for-login/127988
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).
---
### Author:  [@sam](https://meta.discourse.org/u/sam)
#### Post date: [Septembre 12, 2019, 8:33 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/7 "2019-09-12T20:33:07Z")
> [@david](#):
>
> 267 suppressions
Superbe… 👍🏼 🎰 🤗
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Septembre 13, 2019, 10:48 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/8 "2019-09-13T10:48:56Z")
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 🎉
---
### Author:  [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [Septembre 13, 2019, 10:55 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/9 "2019-09-13T10:55:12Z")
@jomaxro pourrais-tu aider à tester ?
---
### Author:  [@jomaxro](https://meta.discourse.org/u/jomaxro)
#### Post date: [Septembre 13, 2019, 10:55 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/10 "2019-09-13T10:55:58Z")
Bien sûr ! Je vais essayer tous les comptes que j’ai.
---
### Author:  [@jomaxro](https://meta.discourse.org/u/jomaxro)
#### Post date: [Septembre 13, 2019, 11:01 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/11 "2019-09-13T11:01:48Z")
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.
---
### Author:  [@jomaxro](https://meta.discourse.org/u/jomaxro)
#### Post date: [Septembre 13, 2019, 11:33 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/12 "2019-09-13T11:33:07Z")
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.
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Septembre 13, 2019, 4:41 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/13 "2019-09-13T16:41:07Z")
Merci pour les tests, @jomaxro.
> [@jomaxro](#):
>
> un problème lié à la déconnexion
La PR est [ici](https://github.com/discourse/discourse/pull/8096#issuecomment-531306773). Il s’agit d’un bug central, aggravé par le fait que [Facebook insiste pour ajouter un hachage à l’URL de rappel](https://developers.facebook.com/bugs/318390728250352/).
> [@jomaxro](#):
>
> un problème de connexion de compte dans IE
C’était un problème de mise en cache qui ne concernait que IE11. Il devrait être résolu par [ce commit](https://github.com/discourse/discourse/commit/b0211772cb38c114ba2eb0b572428c9093fa1b3d). Nous déployons cette correction sur Meta maintenant 🕰
---
### Author:  [@eviltrout](https://meta.discourse.org/u/eviltrout)
#### Post date: [Septembre 13, 2019, 6:50 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/14 "2019-09-13T18:50:15Z")
Ça a l’air super ! Peut-être pourrions-nous fusionner après la prochaine version bêta.
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Septembre 16, 2019, 11:39 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/15 "2019-09-16T11:39:07Z")
Un sujet a été scindé en un nouveau sujet : [Google One-Tap Sign In](https://meta.discourse.org/t/google-one-tap-sign-in/128679)
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Octobre 8, 2019, 11:11 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/16 "2019-10-08T11:11:12Z")
Ceci est désormais fusionné
>
>
> Using popups is becoming increasingly rare. Full page redirects are already used… on mobile, and for some providers. This commit removes all logic related to popup authentication, leaving only the full page redirect method.
>
> For more info, see https://meta.discourse.org/t/do-we-need-popups-for-login/127988
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [Octobre 14, 2019, 5:00 UTC](https://meta.discourse.org/t/do-we-need-popups-for-login/127988/17 "2019-10-14T17:00:06Z")
Ce sujet a été automatiquement fermé après 6 jours. De nouvelles réponses ne sont plus autorisées.